Page MenuHomePhorge
Files F2695773

D25140.1734855074.diff
No OneTemporary
Actions

D25140.1734855074.diff
View Options

diff --git a/src/applications/project/menuitem/PhabricatorProjectManageProfileMenuItem.php b/src/applications/project/menuitem/PhabricatorProjectManageProfileMenuItem.php
--- a/src/applications/project/menuitem/PhabricatorProjectManageProfileMenuItem.php
+++ b/src/applications/project/menuitem/PhabricatorProjectManageProfileMenuItem.php
@@ -31,7 +31,7 @@
PhabricatorProfileMenuItemConfiguration $config) {
$name = $config->getMenuItemProperty('name');
- if (strlen($name)) {
+ if (phutil_nonempty_string($name)) {
return $name;
}
diff --git a/src/applications/project/menuitem/PhabricatorProjectMembersProfileMenuItem.php b/src/applications/project/menuitem/PhabricatorProjectMembersProfileMenuItem.php
--- a/src/applications/project/menuitem/PhabricatorProjectMembersProfileMenuItem.php
+++ b/src/applications/project/menuitem/PhabricatorProjectMembersProfileMenuItem.php
@@ -21,7 +21,7 @@
PhabricatorProfileMenuItemConfiguration $config) {
$name = $config->getMenuItemProperty('name');
- if (strlen($name)) {
+ if (phutil_nonempty_string($name)) {
return $name;
}
diff --git a/src/applications/project/menuitem/PhabricatorProjectSubprojectsProfileMenuItem.php b/src/applications/project/menuitem/PhabricatorProjectSubprojectsProfileMenuItem.php
--- a/src/applications/project/menuitem/PhabricatorProjectSubprojectsProfileMenuItem.php
+++ b/src/applications/project/menuitem/PhabricatorProjectSubprojectsProfileMenuItem.php
@@ -29,7 +29,7 @@
PhabricatorProfileMenuItemConfiguration $config) {
$name = $config->getMenuItemProperty('name');
- if (strlen($name)) {
+ if (phutil_nonempty_string($name)) {
return $name;
}
diff --git a/src/applications/project/menuitem/PhabricatorProjectWorkboardProfileMenuItem.php b/src/applications/project/menuitem/PhabricatorProjectWorkboardProfileMenuItem.php
--- a/src/applications/project/menuitem/PhabricatorProjectWorkboardProfileMenuItem.php
+++ b/src/applications/project/menuitem/PhabricatorProjectWorkboardProfileMenuItem.php
@@ -38,7 +38,7 @@
PhabricatorProfileMenuItemConfiguration $config) {
$name = $config->getMenuItemProperty('name');
- if (strlen($name)) {
+ if (phutil_nonempty_string($name)) {
return $name;
}
diff --git a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
--- a/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
+++ b/src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
@@ -617,7 +617,7 @@
return true;
case PhabricatorTransactions::TYPE_SPACE:
$space_phid = $xaction->getNewValue();
- if (!strlen($space_phid)) {
+ if (!phutil_nonempty_string($space_phid)) {
// If an install has no Spaces or the Spaces controls are not visible
// to the viewer, we might end up with the empty string here instead
// of a strict `null`, because some controller just used `getStr()`

File Metadata

Mime Type
text/plain
Expires
Sun, Dec 22, 08:11 (21 h, 18 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
1025575
Default Alt Text
D25140.1734855074.diff (3 KB)

Event Timeline

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