From 6ea96326237e5c6e682904f246db1f449958ca10 Mon Sep 17 00:00:00 2001
From: Gisle Aune <dev@gisle.me>
Date: Sun, 17 Mar 2019 20:00:49 +0100
Subject: [PATCH] Comments are pretty much done!

---
 marko/components/modal/component.js           |   3 +
 .../components/chapter-meta/component.js      |   7 +
 .../components/chapter-meta/index.marko       |   4 +
 .../components/chapter/component.js           |  28 ++++
 .../components/chapter/index.marko            |   3 +
 .../components/comment/index.marko            |  23 +--
 .../create-comment-modal/component.js         |  53 ++++++-
 .../create-comment-modal/index.marko          |  10 +-
 .../edit-comment-modal/component.js           | 131 ++++++++++++++++++
 .../components/edit-comment-modal/index.marko |  27 ++++
 .../components/page/component.js              |   8 +-
 .../remove-comment-modal/component.js         |  36 +++++
 .../remove-comment-modal/index.marko          |   9 ++
 rpdata/api/Comment.js                         |  50 ++++++-
 14 files changed, 365 insertions(+), 27 deletions(-)
 create mode 100644 marko/page/story-content/components/edit-comment-modal/component.js
 create mode 100644 marko/page/story-content/components/edit-comment-modal/index.marko
 create mode 100644 marko/page/story-content/components/remove-comment-modal/component.js
 create mode 100644 marko/page/story-content/components/remove-comment-modal/index.marko

