From 44dee47c288202a232be02d893fbf125e1791714 Mon Sep 17 00:00:00 2001 From: J Rhodes Date: Sat, 20 Jun 2015 14:22:23 +1000 Subject: [PATCH] Show build plan name on Harbormaster plan view controller Summary: Ref T8096. This shows the build plan name on the Harbormaster build plan view controller. Without this, the name is not displayed anywhere on the page when you're viewing a build plan's configuration (which makes things confusing if you're updating a bunch of build plans at once). Test Plan: Viewed a build plan, saw the build plan name on the page. Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: epriestley, Korvin Projects: #harbormaster Maniphest Tasks: T8096 Differential Revision: https://secure.phabricator.com/D13356 --- .../controller/HarbormasterPlanViewController.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/applications/harbormaster/controller/HarbormasterPlanViewController.php b/src/applications/harbormaster/controller/HarbormasterPlanViewController.php index 7447f2ed7b..8db8eaf118 100644 --- a/src/applications/harbormaster/controller/HarbormasterPlanViewController.php +++ b/src/applications/harbormaster/controller/HarbormasterPlanViewController.php @@ -27,10 +27,10 @@ final class HarbormasterPlanViewController extends HarbormasterPlanController { new HarbormasterBuildPlanTransactionQuery()); $timeline->setShouldTerminate(true); - $title = pht('Plan %d', $id); + $title = $plan->getName(); $header = id(new PHUIHeaderView()) - ->setHeader($title) + ->setHeader($plan->getName()) ->setUser($viewer) ->setPolicyObject($plan);