[CodingStyle] Fixes #5987 Escape content delimiter in pattern for Con…

…sistentPregDelimiterRector
rectorphp · Mar 25, 2021 · 3133d45 · 3133d45
1 parent d266e69
commit 3133d45
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/...uncCall/ConsistentPregDelimiterRector/Fixture/escape_content_delimiter_in_pattern.php.inc b/...uncCall/ConsistentPregDelimiterRector/Fixture/escape_content_delimiter_in_pattern.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\FuncCall\ConsistentPregDelimiterRector\Fixture;
+
+class EscapeContentDelimiterInPattern
+{
+    public function run($value)
+    {
+        preg_match('/^#(([a-f0-9]{3}){1,2})$/i', '#ffffff');
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\FuncCall\ConsistentPregDelimiterRector\Fixture;
+
+class EscapeContentDelimiterInPattern
+{
+    public function run($value)
+    {
+        preg_match('#^\#(([a-f0-9]{3}){1,2})$#i', '#ffffff');
+    }
+}
+
+?>
diff --git a/rules/CodingStyle/Rector/FuncCall/ConsistentPregDelimiterRector.php b/rules/CodingStyle/Rector/FuncCall/ConsistentPregDelimiterRector.php
@@ -168,6 +168,8 @@ private function refactorArgument(Arg $arg): void
 
         $string->value = Strings::replace($value, self::INNER_REGEX, function (array $match): string {
             $innerPattern = $match['content'];
+            $innerPattern = str_replace($this->delimiter, '\\' . $this->delimiter, $innerPattern);
+
             // change delimiter
             if (strlen($innerPattern) > 2 && $innerPattern[0] === $innerPattern[strlen($innerPattern) - 1]) {
                 $innerPattern[0] = $this->delimiter;