diff --git a/marko/components/modal/component.js b/marko/components/modal/component.js
index a94ddcb..39a30ba 100644
--- a/marko/components/modal/component.js
+++ b/marko/components/modal/component.js
@@ -47,6 +47,9 @@ module.exports = class {
     if (this.input.enabled) {
       window.removeEventListener("keydown", this.onPress)
     }
+
+    this.mounted = false
+    this.opened = false
   }
 
   /**
diff --git a/marko/page/story-content/components/chapter-meta/component.js b/marko/page/story-content/components/chapter-meta/component.js
index 0a423c9..212a0f7 100644
--- a/marko/page/story-content/components/chapter-meta/component.js
+++ b/marko/page/story-content/components/chapter-meta/component.js
@@ -50,6 +50,13 @@ module.exports = class {
 
         break
       }
+
+      case "title": {
+        this.state.color = "color-primary"
+        this.state.text = value
+
+        break
+      }
   
       default: {
         this.state.color = "color-menu"
diff --git a/marko/page/story-content/components/chapter-meta/index.marko b/marko/page/story-content/components/chapter-meta/index.marko
index 9165ff5..f554de8 100644
--- a/marko/page/story-content/components/chapter-meta/index.marko
+++ b/marko/page/story-content/components/chapter-meta/index.marko
@@ -9,6 +9,10 @@
         <h2>${input.character.author}</h2>
         <markdown source=input.character.description />
       </div>
+      <div else-if(input.ukAuthor != null) class="tooltip-content color-highlight-dark color-primary">
+        <h1>(Unknown Character)</h1>
+        <h2>${input.ukAuthor}</h2>
+      </div> 
     </span>
   </div>
 </if>
\ No newline at end of file
diff --git a/marko/page/story-content/components/chapter/component.js b/marko/page/story-content/components/chapter/component.js
index 05929a8..924e20a 100644
--- a/marko/page/story-content/components/chapter/component.js
+++ b/marko/page/story-content/components/chapter/component.js
@@ -8,9 +8,37 @@ module.exports = class {
         Message: "Message", 
         Chat: "Chat Message",
       },
+      selectedComment: null,
     }
   }
 
+  onOpenComment(comment, modal) {
+    this.state.selectedComment = comment
+    this.state.modal = modal
+  }
+
+  addComment(comment) {
+    this.updateChapter({
+      comments: this.input.chapter.comments.concat(comment),
+    })
+  }
+
+  editComment(comment) {
+    const comments = this.input.chapter.comments.slice()
+    const index = comments.findIndex(c => c.id === comment.id)
+    if (index != -1) {
+      comments[index] = comment
+    } else {
+      comments.push(comment)
+    }
+
+    this.updateChapter({comments})
+  }
+
+  removeComment(comment) {
+    this.updateChapter({comments: this.input.chapter.comments.filter(c => c.id !== comment.id)})
+  }
+
   updateChapter(data) {
     this.emit("edit", data)
   }
diff --git a/marko/page/story-content/components/chapter/index.marko b/marko/page/story-content/components/chapter/index.marko
index 3b7465d..304e7ad 100644
--- a/marko/page/story-content/components/chapter/index.marko
+++ b/marko/page/story-content/components/chapter/index.marko
@@ -25,6 +25,7 @@
       locked=input.chapter.commentsLocked 
       mode=input.chapter.commentMode 
       user=input.user
+      on-open("onOpenComment", comment)
     />
     <if-permitted if(input.chapter.canComment) user=input.user author=input.chapter.author permission="chapter.edit">
       <div class="add-comment color-menu" on-click("open", "addComment") >Add ${state.commentLabels[input.chapter.commentMode] || "Comment"}</div>
@@ -32,6 +33,8 @@
   </div>
   <edit-chapter-modal enabled=(state.modal === "edit") chapter=input.chapter on-close("close") on-edit("updateChapter") />
   <create-comment-modal enabled=(state.modal === "addComment") chapter=input.chapter user=input.user on-close("close") on-add("addComment") />
+  <edit-comment-modal enabled=(state.modal === "editComment") chapter=input.chapter comment=state.selectedComment user=input.user on-close("close") on-edit("editComment") />
+  <remove-comment-modal enabled=(state.modal === "removeComment") comment=state.selectedComment user=input.user on-close("close") on-remove("removeComment", state.selectedComment) />
   <move-chapter-modal enabled=(state.modal === "move") chapter=input.chapter user=input.user on-close("close") on-move("moveChapter") />
   <remove-chapter-modal enabled=(state.modal === "remove") chapter=input.chapter on-close("close") on-remove("removeChapter") />
 </div>
\ No newline at end of file
diff --git a/marko/page/story-content/components/comment/index.marko b/marko/page/story-content/components/comment/index.marko
index 2cbbe5b..6419aa2 100644
--- a/marko/page/story-content/components/comment/index.marko
+++ b/marko/page/story-content/components/comment/index.marko
@@ -5,10 +5,10 @@ import moment from "moment"
     <div class="metadata">
       <div class="options color-menu">
         <if-permitted user=input.user author=input.comment.author permission="story.edit">
-          <a on-click("open", "editComment", input.comment) >Edit</a>
+          <a on-click("emit", "open", "editComment", input.comment) >Edit</a>
         </if-permitted>
         <if-permitted user=input.user author=input.comment.author permission="story.edit">
-          <a on-click("open", "removeComment", input.comment) >Remove</a>
+          <a on-click("emit", "open", "removeComment", input.comment) >Remove</a>
         </if-permitted>
       </div>
       <chapter-meta weak kind="date" value=input.comment.createdDate />
@@ -34,7 +34,14 @@ import moment from "moment"
                 </div>
               </div>
               <div else>
-                <span>${input.comment.characterName}</span>
+                <span class="tooltip color-primary">
+                  <span>${input.comment.characterName}</span>
+
+                  <div class="tooltip-content color-highlight-dark color-primary">
+                    <h1>(Unknown Character)</h1>
+                    <h2>${input.comment.author}</h2>
+                  </div>
+                </span>
               </div>
             </td>
           </tr>
@@ -51,10 +58,10 @@ import moment from "moment"
     <div class="content">
       <div class="options color-menu">
         <if-permitted user=input.user author=input.comment.author permission="story.edit">
-          <a on-click("open", "editComment", input.comment) >Edit</a>
+          <a on-click("emit", "open", "editComment", input.comment) >Edit</a>
         </if-permitted>
         <if-permitted user=input.user author=input.comment.author permission="story.edit">
-          <a on-click("open", "removeComment", input.comment) >Remove</a>
+          <a on-click("emit", "open", "removeComment", input.comment) >Remove</a>
         </if-permitted>
       </div>
       <for (paragraph in state.paragraphs)>
@@ -91,14 +98,14 @@ import moment from "moment"
     <div class="metadata">
       <div class="options color-menu">
         <if-permitted user=input.user author=input.comment.author permission="story.edit">
-          <a on-click("open", "editComment", input.comment) >Edit</a>
+          <a on-click("emit", "open", "editComment", input.comment) >Edit</a>
         </if-permitted>
         <if-permitted user=input.user author=input.comment.author permission="story.edit">
-          <a on-click("open", "removeComment", input.comment) >Remove</a>
+          <a on-click("emit", "open", "removeComment", input.comment) >Remove</a>
         </if-permitted>
       </div>
       <chapter-meta kind="title" value=input.comment.subject />
-      <chapter-meta kind="name" value=input.comment.characterName character=input.comment.character />
+      <chapter-meta kind="name" value=input.comment.characterName character=(input.comment.character) ukAuthor=input.comment.author />
       <chapter-meta kind="date" value=input.comment.fictionalDate />
       <chapter-meta weak kind="date" value=input.comment.createdDate />
       <chapter-meta weak kind="author" value=input.comment.author />
diff --git a/marko/page/story-content/components/create-comment-modal/component.js b/marko/page/story-content/components/create-comment-modal/component.js
index 806f04f..3617ce1 100644
--- a/marko/page/story-content/components/create-comment-modal/component.js
+++ b/marko/page/story-content/components/create-comment-modal/component.js
@@ -17,6 +17,7 @@ module.exports = class {
       },
       characters: [],
       opened: false,
+      placeholder: "(Required)",
     }
 
     this.first = false
@@ -26,10 +27,30 @@ module.exports = class {
     charactersApi.list({author: (this.input.user||{}).name}).then((characters) => {
       this.state.characters = characters.sort((a,b) => a.name.localeCompare(b.name))
     })
+
+    const comments = this.input.chapter.comments || []
+    const lastComment = comments[comments.length - 1]
+    let date = lastComment.fictionalDate || chapter.fictionalDate
+    if (date != null) {
+      this.change("fictionalDate", {target: {value: moment(date).format("MMM D, YYYY")}})
+    }
   }
 
   change(key, ev) {
-    this.state.values = Object.assign({}, this.state.values, {[key]: ev.target.value})    
+    this.state.values = Object.assign({}, this.state.values, {[key]: ev.target.value})
+
+    if (key === "characterId") {
+      if (ev.target.value !== "") {
+        const character = this.state.characters.find(c => c.id === this.state.values.characterId)
+        if (character != null) {
+          this.state.placeholder = character.name
+        } else {
+          this.state.placeholder = "(Required)"
+        }
+      } else {
+        this.state.placeholder = "(Required)"
+      }
+    }
   }
 
   open() {
@@ -47,13 +68,28 @@ module.exports = class {
   }
 
   save() {
-    const values = this.state.values
+    const values = Object.assign({}, this.state.values)
 
-    if (values.source.length < 1) {
-      this.state.error = "You cannot post an empty comment."
+    if (values.source.length == 0) {
+      this.state.error = "The comment body is empty."
       console.warn(values)
       return
     }
+
+    if (this.input.chapter.commentMode !== "Chat" && values.subject.trim().length == 0) {
+      this.state.error = "The subject is empty."
+      return
+    }
+
+    if (values.characterName.trim().length < 1) {
+      const character = this.state.characters.find(c => c.id === this.state.values.characterId)
+      if (character == null) {
+        this.state.error = "You need to give a display name for an anonymous character."
+        return
+      }
+
+      values.characterName = character.name
+    }
     
     let fictionalDate = new Date(values.fictionalDate + " UTC")
     if (values.fictionalDate != "") {
@@ -74,8 +110,12 @@ module.exports = class {
       fictionalDate: fictionalDate, 
     }
 
-    commentApi.addComment(input).then(chapter => {
-      this.emit("add", chapter)
+    if (input.characterId == "") {
+      input.characterId = null
+    }
+
+    commentApi.addComment(input).then(comment => {
+      this.emit("add", comment)
       this.emit("close")
 
       this.state.values = {
@@ -85,6 +125,7 @@ module.exports = class {
         characterId: "",
         fictionalDate: "",
       }
+      this.state.opened = false
     }).catch(errs => {
       console.warn("Failed to add comemnt:", errs)
 
diff --git a/marko/page/story-content/components/create-comment-modal/index.marko b/marko/page/story-content/components/create-comment-modal/index.marko
index cba8f24..327f4e9 100644
--- a/marko/page/story-content/components/create-comment-modal/index.marko
+++ b/marko/page/story-content/components/create-comment-modal/index.marko
@@ -1,22 +1,22 @@
 <modal class="modal color-text nolabel" key="modal" enabled=(input.enabled) closable on-close("close") on-open("open") >
-  <h1>Create Chapter</h1>
+  <h1>Add Comment</h1>
 
   <p key="error" class="color-error">${state.error}</p>
 
   <if(input.chapter.commentMode !== "Chat")>
     <label>Subject</label>
-    <input key="subject" autofocus placeholder="(Optional)" class="big" on-change("change", "subject") value=state.values.subject />
+    <input key="subject" autofocus placeholder="" class="big" on-change("change", "subject") value=state.values.subject />
   </if>
 
-  <label>Diaply Name</label>
-  <input key="characterName" autofocus placeholder="(Optional)" on-change("change", "characterName") value=state.values.characterName />
-
   <label>Character ${state.characterId}</label>
   <select on-change("change", "characterId")>
     <option value="" selected=state.values.characterId=="">Anonymous</option>
     <option for(character in state.characters) value=character.id selected=(state.values.characterId==character.id)>${character.name}</option>
   </select>
 
+  <label>Display Name</label>
+  <input key="characterName" autofocus placeholder=state.placeholder on-change("change", "characterName") value=state.values.characterName />
+
   <label>IC Date</label>
   <input key="icdate" placeholder="(Optional)" on-change("change", "fictionalDate") value=state.values.fictionalDate />
 
diff --git a/marko/page/story-content/components/edit-comment-modal/component.js b/marko/page/story-content/components/edit-comment-modal/component.js
new file mode 100644
index 0000000..645c20f
--- /dev/null
+++ b/marko/page/story-content/components/edit-comment-modal/component.js
@@ -0,0 +1,131 @@
+const moment = require("moment")
+
+const {commentApi} = require("../../../../../rpdata/api/Comment")
+const {charactersApi} = require("../../../../../rpdata/api/Character")
+
+module.exports = class {
+  onCreate(input) {
+    this.state = {
+      error: null,
+      loading: false,
+      values: {
+        subject: "",
+        source: "",
+        characterName: "",
+        characterId: "",
+        fictionalDate: "",
+      },
+      characters: [],
+      opened: false,
+      placeholder: "(Required)",
+    }
+  }
+
+  onInput(input) {
+    if (input.comment) {
+      charactersApi.list({author: (input.user||{}).name}).then((characters) => {
+        this.state.characters = characters.sort((a,b) => a.name.localeCompare(b.name))
+      })
+
+      const comment = input.comment
+      this.state.values = {
+        subject: comment.subject,
+        characterId: comment.characterId,
+        characterName: comment.characterName,
+        fictionalDate: moment(comment.fictionalDate).format("MMM D, YYYY"),
+        source: comment.source,
+      }
+    }
+  }
+
+  change(key, ev) {
+    this.state.values = Object.assign({}, this.state.values, {[key]: ev.target.value})
+
+    if (key === "characterId") {
+      if (ev.target.value !== "") {
+        const character = this.state.characters.find(c => c.id === this.state.values.characterId)
+        if (character != null) {
+          this.state.placeholder = character.name
+        } else {
+          this.state.placeholder = "(Required)"
+        }
+      } else {
+        this.state.placeholder = "(Required)"
+      }
+    }
+  }
+
+  close() {
+    this.emit("close")
+  }
+
+  save() {
+    const values = Object.assign({}, this.state.values)
+    let clearFictionalDate = false
+
+    if (values.source.length == 0) {
+      this.state.error = "The comment body is empty."
+      console.warn(values)
+      return
+    }
+
+    if (this.input.chapter.commentMode !== "Chat" && values.subject.trim().length == 0) {
+      this.state.error = "The subject is empty."
+      return
+    }
+
+    if (values.characterName.trim().length < 1) {
+      const character = this.state.characters.find(c => c.id === this.state.values.characterId)
+      if (character == null) {
+        this.state.error = "You need to give a display name for an anonymous character."
+        return
+      }
+
+      values.characterName = character.name
+    }
+    
+    let fictionalDate = new Date(values.fictionalDate + " UTC")
+    if (values.fictionalDate != "") {
+      if (Number.isNaN(fictionalDate)) {
+        this.state.error = `Could not parse ${values.fictionalDate} as date!`
+        return
+      }
+    } else {
+      fictionalDate = null
+      clearFictionalDate = true
+    }
+    
+    const input = {
+      commentId: this.input.comment.id, 
+      subject: values.subject, 
+      source: values.source, 
+      characterName: values.characterName, 
+      characterId: values.characterId,
+      fictionalDate: fictionalDate,
+      clearFictionalDate: clearFictionalDate,
+    }
+
+    if (input.characterId == "") {
+      input.characterId = null
+    }
+
+    commentApi.editComment(input).then(comment => {
+      this.emit("edit", comment)
+      this.close()
+
+      this.state.values = {
+        subject: "",
+        source: "",
+        characterName: "",
+        characterId: "",
+        fictionalDate: "",
+      }
+    }).catch(errs => {
+      console.warn("Failed to add comemnt:", errs)
+
+      this.state.error = "Failed to add comemnt: " + (errs[0]||errs||{}).message || errs
+    }).then(() => {
+      this.state.loading = false
+    })
+  }
+}
\ No newline at end of file
diff --git a/marko/page/story-content/components/edit-comment-modal/index.marko b/marko/page/story-content/components/edit-comment-modal/index.marko
new file mode 100644
index 0000000..ce13863
--- /dev/null
+++ b/marko/page/story-content/components/edit-comment-modal/index.marko
@@ -0,0 +1,27 @@
+<modal class="modal color-text nolabel" key="modal" enabled=(input.enabled) closable on-close("close") >
+  <h1>Edit Comment</h1>
+
+  <p key="error" class="color-error">${state.error}</p>
+
+  <if(input.chapter.commentMode !== "Chat")>
+    <label>Subject</label>
+    <input key="subject" autofocus placeholder="" class="big" on-change("change", "subject") value=state.values.subject />
+  </if>
+
+  <label>Character ${state.characterId}</label>
+  <select on-change("change", "characterId")>
+    <option value="" selected=state.values.characterId=="">Anonymous</option>
+    <option for(character in state.characters) value=character.id selected=(state.values.characterId==character.id)>${character.name}</option>
+  </select>
+
+  <label>Display Name</label>
+  <input key="characterName" autofocus placeholder=state.placeholder on-change("change", "characterName") value=state.values.characterName />
+
+  <label>IC Date</label>
+  <input key="icdate" placeholder="(Optional)" on-change("change", "fictionalDate") value=state.values.fictionalDate />
+
+  <label>Content</label>
+  <textarea key="source" placeholder="" class="tall" on-change("change", "source") value=state.values.source />
+
+  <button disabled=state.loading on-click("save")>Save</button>
+</modal>
\ No newline at end of file
diff --git a/marko/page/story-content/components/page/component.js b/marko/page/story-content/components/page/component.js
index 87c08ff..809f58e 100644
--- a/marko/page/story-content/components/page/component.js
+++ b/marko/page/story-content/components/page/component.js
@@ -36,13 +36,7 @@ module.exports = class {
       return
     }
 
-    for (const key in data) {
-      if (!data.hasOwnProperty(key)) {
-        continue
-      }
-
-      this.state.story.chapters[index][key] = data[key]
-    }
+    this.state.story.chapters[index] = Object.assign({}, this.state.story.chapters[index], data);
 
     this.refreshStory()
   }
diff --git a/marko/page/story-content/components/remove-comment-modal/component.js b/marko/page/story-content/components/remove-comment-modal/component.js
new file mode 100644
index 0000000..f0f9baa
--- /dev/null
+++ b/marko/page/story-content/components/remove-comment-modal/component.js
@@ -0,0 +1,36 @@
+const {commentApi} = require("../../../../../rpdata/api/Comment")
+
+module.exports = class {
+  onCreate(input) {
+    this.state = {
+      error: null,
+      loading: false,
+      values: {
+        title: "",
+        source: "",
+        fictionalDate: "",
+      },
+    }
+  }
+
+  change(key, ev) {
+    this.state.values[key] = ev.target.value
+  }
+
+  close() {
+    this.emit("close")
+  }
+
+  doIt() {
+    commentApi.removeComment({commentId: this.input.comment.id}).then(() => {
+      this.emit("remove")
+      this.emit("close")
+    }).catch(errs => {
+      console.warn("Failed to delete:", errs)
+
+      this.state.error = "Failed to delete: " + errs.message || errs[0].message
+    }).then(() => {
+      this.state.loading = false
+    })
+  }
+}
\ No newline at end of file
diff --git a/marko/page/story-content/components/remove-comment-modal/index.marko b/marko/page/story-content/components/remove-comment-modal/index.marko
new file mode 100644
index 0000000..06c3cf9
--- /dev/null
+++ b/marko/page/story-content/components/remove-comment-modal/index.marko
@@ -0,0 +1,9 @@
+<modal class="modal color-text nolabel" key="modal" enabled=(input.enabled) notification closable on-close("close") >
+  <h1>Delete Comment</h1>
+
+  <p class="color-error">${state.error}</p>
+  
+  <p class="color-danger">This is irreversible!</p>
+
+  <button disabled=state.loading on-click("doIt")>${Math.random() > 0.99 ? "Deleet" : "Delete"}</button>
+</modal>
\ No newline at end of file
diff --git a/rpdata/api/Comment.js b/rpdata/api/Comment.js
index 5c8ead7..4d00ac4 100644
--- a/rpdata/api/Comment.js
+++ b/rpdata/api/Comment.js
@@ -107,7 +107,7 @@ class CommentAPI {
           source
         }
       }
-    `, {input}).then(({comment}) => {
+    `, {input}, {permissions:["member", "story.edit"]}).then(({addComment: comment}) => {
       return new Comment(
         comment.id, comment.subject, comment.author,
         comment.characterName, comment.character,
@@ -116,6 +116,54 @@ class CommentAPI {
       )
     })
   }
+
+  /**
+   * @param {any} input 
+   * @returns {Promise<Comment>}
+   */
+  editComment(input) {
+    return query(`
+      mutation EditComment($input: CommentEditInput!) {
+        editComment(input: $input) {
+          id
+          subject
+          author
+          characterName
+          character {
+            id
+            name
+            author
+            description
+          }
+          fictionalDate
+          createdDate
+          editedDate
+          source
+        }
+      }
+    `, {input}, {permissions:["member", "story.edit"]}).then(({editComment: comment}) => {
+      return new Comment(
+        comment.id, comment.subject, comment.author,
+        comment.characterName, comment.character,
+        comment.fictionalDate, comment.createdDate,
+        comment.editedDate, comment.source,
+      )
+    })
+  }
+
+  /**
+   * @param {{commentId:string}} input 
+   * @returns {Promise<{commentId:string}>}
+   */
+  removeComment(input) {
+    return query(`
+      mutation RemoveComment($input: CommentRemoveInput!) {
+        removeComment(input: $input) {
+          id
+        }
+      }
+    `, {input}, {permissions:["member", "story.edit"]})
+  }
 }
 
 module.exports = {Comment, CommentCharacter, commentApi: new CommentAPI}
\ No newline at end of file