From 45a5ea7bf504ca261b5d7c6e76dcb0712c7397c7 Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 29 Sep 2015 09:51:06 -0700 Subject: [PATCH] Show lease owner in Drydock UI Summary: Replaces D13687. Leases track an owner but don't currently show it. Test Plan: Looked at a lease. {F851223} Reviewers: chad Reviewed By: chad Differential Revision: https://secure.phabricator.com/D14191 --- .../drydock/controller/DrydockLeaseViewController.php | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/applications/drydock/controller/DrydockLeaseViewController.php b/src/applications/drydock/controller/DrydockLeaseViewController.php index f37ac1a376..af893ca49b 100644 --- a/src/applications/drydock/controller/DrydockLeaseViewController.php +++ b/src/applications/drydock/controller/DrydockLeaseViewController.php @@ -119,6 +119,14 @@ final class DrydockLeaseViewController extends DrydockLeaseController { pht('Resource Type'), $lease->getResourceType()); + $owner_phid = $lease->getOwnerPHID(); + if ($owner_phid) { + $owner_display = $viewer->renderHandle($owner_phid); + } else { + $owner_display = phutil_tag('em', array(), pht('No Owner')); + } + $view->addProperty(pht('Owner'), $owner_display); + $resource_phid = $lease->getResourcePHID(); if ($resource_phid) { $resource_display = $viewer->renderHandle($resource_phid);