Incorrect behavior of SimplifyRegexPatternRector about rector HOT 5 CLOSED

Apologies for intervening, but I noticed that the original Regexp is invalid.
There is an extra opening square bracket in it.
/[^[a-zA-Z0-9_]/

It was probably intended to be like this:
/[^a-zA-Z0-9_]/

So I'm unsure if it's a bug in Rector.

from rector.

-$purln = preg_replace("/[^[a-zA-Z0-9_]/", "", "test");
+$purln = preg_replace("/[^\\w/", "", "test");

$purln = preg_replace("/[^\\w/", "", "test");
s not valid

from rector.

Seems legit. Could you send PR with a fix?

from rector.

@pkvach the original regex is correct, see https://3v4l.org/Y881D and https://regex101.com/r/I47eIK/1 . That's strange, but correct.

from rector.

@samsonasik
Oh, I see, thank you.
Just noticed it didn't look right. And some Regex validation services are giving an error on it.

And SimplifyRegexPatternRector just replaces the pattern it finds in the original Regex.

from rector.