Fixed bug #1495 : Setting an invalid installed path breaks all commands

squizlabs · Jun 6, 2017 · 78f9c94 · 78f9c94
1 parent 046deb2
commit 78f9c94
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/package.xml b/package.xml
@@ -63,6 +63,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #1451 : Sniff exclusions/restrictions dont work with custom sniffs unless they use the PHP_CodeSniffer NS
   - Fixed bug #1454 : Squiz.WhiteSpace.OperatorSpacing is not checking spacing on either side of a short ternary operator
     -- Thanks to Mponos George for the patch
+  - Fixed bug #1495 : Setting an invalid installed path breaks all commands
   - Fixed bug #1496 : Squiz.Strings.DoubleQuoteUsage not unescaping dollar sign when fixing
     -- Thanks to Michał Bundyra for the patch
  </notes>

diff --git a/src/Util/Standards.php b/src/Util/Standards.php
@@ -175,6 +175,11 @@ public static function getInstalledStandards(
                 continue;
             }
 
+            if (is_dir($standardsDir) === false) {
+                // Doesn't exist.
+                continue;
+            }
+
             $di = new \DirectoryIterator($standardsDir);
             foreach ($di as $file) {
                 if ($file->isDir() === true && $file->isDot() === false) {