Paths

Table of Contentst

Differential D25890

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

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

Tags

None

Referenced Files

	F3248560: D25890.1742261872.diff
	Mon, Mar 17, 01:37

	F3236175: D25890.1742165073.diff
	Sat, Mar 15, 22:44

	F3225331: D25890.1742107271.diff
	Sat, Mar 15, 06:41

	F3222487: D25890.1741919751.diff
	Thu, Mar 13, 02:35

	F3221338: D25890.1741825461.diff
	Wed, Mar 12, 00:24

	F3219802: D25890.1741764269.diff
	Tue, Mar 11, 07:24

	F3215349: D25890.1741637756.diff
	Sun, Mar 9, 20:15

	F3187729: D25890.1741360273.diff
	Thu, Mar 6, 15:11

View All 32 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.Tue, Feb 18, 23:29

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

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

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

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

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

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

Do the easier and more obvious thing that Valerio proposed

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

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

Closed by commit rP5b8713889487: Fix PHP 8.1 "ltrim(null)" exception creating a Nuance Source. · Explain WhyThu, Feb 20, 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...