Page MenuHomePhorge

Fix implicitly nullable parameter declarations for PHP 8.4
ClosedPublic

Authored by aklapper on Oct 23 2024, 09:57.
Tags
None
Referenced Files
F2995430: D25831.1740341526.diff
Sat, Feb 22, 20:12
F2995429: D25831.1740341526.diff
Sat, Feb 22, 20:12
F2995403: D25831.1740340460.diff
Sat, Feb 22, 19:54
F2995394: D25831.1740340030.diff
Sat, Feb 22, 19:47
F2932468: D25831.1737736090.diff
Jan 23 2025, 16:28
F2932464: D25831.1737736083.diff
Jan 23 2025, 16:28
F2932394: D25831.1737734227.diff
Jan 23 2025, 15:57
F2932381: D25831.1737733891.diff
Jan 23 2025, 15:51

Details

Summary

Followup to rARC99e57a70. This patch should cover all remaining issues now that PHPStan covers it (instead of the previous trial-and-error approach).

Implicitly nullable parameter declarations are deprecated in PHP 8.4:
https://php.watch/versions/8.4/implicitly-marking-parameter-type-nullable-deprecated

The proposed syntax was introduced in PHP 7.1 and Phorge requires PHP 7.2 now.

Test Plan

Run static code analysis.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable