Paths

Table of Contentst

Differential D25226

Fix PHP 8.1 "strlen(null)" exceptions which block rendering Legalpad signatures
ClosedPublic
Actions

Authored by aklapper on Sat, May 13, 11:04.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, May 31, 17:50

	Unknown Object (File)
	Wed, May 31, 06:45

	Unknown Object (File)
	Tue, May 30, 17:12

	Unknown Object (File)
	Tue, May 30, 10:55

	Unknown Object (File)
	Tue, May 30, 08:52

	Unknown Object (File)
	Sat, May 27, 11:42

	Unknown Object (File)
	Mon, May 15, 05:38

Subscribers

valerio.bozzolan

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15395: PHP 8.1 "strlen(null)" exceptions block rendering Legalpad signatures

Commits

rP912a933b0214: Fix PHP 8.1 "strlen(null)" exceptions which block rendering Legalpad signatures

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.

Closes T15395

Test Plan

Applied these two changes; afterwards "All Signatures" page at /legalpad/signatures/1/ gets rendered in web browser.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Sat, May 13, 11:04

Owners added a reviewer: O1: Blessed Committers.Sat, May 13, 11:04

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan and 2 others. · View Herald TranscriptSat, May 13, 11:04

aklapper requested review of this revision.Sat, May 13, 11:04

Harbormaster completed remote builds in B409: Diff 733.Sat, May 13, 11:04

aklapper mentioned this in T15396: PHP 8.1 "strlen(null)" exception blocks rendering "Browse Dashboards" dialog.Sat, May 13, 11:23

aklapper mentioned this in D25227: Fix PHP 8.1 "strlen(null)" exception which blocks rendering "Browse Dashboards" dialog.Sat, May 13, 11:24

valerio.bozzolan mentioned this in Wikimedia Hackaton 2023 Athens Code Sprint.Thu, May 18, 10:16

aklapper mentioned this in rP402a63c5de96: Fix PHP 8.1 "strlen(null)" exception which blocks rendering "Browse Dashboards"….Fri, May 19, 10:00

Thanks for this patch (again!)

I tested this locally with fuzzy tests in the Name Contains and Email Contains input fields, without any nuclear implosion.

The function phutil_nonempty_string() will report any alien value that is not string or null, and I verified that this is OK here.

sgtm

This revision is now accepted and ready to land.Fri, May 19, 10:45

Closed by commit rP912a933b0214: Fix PHP 8.1 "strlen(null)" exceptions which block rendering Legalpad signatures (authored by aklapper). · Explain WhyFri, May 19, 15:02

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP912a933b0214: Fix PHP 8.1 "strlen(null)" exceptions which block rendering Legalpad signatures.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

legalpad/

query/

LegalpadDocumentSignatureSearchEngine.php

4 lines

O1: Blessed Committers

Diff 747

src/applications/legalpad/query/LegalpadDocumentSignatureSearchEngine.php

Loading...