New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHP7.4 - Add "str_split" => "mb_str_split" mapping. #4380
PHP7.4 - Add "str_split" => "mb_str_split" mapping. #4380
Conversation
…ePossum) This PR was merged into the 2.12 branch. Discussion ---------- PHP7.4 - Add "str_split" => "mb_str_split" mapping. @ see https://wiki.php.net/rfc/mb_str_split Commits ------- 7ef0040 PHP7.4 - Add "str_split" => "mb_str_split" mapping.
$functions = array_filter( | ||
self::$functionsMap, | ||
static function ($mapping) { | ||
return \function_exists($mapping['alternativeName']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was causing the issue. when we execute this, it's always truth, as we have the mb polyfill on execution time...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ref #4529
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @SpacePossum
…keradus, Slamdunk) This PR was merged into the 2.15 branch. Discussion ---------- Redo PHP7.4 - Add "str_split" => "mb_str_split" mapping This reverts commit d35c431. Reapply #4380 by reverting #4533 Commits ------- 083d062 mb_*: rely on ReflectionFunction::isInternal to detect correct list of functions to fix 3fea39f Revert "Revert PHP7.4 - Add \"str_split\" => \"mb_str_split\" mapping"
@ see https://wiki.php.net/rfc/mb_str_split