diff options
| -rw-r--r-- | app/assets/javascripts/diffs/store/mutations.js | 4 | ||||
| -rw-r--r-- | app/assets/javascripts/diffs/store/utils.js | 4 |
2 files changed, 3 insertions, 5 deletions
diff --git a/app/assets/javascripts/diffs/store/mutations.js b/app/assets/javascripts/diffs/store/mutations.js index a11ac2b292b..c7fcbe01704 100644 --- a/app/assets/javascripts/diffs/store/mutations.js +++ b/app/assets/javascripts/diffs/store/mutations.js @@ -86,8 +86,6 @@ export default { }, [types.SET_LINE_DISCUSSIONS_FOR_FILE](state, { fileHash, discussions, diffPositionByLineCode }) { - if (!state.latestDiff) return; - const selectedFile = state.diffFiles.find(f => f.fileHash === fileHash); const firstDiscussion = discussions[0]; const isDiffDiscussion = firstDiscussion.diff_discussion; @@ -99,7 +97,7 @@ export default { isDiffDiscussion && hasLineCode && diffPosition && - isDiscussionApplicableToLine(firstDiscussion, diffPosition) + isDiscussionApplicableToLine(firstDiscussion, diffPosition, state.latestDiff) ) { const targetLine = selectedFile.parallelDiffLines.find( line => diff --git a/app/assets/javascripts/diffs/store/utils.js b/app/assets/javascripts/diffs/store/utils.js index 36053d8db44..17732f4eeb6 100644 --- a/app/assets/javascripts/diffs/store/utils.js +++ b/app/assets/javascripts/diffs/store/utils.js @@ -254,7 +254,7 @@ export function getDiffPositionByLineCode(diffFiles) { // This method will check whether the discussion is still applicable // to the diff line in question regarding different versions of the MR -export function isDiscussionApplicableToLine(discussion, diffPosition) { +export function isDiscussionApplicableToLine(discussion, diffPosition, latestDiff) { const { lineCode, ...diffPositionCopy } = diffPosition; if (discussion.original_position && discussion.position) { @@ -264,5 +264,5 @@ export function isDiscussionApplicableToLine(discussion, diffPosition) { return _.isEqual(refs, diffPositionCopy) || _.isEqual(originalRefs, diffPositionCopy); } - return discussion.active && lineCode === discussion.line_code; + return latestDiff && discussion.active && lineCode === discussion.line_code; } |
