Skip to content

Comments

PHP 8.5 compatibility#76

Open
martinsoenen wants to merge 3 commits intomainfrom
php8.5-compatibility
Open

PHP 8.5 compatibility#76
martinsoenen wants to merge 3 commits intomainfrom
php8.5-compatibility

Conversation

@martinsoenen
Copy link
Contributor

@martinsoenen martinsoenen commented Feb 5, 2026

Add PHP 8.5 tests
Fix deprecations in 8.5 (about "null" as an array offset https://www.php.net/manual/en/migration85.deprecated.php)

@martinsoenen martinsoenen changed the title PHP8.5 compatibility Draft: PHP8.5 compatibility Feb 5, 2026
@martinsoenen martinsoenen changed the title Draft: PHP8.5 compatibility PHP8.5 compatibility Feb 5, 2026
@martinsoenen martinsoenen changed the title PHP8.5 compatibility PHP 8.5 compatibility Feb 5, 2026
[-0.0001, 0.0001, 6],
[1.111, 1.119, 4],
[2500.0, 5000.0, 0],
[-0.999, 0.999, 2],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a breaking change ?

// We assume we here have multiple extensions declined by locale, we will try
// to get the extension with the current locale, defaulting to first element
if (is_array($extension) && isset($extension['locales'])) {
if (!isset($extension['locales'][$this->getLocale()]) && !isset($extension['locales'][null])) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we switch "default" to a const ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants