diff options
author | Mike Greiling <mike@pixelcog.com> | 2017-03-22 10:57:31 +0300 |
---|---|---|
committer | Mike Greiling <mike@pixelcog.com> | 2017-03-24 01:52:57 +0300 |
commit | 670d9a8bb6e786910b717f76657b1d2a7c41f22d (patch) | |
tree | 832f1c4511558a2bcf6e634d0c043ae897ae77bc /app/assets/javascripts/diff.js | |
parent | 324fb743043e61a400e21764e0496b70f3a3f8ed (diff) |
ensure lineNumbers method always returns an array with two integers
Diffstat (limited to 'app/assets/javascripts/diff.js')
-rw-r--r-- | app/assets/javascripts/diff.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/app/assets/javascripts/diff.js b/app/assets/javascripts/diff.js index dca12bbcf10..88180149715 100644 --- a/app/assets/javascripts/diff.js +++ b/app/assets/javascripts/diff.js @@ -33,7 +33,6 @@ class Diff { handleClickUnfold(e) { const $target = $(e.target); - // current babel config relies on iterators implementation, so we cannot simply do: const [oldLineNumber, newLineNumber] = this.lineNumbers($target.parent()); const offset = newLineNumber - oldLineNumber; const bottom = $target.hasClass('js-unfold-bottom'); @@ -102,10 +101,11 @@ class Diff { } lineNumbers(line) { - if (!line.children().length) { + const children = line.find('.diff-line-num').toArray(); + if (children.length !== 2) { return [0, 0]; } - return line.find('.diff-line-num').map((i, elm) => parseInt($(elm).data('linenumber'), 10)); + return children.map(elm => parseInt($(elm).data('linenumber'), 10) || 0); } highlightSelectedLine() { |