Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Oct 25, 2025

. grapheme_strpos(), grapheme_stripos(), grapheme_strrpos(),
grapheme_strripos(), grapheme_strstr(), grapheme_stristr() and
grapheme_levenshtein() functions add $locale parameter.
RFC: https://wiki.php.net/rfc/grapheme_add_locale_for_case_insensitive

This commit accounts for detecting the new parameters.

Note: the changelog entry is missing one function and has one too many. See PR php/php-src#20239

Refs:

Related to #1849

…RFC)

> . grapheme_strpos(), grapheme_stripos(), grapheme_strrpos(),
>   grapheme_strripos(), grapheme_strstr(), grapheme_stristr() and
>   grapheme_levenshtein() functions add $locale parameter.
>   RFC: https://wiki.php.net/rfc/grapheme_add_locale_for_case_insensitive

This commit accounts for detecting the new parameters.

Note: the changelog entry is missing one function and has one too many. See PR php/php-src 20239

Refs:
* https://wiki.php.net/rfc/grapheme_add_locale_for_case_insensitive
* https://github.com/php/php-src/blob/c2523ed581d622df802dfe3878c01de89828bf21/UPGRADING#L608-L611
* php/php-src 18792
* php/php-src@ad75c26
* https://www.php.net/manual/en/ref.intl.grapheme.php

Related to 1849
@jrfnl jrfnl added this to the 10.0.0-alpha2 milestone Oct 25, 2025
@jrfnl jrfnl requested a review from wimg October 25, 2025 23:14
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels Oct 25, 2025
@wimg wimg merged commit 4423c33 into develop Oct 27, 2025
76 checks passed
@wimg wimg deleted the php-8.5/newfunctionparam-grapheme-locale branch October 27, 2025 20:28
@github-actions github-actions bot removed PR: quick merge PR only contains relatively simple changes PR: ready for review 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