Homestyx hydra
Diffusion hydra 747d7db24161

Fix a PHP 8.1/8.2 deprecated use of ltrim and rtrim with a NULL argument

Description

Fix a PHP 8.1/8.2 deprecated use of ltrim and rtrim with a NULL argument

Summary:
These calls were preventing notification servers configuration to be properly initialized.
Indeed, PHP 8.X is stricter concerning
This behavior is deprecated since PHP 8.1. Phorge adopts phutil_nonempty_string() as a replacement.

Fix T15598

Test Plan:
Sign in as an administrator, configure the notification server without filling admin path field,
you shouldn't get both an RuntimeException and a warning indicating that Phorge is unable to connect
to Notification Server but a message indicating that everything is fine.

Reviewers: O1 Blessed Committers, valerio.bozzolan, avivey

Reviewed By: O1 Blessed Committers, valerio.bozzolan, avivey

Subscribers: avivey, Cigaryno, Matthew, valerio.bozzolan, tobiaswiese, speck

Maniphest Tasks: T15598

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

Details

Provenance
bobAuthored on Aug 17 2023, 4:41 AM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:17befe9bca5b: Fix source code viewer background color when using Dark Mode in Diffusion
Branches
Unknown
Tags
Unknown

Event Timeline