From d39117798419400ef347594d1a1c1ecf0555f40c Mon Sep 17 00:00:00 2001 From: Alan Huang Date: Wed, 1 Aug 2012 19:03:17 -0700 Subject: [PATCH] Fix Differential flag view bug Summary: See D3125#3 Test Plan: Well, apparently the FB test Phabricator has no other flags in it, so I modified the database by hand... the changed revision was not marked as flagged in Differential. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Maniphest Tasks: T1557 Differential Revision: https://secure.phabricator.com/D3130 --- .../differential/view/DifferentialRevisionListView.php | 1 + 1 file changed, 1 insertion(+) diff --git a/src/applications/differential/view/DifferentialRevisionListView.php b/src/applications/differential/view/DifferentialRevisionListView.php index f1c39ed0b2..53df9b23e1 100644 --- a/src/applications/differential/view/DifferentialRevisionListView.php +++ b/src/applications/differential/view/DifferentialRevisionListView.php @@ -68,6 +68,7 @@ final class DifferentialRevisionListView extends AphrontView { } $flags = id(new PhabricatorFlagQuery()) + ->withOwnerPHIDs(array($user->getPHID())) ->withObjectPHIDs(mpull($this->revisions, 'getPHID')) ->execute(); $flagged = mpull($flags, null, 'getObjectPHID');