Skip to content
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

PHP Warning: mb_ereg_replace(): mbregex search failure in php_mbereg_replace_exec(): retry-limit-in-match over in /vendor/vanderlee/php-sentence/src/Multibyte.php on line 59 #27

Open
splitbrain opened this issue Jun 9, 2023 · 0 comments

Comments

@splitbrain
Copy link

I get above warning for some of my content. The problem happens in the trim() method:

public static function trim($string)
{
return mb_ereg_replace('^\s*([\s\S]*?)\s*$', '\1', $string);
}

When the warning is throw, mb_ereg_replace will return false instead of a string, breaking the rest of the script.

I created a test case to demonstrate the problem: splitbrain-forks@914dd45

I have no idea why it fails. I'm also not 100% sure why a UTF-8 aware method is needed here. Is it to remove non-ASCII whitespaces like non-breaking or half-width spaces? Would using preg_replace with the /u flag be an alternative?

splitbrain added a commit to splitbrain-forks/php-sentence that referenced this issue Jun 9, 2023
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

No branches or pull requests

1 participant