[StrContainsRector] also replace multibyte functions fro strpos and strstr

Author: JohJohan

config/set/php85.php

@@ -6,9 +6,5 @@
 use Rector\Php85\Rector\ArrayDimFetch\ArrayFirstLastRector;
 
 return static function (RectorConfig $rectorConfig): void {
-    $rectorConfig->rules(
-        [
-            ArrayFirstLastRector::class,
-        ]
-    );
+    $rectorConfig->rules([ArrayFirstLastRector::class]);
 };

rules-tests/Php80/Rector/NotIdentical/StrContainsRector/Fixture/multibyte_offset_strpos.php.inc

@@ -0,0 +1,25 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\NotIdentical\StrContainsRector\Fixture;
+
+class MultibyteOffsetStrpos
+{
+    public function run()
+    {
+        $isMatch = mb_strpos('abc', 'a', 1) !== false;
+    }
+}
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\NotIdentical\StrContainsRector\Fixture;
+
+class MultibyteOffsetStrpos
+{
+    public function run()
+    {
+        $isMatch = str_contains(substr('abc', 1), 'a');
+    }
+}
+?>

rules-tests/Php80/Rector/NotIdentical/StrContainsRector/Fixture/multibyte_strpos_function.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\NotIdentical\StrContainsRector\Fixture;
+
+class MultibyteStrposFunction
+{
+    public function run()
+    {
+        $isMatch = mb_strpos('abc', 'a') !== false;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\NotIdentical\StrContainsRector\Fixture;
+
+class MultibyteStrposFunction
+{
+    public function run()
+    {
+        $isMatch = str_contains('abc', 'a');
+    }
+}
+
+?>

rules-tests/Php80/Rector/NotIdentical/StrContainsRector/Fixture/multibyte_strstr_function.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\NotIdentical\StrContainsRector\Fixture;
+
+class MultibyteStrstrFunction
+{
+    public function run()
+    {
+        $isMatch = mb_strstr('abc', 'a') === false;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\NotIdentical\StrContainsRector\Fixture;
+
+class MultibyteStrstrFunction
+{
+    public function run()
+    {
+        $isMatch = !str_contains('abc', 'a');
+    }
+}
+
+?>

rules/Php80/Rector/NotIdentical/StrContainsRector.php

@@ -32,7 +32,7 @@ final class StrContainsRector extends AbstractRector implements MinPhpVersionInt
     /**
      * @var string[]
      */
-    private const OLD_STR_NAMES = ['strpos', 'strstr'];
+    private const OLD_STR_NAMES = ['mb_strpos', 'mb_strstr', 'strpos', 'strstr'];
 
     public function __construct(
         private readonly ValueResolver $valueResolver
@@ -47,7 +47,7 @@ public function provideMinPhpVersion(): int
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition(
-            'Replace strpos() !== false and strstr()  with str_contains()',
+            'Replace strpos|mb_strpos() !== false and strstr()|mb_strstr() with str_contains()',
             [
                 new CodeSample(
                     <<<'CODE_SAMPLE'
@@ -100,7 +100,7 @@ public function refactor(Node $node): ?Node
         if (isset($funcCall->getArgs()[2])) {
             $secondArg = $funcCall->getArgs()[2];
 
-            if ($this->isName($funcCall->name, 'strpos') && ! $this->isIntegerZero($secondArg->value)) {
+            if ($this->isNames($funcCall->name, ['strpos', 'mb_strpos']) && ! $this->isIntegerZero($secondArg->value)) {
                 $funcCall->args[0] = new Arg($this->nodeFactory->createFuncCall(
                     'substr',
                     [$funcCall->args[0], $secondArg]