Paths

Table of Contentst

Differential D25886

Remove use of deprecated E_STRICT PHP constant
Needs ReviewPublic
Actions

Authored by aklapper on Feb 17 2025, 11:37.

Details

Reviewers

None

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15989: `arc unit`: Constant E_STRICT is deprecated (with PHP 8.4)

Summary

The E_STRICT PHP constant is deprecated since PHP 8.4 per https://www.php.net/manual/en/migration84.deprecated.php.
Per https://wiki.php.net/rfc/deprecations_php_8_4#remove_e_strict_error_level_and_deprecate_e_strict_constant, the E_STRICT constant was still in use in-between PHP 7.0 and 7.4 for PHP's mysqli extension and PHP's htmlentities() function. Since rP23a49eb403c9ea6c58f4ae2f22416e90a9d24c14, Phorge requires 7.2.25. Phorge neither uses PHP's mysqli extension nor calls PHP's htmlentities() function.
Thus remove its usage, as E_STRICT is a part of E_ALL since PHP 5.4.0 per https://github.com/php/php-src/blob/php-5.4.0/NEWS#L69.

Closes T15989

Test Plan

None.

Diff Detail

Repository

rARC Arcanist

Branch

T15989 (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1724
Build 1724: arc lint + arc unit

Event Timeline

aklapper created this revision.Feb 17 2025, 11:37

Owners added a reviewer: O1: Blessed Committers.Feb 17 2025, 11:37

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptFeb 17 2025, 11:37

aklapper requested review of this revision.Feb 17 2025, 11:37

Harbormaster completed remote builds in B1724: Diff 2672.Feb 17 2025, 11:37

valerio.bozzolan edited the summary of this revision. (Show Details)Feb 17 2025, 11:44

Finally something that can be tested using arc anoid - I guess?

In D25886#23757, @valerio.bozzolan wrote:

Finally something that can be tested using arc anoid - I guess?

@valerio.bozzolan Could you elaborate? I don't see how to introduce an error to make arc anoid print a warning or such (running PHP 8.4.4 without this patch).
But in any case, I hope the explanation and all the links in the commit message make it clear enough; same for D25887.

Since using E_STRICT is deprecated, I thought the simple fact of using it from whatever Arcanist CLI (e.g. arc anoid) was triggering this new warning (with or without having an exception - since that line 18 is supposed to be always executed I think)

Revision Contents
Changeset List

Path

Size

Packages

support/

init/

init-script.php

2 lines

O1: Blessed Committers

Diff 2672

View Options

support/init/init-script.php

Remove use of deprecated E_STRICT PHP constantNeeds ReviewPublicActions