From 3fc807bed162e6ce94c7ebcc63a7eb55ca84e0be Mon Sep 17 00:00:00 2001 From: epriestley Date: Mon, 13 Jan 2014 15:32:37 -0800 Subject: [PATCH] Fix exception for new error stuff in Conduit 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 --- .../controller/PhabricatorConduitConsoleController.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/applications/conduit/controller/PhabricatorConduitConsoleController.php b/src/applications/conduit/controller/PhabricatorConduitConsoleController.php index e1b5da5ed0..e6d8d975c3 100644 --- a/src/applications/conduit/controller/PhabricatorConduitConsoleController.php +++ b/src/applications/conduit/controller/PhabricatorConduitConsoleController.php @@ -112,9 +112,12 @@ final class PhabricatorConduitConsoleController $form_box = id(new PHUIObjectBoxView()) ->setHeaderText($method->getAPIMethodName()) - ->setErrorView($status_view) ->setForm($form); + if ($status_view) { + $form_box->setErrorView($status_view); + } + $crumbs = $this->buildApplicationCrumbs(); $crumbs->addTextCrumb($method->getAPIMethodName());