Commit 86f4e987 authored by Thomas Randolph's avatar Thomas Randolph

Fix incorrect diff file property reference

Before, the diffs *always* had both parallel and inline
diffs, so this should have never hit the else block.
Now that the split diffs feature is enabled, there are
cases (about 50% of the time) where the else is entered.

The test for inline diff lines was then trying to iterate over
the parallel lines, which of course don't exist in a lot of
situations.
parent fa4b8c25
...@@ -498,7 +498,7 @@ export const allDiscussionWrappersExpanded = diff => { ...@@ -498,7 +498,7 @@ export const allDiscussionWrappersExpanded = diff => {
} }
}); });
} else if (diff.highlighted_diff_lines) { } else if (diff.highlighted_diff_lines) {
diff.parallel_diff_lines.forEach(line => { diff.highlighted_diff_lines.forEach(line => {
if (line.discussions.length) { if (line.discussions.length) {
discussionsExpandedArray.push(line.discussionsExpanded); discussionsExpandedArray.push(line.discussionsExpanded);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment