From 973101cae6c71d857ca3f379b678da80fb755b47 Mon Sep 17 00:00:00 2001 From: lkassianik Date: Mon, 8 Jun 2015 11:05:54 -0700 Subject: [PATCH] Correctly display event id and sequence in crumb and page title on eventviewcontroller Summary: Fixes T8459, Correctly display event id and sequence in crumb and page title on eventviewcontroller Test Plan: Open `E111/3`, crumb and title should display '`E111 (3)`' instead of '`E111`'. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin, epriestley Maniphest Tasks: T8459 Differential Revision: https://secure.phabricator.com/D13211 --- .../PhabricatorCalendarEventViewController.php | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/src/applications/calendar/controller/PhabricatorCalendarEventViewController.php b/src/applications/calendar/controller/PhabricatorCalendarEventViewController.php index 2cbc8dcdb2..004c2e0669 100644 --- a/src/applications/calendar/controller/PhabricatorCalendarEventViewController.php +++ b/src/applications/calendar/controller/PhabricatorCalendarEventViewController.php @@ -46,12 +46,19 @@ final class PhabricatorCalendarEventViewController } else if ($sequence) { return new Aphront404Response(); } - } - $title = 'E'.$event->getID(); - $page_title = $title.' '.$event->getName(); - $crumbs = $this->buildApplicationCrumbs(); - $crumbs->addTextCrumb($title, '/E'.$event->getID()); + $title = $event->getMonogram().' ('.$sequence.')'; + $page_title = $title.' '.$event->getName(); + $crumbs = $this->buildApplicationCrumbs(); + $crumbs->addTextCrumb($title, '/'.$event->getMonogram().'/'.$sequence); + + + } else { + $title = 'E'.$event->getID(); + $page_title = $title.' '.$event->getName(); + $crumbs = $this->buildApplicationCrumbs(); + $crumbs->addTextCrumb($title, '/E'.$event->getID()); + } $timeline = $this->buildTransactionTimeline( $event,