Page MenuHomePhabricator

Fix more PHP 8.1 "strlen(null)" callsites in PhutilURI
ClosedPublic

Authored by epriestley on May 3 2022, 7:00 PM.
Tags
None
Referenced Files
F18199831: D21798.id51957.diff
Mon, Aug 18, 2:40 AM
F18106680: D21798.id.diff
Sun, Aug 10, 11:12 PM
F18103979: D21798.id.diff
Sun, Aug 10, 9:02 AM
F18044227: D21798.diff
Sun, Aug 3, 6:57 AM
F17869058: D21798.id.diff
Mon, Jul 28, 10:23 AM
F17868693: D21798.id51957.diff
Jul 28 2025, 8:41 AM
F17843393: D21798.diff
Jul 27 2025, 12:51 AM
F17823766: D21798.diff
Jul 26 2025, 4:51 AM
Subscribers
None

Details

Summary

Ref T13676. Ref T13588. These properties on PhutilURI have flexible types, just wrap them.

Test Plan

Built a new Drydock working copy with an HTTPS URI under PHP 8.1, see T13676.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.May 3 2022, 8:42 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.