Page MenuHomePhorge

Fix implicitly nullable parameter declarations for PHP 8.4
ClosedPublic

Authored by aklapper on Oct 23 2024, 10:14.
Tags
None
Referenced Files
F3302022: D25832.1743061602.diff
Wed, Mar 26, 07:46
F3299455: D25832.1743022174.diff
Tue, Mar 25, 20:49
F3254951: D25832.1742488181.diff
Wed, Mar 19, 16:29
F3225238: D25832.1742098984.diff
Sat, Mar 15, 04:23
F3219796: D25832.1741763476.diff
Tue, Mar 11, 07:11
F3005901: D25832.1740548851.diff
Tue, Feb 25, 05:47
F3005560: D25832.1740530225.diff
Tue, Feb 25, 00:37
F2995435: D25832.1740341531.diff
Feb 22 2025, 20:12

Details

Summary

Followup to rPdb61eb20 and rPf3d49f74. 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
rP Phorge
Branch
nullable84rest
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1611
Build 1611: arc lint + arc unit