revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-03-28 12:08:14 +01:00
Code Issues Releases Wiki Activity

Fix title for phabricator home page

Browse source 
Summary:
My browser was showing the url of phabricator as the title instead of the real
title. Fix it so the title set for the home page actually gets used.

Test Plan: load phabricator and see "Bacon Ice Cream for Breakfast" as the title

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin, chad

Differential Revision: https://secure.phabricator.com/D5428
This commit is contained in:
Nick Harper 2013-03-26 15:40:14 -07:00
parent 42e875e5bc
commit d2b7bb9504
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/base/controller/PhabricatorController.php
View file

@ -148,9 +148,9 @@ abstract class PhabricatorController extends AphrontController {
$page = $this->buildStandardPageView();
$application = $this->getCurrentApplication();
$page->setTitle(idx($options, 'title'));
if ($application) {
$page->setApplicationName($application->getName());
$page->setTitle(idx($options, 'title'));
if ($application->getTitleGlyph()) {
$page->setGlyph($application->getTitleGlyph());
}

6
src/applications/directory/controller/PhabricatorDirectoryMainController.php
View file

@ -65,10 +65,14 @@ final class PhabricatorDirectoryMainController
$nav->appendChild($content);
$nav->appendChild(new PhabricatorGlobalUploadTargetView());
$title = PhabricatorEnv::getEnvConfig('phabricator.serious-business') ?
'Phabricator' :
pht('Bacon Ice Cream for Breakfast');
return $this->buildApplicationPage(
$nav,
array(
'title' => 'Phabricator',
'title' => $title,
'device' => true,
));
}