Allow suggestion editing

Author: ilmartyrk

commentManager.js

@@ -207,3 +207,52 @@ exports.changeCommentText = async (padId, commentId, commentText, authorId) => {
   // save the comment updated back
   await db.set(prefix + padId, comments);
 };
+
+exports.changeComment = async (padId, commentId, commentText, changeFrom, changeTo, authorId, state) => { // eslint-disable max-len
+  if (commentText.length <= 0) {
+    logger.debug(`ignoring attempt to change comment ${commentId} to the empty string`);
+    throw new Error('comment_cannot_be_empty');
+  }
+
+  // Given a comment we update the comment text
+
+  // If we're dealing with comment replies we need to a different query
+  let prefix = 'comments:';
+  if (commentId.substring(0, 7) === 'c-reply') {
+    prefix = 'comment-replies:';
+  }
+
+  // get the entry
+  const comments = await db.get(prefix + padId);
+  if (comments == null || comments[commentId] == null) {
+    logger.debug(`ignoring attempt to edit non-existent comment ${commentId}`);
+    throw new Error('no_such_comment');
+  }
+  if (comments[commentId].author !== authorId) {
+    logger.debug(`author ${authorId} attempted to edit comment ${commentId} ` +
+                 `belonging to author ${comments[commentId].author}`);
+    throw new Error('unauth');
+  }
+  // update the comment text
+  comments[commentId].text = commentText;
+  if (changeTo) {
+    comments[commentId].changeTo = changeTo;
+    if (!comments[commentId].changeFrom) {
+      comments[commentId].changeFrom = changeFrom;
+    }
+  }
+
+  if (comments[commentId].changeTo && !changeTo) {
+    comments[commentId].changeTo = null;
+  }
+
+  if (state) {
+    comments[commentId].changeAccepted = true;
+    comments[commentId].changeReverted = false;
+  } else {
+    comments[commentId].changeAccepted = false;
+    comments[commentId].changeReverted = true;
+  }
+  // save the comment updated back
+  await db.set(prefix + padId, comments);
+};

exportHTML.js

@@ -30,7 +30,7 @@ exports.getLineHTMLForExport = async (hookName, context) => {
   let hasPlugin = false;
   // Load the HTML into a throwaway div instead of calling $.load() to avoid
   // https://github.com/cheeriojs/cheerio/issues/1031
-  //const content = $('<div>').html(context.lineContent);
+  // const content = $('<div>').html(context.lineContent);
   // include links for each comment which we will add content later.
   content.find('span').each(function () {
     const span = $(this);

index.js

@@ -137,6 +137,13 @@ exports.socketio = (hookName, args, cb) => {
       socket.broadcast.to(padId).emit('textCommentUpdated', commentId, commentText);
     }));
 
+    socket.on('updateComment', handler(async (data) => {
+      const {commentId, commentText, authorId, changeFrom, changeTo, changeAcceptedState} = data;
+      const {padId} = await readOnlyManager.getIds(data.padId);
+      await commentManager.changeComment(padId, commentId, commentText, changeFrom, changeTo, authorId, changeAcceptedState); // eslint-disable max-len
+      socket.broadcast.to(padId).emit('commentUpdated', commentId, commentText, changeFrom, changeTo); // eslint-disable max-len
+    }));
+
     socket.on('addCommentReply', handler(async (data) => {
       const {padId} = await readOnlyManager.getIds(data.padId);
       const [replyId, reply] = await commentManager.addCommentReply(padId, data);