revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 08:42:41 +01:00
Code Issues Releases Wiki Activity

[diffusion] Fix pending differential revisions list

Browse source 
Summary:
DifferentialRevisionListView requires setFields to be called before
calling getRequiredHandlePHIDs; this adds that call for DiffusionController

Test Plan:
loaded diffusion and saw the "Pending Differential Revisions" section
populated, and no errors in the darkconsole

Reviewers: epriestley, btrahan, jungejason

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1665
This commit is contained in:
Nick Harper 2012-02-21 22:15:13 -08:00
parent af295e0b26
commit 07e5591015
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/applications/diffusion/controller/base/DiffusionController.php
View file

@ -143,6 +143,7 @@ abstract class DiffusionController extends PhabricatorController {
$view = id(new DifferentialRevisionListView()) $view = id(new DifferentialRevisionListView())
->setRevisions($revisions) ->setRevisions($revisions)
->setFields(DifferentialRevisionListView::getDefaultFields())
->setUser($this->getRequest()->getUser()); ->setUser($this->getRequest()->getUser());
$phids = $view->getRequiredHandlePHIDs(); $phids = $view->getRequiredHandlePHIDs();