revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-18 19:40:55 +01:00
Code Issues Releases Wiki Activity

Correctly display event id and sequence in crumb and page title on eventviewcontroller

Browse source 
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
This commit is contained in:
lkassianik 2015-06-08 11:05:54 -07:00
parent c3b11439f2
commit 973101cae6
1 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/applications/calendar/controller/PhabricatorCalendarEventViewController.php
View file

@ -46,12 +46,19 @@ final class PhabricatorCalendarEventViewController
} else if ($sequence) { } else if ($sequence) {
return new Aphront404Response(); return new Aphront404Response();
} }
}
$title = 'E'.$event->getID(); $title = $event->getMonogram().' ('.$sequence.')';
$page_title = $title.' '.$event->getName(); $page_title = $title.' '.$event->getName();
$crumbs = $this->buildApplicationCrumbs(); $crumbs = $this->buildApplicationCrumbs();
$crumbs->addTextCrumb($title, '/E'.$event->getID()); $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( $timeline = $this->buildTransactionTimeline(
$event, $event,