Skip to content

Fix GH-19070: setlocale($type, NULL) should not be deprecated - #19071

Closed
ndossche wants to merge 1 commit into
php:masterfrom
ndossche:fix-19070
Closed

ndossche wants to merge 1 commit into
php:masterfrom
ndossche:fix-19070

Conversation

@ndossche

@ndossche ndossche commented Jul 8, 2025

Copy link
Copy Markdown
Member

This restores the old behaviour.

@remicollet remicollet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, old behaviour restored

@remicollet

remicollet commented Jul 9, 2025 •

Copy link
Copy Markdown
Member

An alternative way (to follow strictly doc) will be to accept int|string|null for 1st param, and only string for others, but this is probably a BC break.

So this PR seems the right fix.

Thanks for the quick fix

@ndossche

Copy link
Copy Markdown
Member Author

An alternative way (to follow strictly doc) will be to accept int|string|null for 1st param, and only string for others, but this is probably a BC break.

Yes I think so as well, since we allowed [null] to behave the same way.
Thanks for checking.

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.

setlocale($type, NULL) should not be deprecated

2 participants