Swap conditions, prevent ".." file lookup (#938)

atk4 · Feb 11, 2020 · 5407184 · 5407184
1 parent 907ba86
commit 5407184
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/tests/DemoCallExitTest.php b/tests/DemoCallExitTest.php
@@ -14,7 +14,7 @@ class DemoCallExitTest extends BuiltInWebServerAbstract
     private $regexJSON = '
   /
   (?(DEFINE)
-     (?<number>   -? (?= [1-9]|0(?!\d) ) \d+ (\.\d+)? ([eE] [+-]? \d+)? )    
+     (?<number>   -? (?= [1-9]|0(?!\d) ) \d+ (\.\d+)? ([eE] [+-]? \d+)? )
      (?<boolean>   true | false | null )
      (?<string>    " ([^"\\\\]* | \\\\ ["\\\\bfnrt\/] | \\\\ u [0-9a-f]{4} )* " )
      (?<array>     \[  (?:  (?&json)  (?: , (?&json)  )*  )?  \s* \] )
@@ -23,15 +23,15 @@ class DemoCallExitTest extends BuiltInWebServerAbstract
      (?<json>   \s* (?: (?&number) | (?&boolean) | (?&string) | (?&array) | (?&object) ) \s* )
   )
   \A (?&json) \Z
-  /six   
+  /six
 ';
     private $regexSSE = '/^[data|id|event].*$/m';
 
     public function testableDemoFilesdataProvider()
     {
         $files = [];
         foreach (scandir(dirname(__DIR__).DIRECTORY_SEPARATOR.'demos') as $file) {
-            if (is_dir($file) || substr($file, -3) !== 'php') {
+            if (substr($file, -3) !== 'php' || is_dir($file)) {
                 continue;
             }