Skip to content

Commit a41b857

Browse files
author
lkassianik
committed
Ref T6049, Phurl object view should always display some sort of header.
Summary: Ref T6049, Phurl object view should display Phurl name or Phurl long url as header. Test Plan: - Create Phurl with no name. Header should show long url as header. - Add name to Phurl. Header should be new Phurl name. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin Maniphest Tasks: T6049 Differential Revision: https://secure.phabricator.com/D14502
1 parent 2e09a93 commit a41b857

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/applications/phurl/controller/PhabricatorPhurlURLViewController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ private function buildHeaderView(PhabricatorPhurlURL $url) {
7979

8080
$header = id(new PHUIHeaderView())
8181
->setUser($viewer)
82-
->setHeader($url->getName())
82+
->setHeader($url->getDisplayName())
8383
->setStatus($icon, $color, $status)
8484
->setPolicyObject($url);
8585

src/applications/phurl/storage/PhabricatorPhurlURL.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public function getDisplayName() {
8383
if ($this->getName()) {
8484
return $this->getName();
8585
} else {
86-
return $this->getMonogram();
86+
return $this->getLongURL();
8787
}
8888
}
8989

0 commit comments

Comments
 (0)