[FileSystem] added if-windows check

Erik Trapman · Erik Trapman · commit fc3ebb8c6563 · 2012-06-13T16:33:54.000+02:00
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -94,12 +94,15 @@ public function remove($files)
 
             if (is_dir($file) && !is_link($file)) {
                 $this->remove(new \FilesystemIterator($file));
-                rmdir($file);
-            } elseif(is_dir($file) && is_link($file)) {
-                // https://bugs.php.net/bug.php?id=52176 windows thinks symlinks are directories
+
                 rmdir($file);
             } else {
-                unlink($file);
+                // https://bugs.php.net/bug.php?id=52176
+                if (defined('PHP_WINDOWS_VERSION_MAJOR') && is_dir($file)) {
+                    rmdir($file);
+                } else {
+                    unlink($file);
+                }
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -94,12 +94,15 @@ public function remove($files)`
`94`	`94`
`95`	`95`	`if (is_dir($file) && !is_link($file)) {`
`96`	`96`	`$this->remove(new \FilesystemIterator($file));`
`97`		`- rmdir($file);`
`98`		`- } elseif(is_dir($file) && is_link($file)) {`
`99`		`- // https://bugs.php.net/bug.php?id=52176 windows thinks symlinks are directories`
	`97`	`+`
`100`	`98`	`rmdir($file);`
`101`	`99`	`} else {`
`102`		`- unlink($file);`
	`100`	`+ // https://bugs.php.net/bug.php?id=52176`
	`101`	`+ if (defined('PHP_WINDOWS_VERSION_MAJOR') && is_dir($file)) {`
	`102`	`+ rmdir($file);`
	`103`	`+ } else {`
	`104`	`+ unlink($file);`
	`105`	`+ }`
`103`	`106`	`}`
`104`	`107`	`}`
`105`	`108`	`}`