[Config] Fix ** GlobResource on Windows

nicolas-grekas · nicolas-grekas · commit 44955bea53f6 · 2017-06-14T09:11:58.000+02:00
diff --git a/src/Symfony/Component/Config/Resource/GlobResource.php b/src/Symfony/Component/Config/Resource/GlobResource.php
@@ -134,7 +134,7 @@ function (\SplFileInfo $file) { return '.' !== $file->getBasename()[0]; }
 
         $prefixLen = strlen($this->prefix);
         foreach ($finder->followLinks()->sortByName()->in($this->prefix) as $path => $info) {
-            if (preg_match($regex, substr($path, $prefixLen)) && $info->isFile()) {
+            if (preg_match($regex, substr('\\' === \DIRECTORY_SEPARATOR ? str_replace('\\', '/', $path) : $path, $prefixLen)) && $info->isFile()) {
                 yield $path => $info;
             }
         }
diff --git a/src/Symfony/Component/Config/Tests/Resource/GlobResourceTest.php b/src/Symfony/Component/Config/Tests/Resource/GlobResourceTest.php
@@ -36,6 +36,15 @@ public function testIterator()
         $this->assertEquals(array($file => new \SplFileInfo($file)), $paths);
         $this->assertInstanceOf('SplFileInfo', current($paths));
         $this->assertSame($dir, $resource->getPrefix());
+
+        $resource = new GlobResource($dir, '/**/Resource', true);
+
+        $paths = iterator_to_array($resource);
+
+        $file = $dir.DIRECTORY_SEPARATOR.'Resource'.DIRECTORY_SEPARATOR.'ConditionalClass.php';
+        $this->assertEquals(array($file => $file), $paths);
+        $this->assertInstanceOf('SplFileInfo', current($paths));
+        $this->assertSame($dir, $resource->getPrefix());
     }
 
     public function testIsFreshNonRecursiveDetectsNewFile()
diff --git a/src/Symfony/Component/Config/composer.json b/src/Symfony/Component/Config/composer.json
@@ -20,10 +20,12 @@
         "symfony/filesystem": "~2.8|~3.0"
     },
     "require-dev": {
+        "symfony/finder": "~3.3",
         "symfony/yaml": "~3.0",
         "symfony/dependency-injection": "~3.3"
     },
     "conflict": {
+        "symfony/finder": "<3.3",
         "symfony/dependency-injection": "<3.3"
     },
     "suggest": {

Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,7 @@ function (\SplFileInfo $file) { return '.' !== $file->getBasename()[0]; }`
`134`	`134`
`135`	`135`	`$prefixLen = strlen($this->prefix);`
`136`	`136`	`foreach ($finder->followLinks()->sortByName()->in($this->prefix) as $path => $info) {`
`137`		`- if (preg_match($regex, substr($path, $prefixLen)) && $info->isFile()) {`
	`137`	`+ if (preg_match($regex, substr('\\' === \DIRECTORY_SEPARATOR ? str_replace('\\', '/', $path) : $path, $prefixLen)) && $info->isFile()) {`
`138`	`138`	`yield $path => $info;`
`139`	`139`	`}`
`140`	`140`	`}`