[CodeQuality] Skip windows absolute path on AbsolutizeRequireAndInclu…

…dePathRector
rectorphp · Jan 20, 2025 · c3684df · c3684df
1 parent 7ec3ae3
commit c3684df
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/...nclude_/AbsolutizeRequireAndIncludePathRector/Fixture/skip_windows_absolute_paths.php.inc b/...nclude_/AbsolutizeRequireAndIncludePathRector/Fixture/skip_windows_absolute_paths.php.inc
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Include_\AbsolutizeRequireAndIncludePathRector\Fixture;
+
+final class SkipAbsolutePaths
+{
+    public function run()
+    {
+        include "C:\\Users\\prj\\vendor\\autoload.php";
+    }
+}
diff --git a/rules/CodeQuality/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php b/rules/CodeQuality/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php
@@ -20,6 +20,12 @@
  */
 final class AbsolutizeRequireAndIncludePathRector extends AbstractRector
 {
+    /**
+     * @var string
+     * @see https://regex101.com/r/spMHn4/1
+     */
+    private const WINDOWS_DRIVE_REGEX = '#^[a-zA-z]+.*\:[\/\\\]#';
+
     public function __construct(
         private readonly ValueResolver $valueResolver
     ) {
@@ -105,6 +111,10 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
+        if (Strings::match($includeValue, self::WINDOWS_DRIVE_REGEX)) {
+            return null;
+        }
+
         // add preslash to string
         $node->expr->value = \str_starts_with($includeValue, './') ? Strings::substring(
             $includeValue,