merged branch fabpot/exec-fix (PR #6882)
This PR was submitted for the master branch but it was merged into the 2.2 branch instead (closes #6882). Commits ------- b8ef96e [Finder] fixed regression in the Finder component (it was possible to use it without using exec before, closes #6357) Discussion ---------- [Finder] fixed regression in the Finder component (it was possible to use it without using exec before, closes #6357) | Q | A | ------------- | --- | Bug fix? | yes (fixed a BC break) | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #6357 | License | MIT | Doc PR | n/a
This commit is contained in:
commit
81aed8f5db
@ -52,7 +52,11 @@ class Shell
|
||||
{
|
||||
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)
|
||||
|
Reference in New Issue
Block a user