revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-03-01 06:59:15 +01:00
Code Issues Releases Wiki Activity

Fix exception for new error stuff in Conduit

Browse source 
Summary: This might be `null`.

Test Plan: Loaded, got exception, applied patch, no exception. Viewed a method with an actual message too, that also worked.

Reviewers: chad, btrahan

Reviewed By: chad

CC: aran

Differential Revision: https://secure.phabricator.com/D7955
This commit is contained in:
epriestley 2014-01-13 15:32:37 -08:00
parent 8a7429b879
commit 3fc807bed1
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/applications/conduit/controller/PhabricatorConduitConsoleController.php
View file

@ -112,9 +112,12 @@ final class PhabricatorConduitConsoleController
$form_box = id(new PHUIObjectBoxView()) $form_box = id(new PHUIObjectBoxView())
->setHeaderText($method->getAPIMethodName()) ->setHeaderText($method->getAPIMethodName())
->setErrorView($status_view)
->setForm($form); ->setForm($form);
if ($status_view) {
$form_box->setErrorView($status_view);
}
$crumbs = $this->buildApplicationCrumbs(); $crumbs = $this->buildApplicationCrumbs();
$crumbs->addTextCrumb($method->getAPIMethodName()); $crumbs->addTextCrumb($method->getAPIMethodName());