fixed regression in the Finder component (it was possible to use it without using exec before, closes #6357)

fabpot · fabpot · commit 3166e5b763a9 · 2013-02-04T13:41:13.000+01:00
diff --git a/src/Symfony/Component/Finder/Shell/Shell.php b/src/Symfony/Component/Finder/Shell/Shell.php
@@ -52,7 +52,11 @@ public function testCommand($command)
     {
         if (self::TYPE_WINDOWS === $this->type) {
             // todo: find a way to test if windows command exists
-            return true;
+            return false;
+        }
+
+        if (!function_exists('exec')) {
+            return false;
         }
 
         // todo: find a better way (command could not be available)

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,11 @@ public function testCommand($command)`
`52`	`52`	`{`
`53`	`53`	`if (self::TYPE_WINDOWS === $this->type) {`
`54`	`54`	`// todo: find a way to test if windows command exists`
`55`		`- return true;`
	`55`	`+ return false;`
	`56`	`+ }`
	`57`	`+`
	`58`	`+ if (!function_exists('exec')) {`
	`59`	`+ return false;`
`56`	`60`	`}`
`57`	`61`
`58`	`62`	`// todo: find a better way (command could not be available)`