From b642359eb64d3c3b5a06834d2a47bd0989b110d2 Mon Sep 17 00:00:00 2001 From: Fabian Stelzer Date: Tue, 30 Dec 2014 07:00:52 -0800 Subject: [PATCH] add missing needprojectphids for project reports Summary: Adds a missing needProjectPHIDs called in the project reports. Fixes T6833 Test Plan: visited all task reports Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley Maniphest Tasks: T6833 Differential Revision: https://secure.phabricator.com/D11086 --- .../controller/ManiphestReportController.php | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/applications/maniphest/controller/ManiphestReportController.php b/src/applications/maniphest/controller/ManiphestReportController.php index ba2c227bf5..e5c94ee368 100644 --- a/src/applications/maniphest/controller/ManiphestReportController.php +++ b/src/applications/maniphest/controller/ManiphestReportController.php @@ -700,10 +700,17 @@ final class ManiphestReportController extends ManiphestController { $ids = ipull($rows, 'id'); - return id(new ManiphestTaskQuery()) + $query = id(new ManiphestTaskQuery()) ->setViewer($this->getRequest()->getUser()) - ->withIDs($ids) - ->execute(); + ->withIDs($ids); + + switch ($this->view) { + case 'project': + $query->needProjectPHIDs(true); + break; + } + + return $query->execute(); } /**