Skip to content

Commit ba4b8cb

Browse files
committed
PHP 8 and Diviner Technical Documentation
Summary: This also fixes the field Personal Settings > Display Preferences > Monospaced Font, when empty. Close T15621 Test Plan: Generate diviner stuff, navigate to the "Technical Documentation" section (In php 8.1+) Reviewers: O1 Blessed Committers, mturdus, valerio.bozzolan Reviewed By: O1 Blessed Committers, mturdus, valerio.bozzolan Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno Maniphest Tasks: T15621 Differential Revision: https://we.phorge.it/D25412
1 parent 3cc5ee6 commit ba4b8cb

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/applications/diviner/storage/DivinerLiveSymbol.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ public function save() {
172172
public function getTitle() {
173173
$title = parent::getTitle();
174174

175-
if (!strlen($title)) {
175+
if (!phutil_nonempty_string($title)) {
176176
$title = $this->getName();
177177
}
178178

src/applications/settings/setting/PhabricatorMonospacedFontSetting.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ protected function getControlInstructions() {
2525
}
2626

2727
public function validateTransactionValue($value) {
28-
if (!strlen($value)) {
28+
if (!phutil_nonempty_string($value)) {
2929
return;
3030
}
3131

0 commit comments

Comments
 (0)