Homestyx hydra
Diffusion hydra 17b568b94f11

Fix PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel…

Description

Fix PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined

Summary:
strlen() was used in Phabricator to check if a generic value is a non-empty string.
This behavior is deprecated since PHP 8.1. Phorge adopts phutil_nonempty_string() as a replacement.

Note: this may highlight other absurd input values that might be worth correcting
instead of just ignoring. If phutil_nonempty_string() throws an exception in your
instance, report it to Phorge to evaluate and fix that specific corner case.

ERROR 8192: strlen(): Passing null to parameter #1 ($string) of type string is deprecated at [/var/www/html/phorge/phorge/src/applications/dashboard/editfield/PhabricatorDashboardQueryPanelQueryEditField.php:41]
ERROR 8192: strlen(): Passing null to parameter #1 ($string) of type string is deprecated at [/var/www/html/phorge/phorge/src/applications/dashboard/editfield/PhabricatorDashboardQueryPanelQueryEditField.php:59]

Closes T15791

Test Plan: Successfully create a Dashboard query panel searching for "Diffusion Raw Commits" (which has no Query defined per T15790) and try to edit it afterwards under PHP 8.1 or later.

Reviewers: O1 Blessed Committers, valerio.bozzolan

Reviewed By: O1 Blessed Committers, valerio.bozzolan

Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Maniphest Tasks: T15791

Differential Revision: https://we.phorge.it/D25596

Details

Provenance
Andre KlapperAuthored on Apr 18 2024, 1:50 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:3349c3be17c7: Fix call to non-existing newWorkCommands() in NuanceItemType
Branches
Unknown
Tags
Unknown

Event Timeline