Cosmetic changes to let setup

app/assets/javascripts/diffs/store/mutations.js

@@ -86,14 +86,14 @@ export default {
 
   [types.SET_LINE_DISCUSSIONS_FOR_FILE](state, { fileHash, discussions }) {
     const selectedFile = state.diffFiles.find(f => f.fileHash === fileHash);
-    if (selectedFile) {
+    const firstDiscussion = discussions[0];
     const targetLine = selectedFile.parallelDiffLines.find(
       line =>
-          (line.left && line.left.lineCode === discussions[0].line_code) ||
-          (line.right && line.right.lineCode === discussions[0].line_code),
+        (line.left && line.left.lineCode === firstDiscussion.line_code) ||
+        (line.right && line.right.lineCode === firstDiscussion.line_code),
     );
     if (targetLine) {
-        if (targetLine.left && targetLine.left.lineCode === discussions[0].line_code) {
+      if (targetLine.left && targetLine.left.lineCode === firstDiscussion.line_code) {
         Object.assign(targetLine.left, {
           discussions,
         });
@@ -106,7 +106,7 @@ export default {
 
     if (selectedFile.highlightedDiffLines) {
       const targetInlineLine = selectedFile.highlightedDiffLines.find(
-          line => line.lineCode === discussions[0].line_code,
+        line => line.lineCode === firstDiscussion.line_code,
       );
 
       if (targetInlineLine) {
@@ -115,28 +115,22 @@ export default {
         });
       }
     }
-    }
   },
 
   [types.REMOVE_LINE_DISCUSSIONS_FOR_FILE](state, { fileHash, lineCode }) {
     const selectedFile = state.diffFiles.find(f => f.fileHash === fileHash);
-    if (selectedFile) {
     const targetLine = selectedFile.parallelDiffLines.find(
       line =>
         (line.left && line.left.lineCode === lineCode) ||
         (line.right && line.right.lineCode === lineCode),
     );
     if (targetLine) {
-        if (targetLine.left && targetLine.left.lineCode === lineCode) {
-          Object.assign(targetLine.left, {
-            discussions: [],
-          });
-        } else {
-          Object.assign(targetLine.right, {
+      const side = targetLine.left && targetLine.left.lineCode === lineCode ? 'left' : 'right';
+
+      Object.assign(targetline[side], {
         discussions: [],
       });
     }
-      }
 
     if (selectedFile.highlightedDiffLines) {
       const targetInlineLine = selectedFile.highlightedDiffLines.find(
@@ -149,6 +143,5 @@ export default {
         });
       }
     }
-    }
   },
 };

app/assets/javascripts/diffs/store/utils.js

@@ -186,14 +186,12 @@ export function trimFirstCharOfLineContent(line = {}) {
 export function prepareDiffData(diffData) {
   const filesLength = diffData.diffFiles.length;
   let showingLines = 0;
-  let i;
-  for (i = 0; i < filesLength; i += 1) {
+  for (let i = 0; i < filesLength; i += 1) {
     const file = diffData.diffFiles[i];
 
     if (file.parallelDiffLines) {
       const linesLength = file.parallelDiffLines.length;
-      let u = 0;
-      for (u = 0; u < linesLength; u += 1) {
+      for (let u = 0; u < linesLength; u += 1) {
         const line = file.parallelDiffLines[u];
         if (line.left) {
           line.left = trimFirstCharOfLineContent(line.left);
@@ -206,8 +204,7 @@ export function prepareDiffData(diffData) {
 
     if (file.highlightedDiffLines) {
       const linesLength = file.highlightedDiffLines.length;
-      let u;
-      for (u = 0; u < linesLength; u += 1) {
+      for (let u = 0; u < linesLength; u += 1) {
         trimFirstCharOfLineContent(file.highlightedDiffLines[u]);
       }
       showingLines += file.parallelDiffLines.length;

app/assets/javascripts/notes/stores/actions.js

@@ -54,6 +54,7 @@ export const refetchDiscussionById = ({ commit, state }, { path, discussionId })
           commit(types.UPDATE_DISCUSSION, selectedDiscussion);
           // We need to refetch as it is now the transformed one in state
           const discussion = utils.findNoteObjectById(state.discussions, discussionId);
+
           resolve(discussion);
         }
       })