From 7fca451263dba47e13573f8a4a30f0d2df9b7bba Mon Sep 17 00:00:00 2001
From: Belen Curcio <curciobelen@gmail.com>
Date: Wed, 12 Apr 2017 19:21:55 -0300
Subject: [PATCH] Immutable plugins

---
 .../graphql/mutations/index.js                |   4 +-
 client/coral-plugin-commentbox/CommentBox.js  | 143 ++++++++++--------
 .../client/components/OffTopicCheckbox.js     |  38 +++--
 3 files changed, 107 insertions(+), 78 deletions(-)

diff --git a/client/coral-framework/graphql/mutations/index.js b/client/coral-framework/graphql/mutations/index.js
index 16bac9b1a..8b1feed66 100644
--- a/client/coral-framework/graphql/mutations/index.js
+++ b/client/coral-framework/graphql/mutations/index.js
@@ -16,7 +16,7 @@ export const postComment = graphql(POST_COMMENT, {
   props: ({ownProps, mutate}) => ({
     postItem: comment => {
       const {asset_id, body, parent_id, tags = []} = comment;
-
+      console.log(tags)
       return mutate({
         variables: {
           comment
@@ -77,7 +77,7 @@ export const postComment = graphql(POST_COMMENT, {
             return updatedAsset;
           }
         }
-      })
+      });
     }
   }),
 });
diff --git a/client/coral-plugin-commentbox/CommentBox.js b/client/coral-plugin-commentbox/CommentBox.js
index a25dbc76c..c506da0cc 100644
--- a/client/coral-plugin-commentbox/CommentBox.js
+++ b/client/coral-plugin-commentbox/CommentBox.js
@@ -8,49 +8,42 @@ const name = 'coral-plugin-commentbox';
 
 class CommentBox extends Component {
 
-  static propTypes = {
-    commentPostedHandler: PropTypes.func,
-    postItem: PropTypes.func.isRequired,
-    cancelButtonClicked: PropTypes.func,
-    assetId: PropTypes.string.isRequired,
-    parentId: PropTypes.string,
-    authorId: PropTypes.string.isRequired,
-    isReply: PropTypes.bool.isRequired,
-    canPost: PropTypes.bool,
-    currentUser: PropTypes.object
+  constructor(props) {
+    super(props);
+
+    this.state = {
+      username: '',
+      comment: {
+        body: '',
+        tags: []
+      },
+      hooks: {
+        preSubmit: [],
+        postSubmit: []
+      }
+    };
   }
 
-  state = {
-    body: '',
-    username: '',
-    hooks: {
-      preSubmit: [],
-      postSubmit: []
-    }
-  }
+  updateComment = cb => this.setState(cb);
 
   postComment = () => {
     const {
-      commentPostedHandler,
-      postItem,
-      assetId,
-      updateCountCache,
       isReply,
-      countCache,
+      assetId,
       parentId,
+      postItem,
+      countCache,
       addNotification,
-      authorId
+      updateCountCache,
+      commentPostedHandler,
     } = this.props;
 
     let comment = {
-      body: this.state.body,
       asset_id: assetId,
-      parent_id: parentId
+      parent_id: parentId,
+      ...this.state.comment
     };
 
-    if (this.props.charCount && this.state.body.length > this.props.charCount) {
-      return;
-    }
     !isReply && updateCountCache(assetId, countCache + 1);
 
     // Execute preSubmit Hooks
@@ -75,33 +68,37 @@ class CommentBox extends Component {
           commentPostedHandler();
         }
       })
-    .catch((err) => console.error(err));
-    this.setState({body: ''});
+      .catch((err) => console.error(err));
+
+    this.setState({
+      comment: {
+        ...this.state.comment,
+        body: ''
+      }
+    });
   }
 
   registerHook = (hookType = '', hook = () => {}) => {
-    if (typeof hook === 'function') {
-      if (typeof hookType === 'string') {
-        this.setState(state => ({
-          hooks: {
-            ...state.hooks,
-            [hookType]: [
-              ...state.hooks[hookType],
-              hook
-            ]
-          }
-        }));
+    if (typeof hook !== 'function') {
+      return console.warn(`Hooks must be functions. Please check your ${hookType} hooks`);
+    } else if (typeof hookType === 'string') {
+      this.setState(state => ({
+        hooks: {
+          ...state.hooks,
+          [hookType]: [
+            ...state.hooks[hookType],
+            hook
+          ]
+        }
+      }));
 
-        return {
-          hookType,
-          hook
-        };
+      return {
+        hookType,
+        hook
+      };
 
-      } else {
-        console.warn('hookTypes must be a string. Please check your hooks');
-      }
     } else {
-      console.warn(`Hooks must be functions. Please check your ${hookType} hooks`);
+      return console.warn('hookTypes must be a string. Please check your hooks');
     }
   }
 
@@ -129,10 +126,21 @@ class CommentBox extends Component {
     });
   }
 
