Page MenuHomePhabricator

D21178.diff
No OneTemporary

D21178.diff

diff --git a/src/applications/differential/parser/DifferentialChangesetParser.php b/src/applications/differential/parser/DifferentialChangesetParser.php
--- a/src/applications/differential/parser/DifferentialChangesetParser.php
+++ b/src/applications/differential/parser/DifferentialChangesetParser.php
@@ -980,10 +980,15 @@
$new_side = $this->isCommentOnRightSideWhenDisplayed($comment);
$line = $comment->getLineNumber();
- if ($new_side) {
- $back_line = $new_backmap[$line];
+
+ // See T13524. Lint inlines from Harbormaster may not have a line
+ // number.
+ if ($line === null) {
+ $back_line = null;
+ } else if ($new_side) {
+ $back_line = idx($new_backmap, $line);
} else {
- $back_line = $old_backmap[$line];
+ $back_line = idx($old_backmap, $line);
}
if ($back_line != $line) {
@@ -1002,7 +1007,6 @@
$comment->setLineNumber($back_line);
$comment->setLineLength(0);
-
}
$start = max($comment->getLineNumber() - $lines_context, 0);

File Metadata

Mime Type
text/plain
Expires
Sun, May 26, 2:53 AM (3 w, 1 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6285527
Default Alt Text
D21178.diff (1 KB)

Event Timeline