From 0ce31211708b873279a5a4fa77e05f326702de65 Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 11 Sep 2013 08:50:39 -0700 Subject: [PATCH] Continue not fataling on legacy Maniphest queries Summary: Ref T3817. See that task for discussion. Auditors: btrahan --- src/applications/maniphest/view/ManiphestTaskListView.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/applications/maniphest/view/ManiphestTaskListView.php b/src/applications/maniphest/view/ManiphestTaskListView.php index b0e6631793..14c5d5dba2 100644 --- a/src/applications/maniphest/view/ManiphestTaskListView.php +++ b/src/applications/maniphest/view/ManiphestTaskListView.php @@ -60,8 +60,11 @@ final class ManiphestTaskListView extends ManiphestView { $item->setHref('/T'.$task->getID()); if ($task->getOwnerPHID()) { - $owner = $handles[$task->getOwnerPHID()]; - $item->addByline(pht('Assigned: %s', $owner->renderLink())); + $owner = idx($handles, $task->getOwnerPHID()); + // TODO: This should be guaranteed, see T3817. + if ($owner) { + $item->addByline(pht('Assigned: %s', $owner->renderLink())); + } } $status = $task->getStatus();