+  handleChange = e => {
+    this.setState({
+      comment: {
+        ...this.state.comment,
+        body: e.target.value
+      }
+    });
+  }
+
   render () {
     const {styles, isReply, authorId, charCount} = this.props;
     let {cancelButtonClicked} = this.props;
-    const length = this.state.body.length;
+
+    const length = this.state.comment.body.length;
+    const enablePostComment = !length || (charCount && length > charCount);
 
     if (isReply && typeof cancelButtonClicked !== 'function') {
       console.warn('the CommentBox component should have a cancelButtonClicked callback defined if it lives in a Reply');
@@ -151,46 +159,39 @@ class CommentBox extends Component {
           <textarea
             className={`${name}-textarea`}
             style={styles && styles.textarea}
-            value={this.state.body}
+            value={this.state.comment.body}
             placeholder={lang.t('comment')}
             id={isReply ? 'replyText' : 'commentText'}
-            onChange={(e) => this.setState({body: e.target.value})}
+            onChange={this.handleChange}
             rows={3}/>
         </div>
         <div className={`${name}-char-count ${length > charCount ? `${name}-char-max` : ''}`}>
-          {
-            charCount &&
-            `${charCount - length} ${lang.t('characters-remaining')}`
-          }
+          {charCount && `${charCount - length} ${lang.t('characters-remaining')}`}
         </div>
         <div className={`${name}-button-container`}>
-
           <Slot
             fill="commentBoxDetail"
+            updateComment={this.updateComment}
             registerHook={this.registerHook}
             unregisterHook={this.unregisterHook}
             inline
           />
-
           {
             isReply && (
               <Button
                 cStyle='darkGrey'
                 className={`${name}-cancel-button`}
-                onClick={() => {
-                  cancelButtonClicked('');
-                }}>
+                onClick={() => cancelButtonClicked('')}>
                 {lang.t('cancel')}
               </Button>
             )
           }
           { authorId && (
               <Button
-                cStyle={!length || (charCount && length > charCount) ? 'lightGrey' : 'darkGrey'}
+                cStyle={enablePostComment ? 'lightGrey' : 'darkGrey'}
                 className={`${name}-button`}
                 onClick={this.postComment}
-                disabled={!length || (charCount && length > charCount) ? 'disabled' : ''}
-              >
+                disabled={enablePostComment ? 'disabled' : ''}>
                 {lang.t('post')}
               </Button>
             )
@@ -200,6 +201,18 @@ class CommentBox extends Component {
   }
 }
 
+CommentBox.propTypes = {
+  commentPostedHandler: PropTypes.func,
+  postItem: PropTypes.func.isRequired,
+  cancelButtonClicked: PropTypes.func,
+  assetId: PropTypes.string.isRequired,
+  parentId: PropTypes.string,
+  authorId: PropTypes.string.isRequired,
+  isReply: PropTypes.bool.isRequired,
+  canPost: PropTypes.bool,
+  currentUser: PropTypes.object
+};
+
 export default CommentBox;
 
 const lang = new I18n(translations);
diff --git a/plugins/coral-plugin-offtopic/client/components/OffTopicCheckbox.js b/plugins/coral-plugin-offtopic/client/components/OffTopicCheckbox.js
index 84b059d16..8944bee89 100644
--- a/plugins/coral-plugin-offtopic/client/components/OffTopicCheckbox.js
+++ b/plugins/coral-plugin-offtopic/client/components/OffTopicCheckbox.js
@@ -1,21 +1,37 @@
 import React from 'react';
 import styles from './styles.css';
-import {addCommentTag} from 'coral-framework/graphql/mutations';
-import {compose} from 'react-apollo';
 
 class OffTopicCheckbox extends React.Component {
+  constructor() {
+    super();
+    this.label = 'OFF_TOPIC';
+  }
 
   handleChange = (e) => {
     if (e.target.checked) {
-      this.addCommentTagHook = this.props.registerHook('postSubmit', (data) => {
-        const {comment} = data.createComment;
-        this.props.addCommentTag({
-          id: comment.id,
-          tag: 'OFF_TOPIC'
-        });
-      })
+      this.props.updateComment(this.addTag)
     } else {
-      this.props.unregisterHook(this.addCommentTagHook);
+      this.props.updateComment(this.removeTag)
+    }
+  }
+
+  addTag = (state) => ({
+    comment: {
+      ...state.comment,
+      tags: [...state.comment.tags, this.label]
+    }
+  })
+
+  removeTag = (state) => {
+    const idx = state.comment.tags.indexOf(this.label);
+    return {
+      comment: {
+        ...state.comment,
+        tags: [
+          ...state.comment.tags.slice(0, idx),
+          ...state.comment.tags.slice(idx + 1)
+        ]
+      }
     }
   }
 
@@ -31,4 +47,4 @@ class OffTopicCheckbox extends React.Component {
   }
 }
 
-export default compose(addCommentTag)(OffTopicCheckbox);
+export default OffTopicCheckbox;