Changeset View
Changeset View
Standalone View
Standalone View
src/view/page/PhabricatorStandardPageView.php
Show First 20 Lines • Show All 182 Lines • ▼ Show 20 Lines | if ($use_glyph) { | ||||
$prefix = $this->getGlyph(); | $prefix = $this->getGlyph(); | ||||
} else { | } else { | ||||
$application_name = $this->getApplicationName(); | $application_name = $this->getApplicationName(); | ||||
if (strlen($application_name)) { | if (strlen($application_name)) { | ||||
$prefix = '['.$application_name.']'; | $prefix = '['.$application_name.']'; | ||||
} | } | ||||
} | } | ||||
if (strlen($prefix)) { | if (phutil_nonempty_string($prefix)) { | ||||
valerio.bozzolan: ✅ OK since the expected input domain is **NULL** (default) or a **string**. The phutil will… | |||||
$title = $prefix.' '.$title; | $title = $prefix.' '.$title; | ||||
} | } | ||||
return $title; | return $title; | ||||
} | } | ||||
protected function willRenderPage() { | protected function willRenderPage() { | ||||
▲ Show 20 Lines • Show All 715 Lines • Show Last 20 Lines |
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0
✅ OK since the expected input domain is NULL (default) or a string. The phutil will throw for alien types, and it's OK.