From d41aaba2a184e0f57c68dfd56c95e628d55d8995 Mon Sep 17 00:00:00 2001 From: Mike Riley Date: Mon, 1 Feb 2016 19:04:28 +0000 Subject: [PATCH] Fix coverage line index lookup in diffusion browser Summary: I believe this got clobbered in rP8b6edaa4e238a809fe78e6d14ad0705545f8179f. This index doesn't seem to be present in the line dictionary and we're now relying on `$line_index` for the current position. Test Plan: before {F1085522} after {F1085521} Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley Differential Revision: https://secure.phabricator.com/D15156 --- .../diffusion/controller/DiffusionBrowseController.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/applications/diffusion/controller/DiffusionBrowseController.php b/src/applications/diffusion/controller/DiffusionBrowseController.php index e5d2f7bdfa..a2e9a3a4b3 100644 --- a/src/applications/diffusion/controller/DiffusionBrowseController.php +++ b/src/applications/diffusion/controller/DiffusionBrowseController.php @@ -1180,7 +1180,7 @@ final class DiffusionBrowseController extends DiffusionController { if ($this->coverage) { require_celerity_resource('differential-changeset-view-css'); - $cov_index = $line['line'] - 1; + $cov_index = $line_index; if (isset($this->coverage[$cov_index])) { $cov_class = $this->coverage[$cov_index];