Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Oct 27, 2025

This commit accounts for the deprecation of the ini setting.

In the RFC (and the implementation), the deprecation is conditional on the value:

we propose deprecating this INI setting, meaning setting it to any value different than 0 will raise an E_DEPRECATED diagnostic.

However, for the purpose of PHPCompatibility, I believe checking the value for a call to ini_set() is a little over the top. If anyone would report false positives because of this, we can always consider adding such a check later.

Refs:

Related to #1849

> - Intl:
>   . The intl.error_level INI setting has been deprecated.
>     Errors should either be checked manually or exceptions should be enabled
>     by using the intl.use_exceptions INI setting.
>     RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_intlerror_level_ini_setting

This commit accounts for the deprecation of the ini setting.

In the RFC (and the implementation), the deprecation is conditional on the value:

> we propose deprecating this INI setting, meaning setting it to any value different than `0` will raise an `E_DEPRECATED` diagnostic.

However, for the purpose of PHPCompatibility, I believe checking the value for a call to `ini_set()` is a little over the top. If anyone would report false positives because of this, we can always consider adding such a check later.

Refs:
* https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_intlerror_level_ini_setting
* https://github.com/php/php-src/blob/1570ce9f551f7dff3f51690e5c03b0f5ae0e159f/UPGRADING#L446-L450
* php/php-src 19430
* php/php-src@69f67f9

Related to 1849
@jrfnl jrfnl added this to the 10.0.0-alpha2 milestone Oct 27, 2025
@jrfnl jrfnl requested a review from wimg October 27, 2025 22:00
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels Oct 27, 2025
@wimg wimg enabled auto-merge October 27, 2025 22:01
@wimg wimg merged commit 900c9ab into develop Oct 27, 2025
76 checks passed
@wimg wimg deleted the php-8.5/removedinidirectives-intl-errorlevel branch October 27, 2025 22:26
@github-actions github-actions bot removed PR: ready for review PR: quick merge PR only contains relatively simple changes labels Oct 27, 2025
@jrfnl jrfnl added the PHP: 8.5 label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants