Paths

Table of Contentst

Differential D25890

Fix PHP 8.1 "ltrim(null)" exception creating a Nuance Source
ClosedPublic
Actions

Authored by aklapper on Feb 18 2025, 23:29.

Tags

None

Referenced Files

	F3720171: D25890.1745700222.diff
	Fri, Apr 25, 20:43

	F3717115: D25890.1745695345.diff
	Fri, Apr 25, 19:22

	F3711839: D25890.1745659716.diff
	Fri, Apr 25, 09:28

	F3709167: D25890.1745644379.diff
	Fri, Apr 25, 05:12

	F3702920: D25890.1745629129.diff
	Fri, Apr 25, 00:58

	F3695605: D25890.1745594116.diff
	Thu, Apr 24, 15:15

	F3605126: D25890.1745252586.diff
	Sun, Apr 20, 16:23

	F3603246: D25890.1745165411.diff
	Sat, Apr 19, 16:10

View All 63 Files

Subscribers

valerio.bozzolan

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T16000: PHP 8.1 "ltrim(null)" exception creating a Nuance source

Commits

rP5b8713889487: Fix PHP 8.1 "ltrim(null)" exception creating a Nuance Source

Summary

Passing null as the first parameter to ltrim() is deprecated since PHP 8.1.
Thus change the optional parameter of the function from null to an empty string.

ERROR 8192: ltrim(): Passing null to parameter #1 ($string) of type string is deprecated at [/var/www/html/phorge/src/applications/nuance/source/NuanceSourceDefinition.php:211]

Closes T16000

Test Plan

Create a Nuance queue at /nuance/queue/edit/form/default/
Create a Nuance "Web Form" type source at /nuance/source/edit/form/default/

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Feb 18 2025, 23:29

Owners added a reviewer: O1: Blessed Committers.Feb 18 2025, 23:29

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptFeb 18 2025, 23:29

aklapper requested review of this revision.Feb 18 2025, 23:29

Harbormaster completed remote builds in B1730: Diff 2678.Feb 18 2025, 23:29

Another option is just declaring $path = '' as default in 209 maybe

This revision is now accepted and ready to land.Feb 19 2025, 08:11

Do the easier and more obvious thing that Valerio proposed

Harbormaster completed remote builds in B1737: Diff 2685.Feb 20 2025, 18:07

aklapper edited the summary of this revision. (Show Details)Feb 20 2025, 18:09

Closed by commit rP5b8713889487: Fix PHP 8.1 "ltrim(null)" exception creating a Nuance Source. · Explain WhyFeb 20 2025, 18:10

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP5b8713889487: Fix PHP 8.1 "ltrim(null)" exception creating a Nuance Source.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

nuance/

source/

NuanceSourceDefinition.php

2 lines

O1: Blessed Committers

Diff 2686

src/applications/nuance/source/NuanceSourceDefinition.php

Loading...