suralc / pvra

src/Analysers/Php53Features.php

Spacing +5 added lines, -5 removed lines

@@ -99,14 +99,14 @@ 
                 Reason::GOTO_KEYWORD,
                 $node->getLine(),
                 null,
-                ['name' => $node->name]
+                [ 'name' => $node->name ]
             );
         } elseif ($node instanceof Node\Stmt\Label) {
             $this->getResult()->addRequirement(
                 Reason::JUMP_LABEL,
                 $node->getLine(),
                 null,
-                ['name' => $node->name]
+                [ 'name' => $node->name ]
             );
         }
     }
@@ -121,7 +121,7 @@ 
                 Reason::NAMESPACE_DECLERATION,
                 $node->getLine(),
                 null,
-                ['name' => isset($node->name) ? $node->name->toString() : '::global::']
+                [ 'name' => isset($node->name) ? $node->name->toString() : '::global::' ]
             );
         } elseif ($node instanceof Node\Scalar\MagicConst\Namespace_) {
             $this->getResult()->addRequirement(Reason::NAMESPACE_MAGIC_CONSTANT, $node->getLine());
@@ -130,7 +130,7 @@ 
                 Reason::NAMESPACE_IMPORT,
                 $node->getLine(),
                 null,
-                ['import_count' => count($node->uses)]
+                [ 'import_count' => count($node->uses) ]
             );
         }
     }
@@ -181,7 +181,7 @@ 
                         Reason::CONST_KEYWORD_DOC_SYNTAX,
                         $const->value->getLine(),
                         null,
-                        ['name' => $const->name]
+                        [ 'name' => $const->name ]
                     );
                 }
             }

src/Analysers/Php70Features.php

Spacing +9 added lines, -9 removed lines

@@ -42,8 +42,8 @@ 
 class Php70Features extends LanguageFeatureAnalyser implements AnalyserAwareInterface
 {
     // move both to be const once 5.6+ is mandatory
-    private static $reservedNames = ['string', 'int', 'float', 'bool', 'null', 'false', 'true'];
-    private static $softReservedNames = ['object', 'resource', 'mixed', 'numeric'];
+    private static $reservedNames = [ 'string', 'int', 'float', 'bool', 'null', 'false', 'true' ];
+    private static $softReservedNames = [ 'object', 'resource', 'mixed', 'numeric' ];
 
     /**
      * @inheritdoc
@@ -103,12 +103,12 @@ 
                         Reason::PHP4_CONSTRUCTOR,
                         $method->getLine(),
                         null,
-                        ['name' => $method->name],
+                        [ 'name' => $method->name ],
                     ];
                 }
             }
             if ($possibleCtorInfo !== null) {
-                call_user_func_array([$this->getResult(), 'addLimit'], $possibleCtorInfo);
+                call_user_func_array([ $this->getResult(), 'addLimit' ], $possibleCtorInfo);
             }
         }
     }
@@ -136,9 +136,9 @@ 
     {
         if ($call->name instanceof Node\Name && strcasecmp('class_alias', $call->name->getLast()) === 0
         ) {
-            if (isset($call->args[1]) && $call->args[1]->value instanceof Node\Scalar\String_) {
-                $value = $call->args[1]->value->value;
-                $this->handleClassName($value, $call->args[1]->value->getLine());
+            if (isset($call->args[ 1 ]) && $call->args[ 1 ]->value instanceof Node\Scalar\String_) {
+                $value = $call->args[ 1 ]->value->value;
+                $this->handleClassName($value, $call->args[ 1 ]->value->getLine());
             }
         }
     }
@@ -182,10 +182,10 @@ 
             $baseName = basename(str_replace('\\', '/', $name));
             if ($this->mode & self::MODE_DEPRECATION && $this->isNameSoftReserved($name)) {
                 $this->getResult()->addLimit(Reason::SOFT_RESERVED_NAME, $line, null,
-                    ['fqn' => $name, 'class' => $baseName]);
+                    [ 'fqn' => $name, 'class' => $baseName ]);
             } elseif ($this->mode & self::MODE_REMOVAL && $this->isNameReserved($name)) {
                 $this->getResult()->addLimit(Reason::RESERVED_CLASS_NAME, $line, null,
-                    ['fqn' => $name, 'class' => $baseName]);
+                    [ 'fqn' => $name, 'class' => $baseName ]);
             }
         }
     }

src/Console/Commands/DirCommand.php

Spacing +12 added lines, -12 removed lines

@@ -61,7 +61,7 @@ 
                 FileFinderBuilder::SORT_BY_NAME)
             ->addOption('listFilesOnly', null, InputOption::VALUE_NONE,
                 'Only list matched files and do not run analysis.')
-            ->addArgument('filters', InputArgument::IS_ARRAY | InputArgument::OPTIONAL, 'Filter', ['name:*.php']);
+            ->addArgument('filters', InputArgument::IS_ARRAY | InputArgument::OPTIONAL, 'Filter', [ 'name:*.php' ]);
     }
 
     /**
@@ -133,16 +133,16 @@ 
             $highestRequirement->getAnalysisTargetId()));
 
         if ($highestRequirement->count() !== 0) {
-            $tableData = [];
+            $tableData = [ ];
             // order by version->descending. Might want to implement ordering by line later.
             foreach (array_reverse($highestRequirement->getRequirements()) as $version => $reasons) {
                 foreach ($reasons as $reason) {
-                    $tableData[] = [$reason['version'], $reason['msg'], $reason['line']];
+                    $tableData[ ] = [ $reason[ 'version' ], $reason[ 'msg' ], $reason[ 'line' ] ];
                 }
             }
 
             (new Table($out))
-                ->setHeaders(['Version', 'Message', 'Line'])
+                ->setHeaders([ 'Version', 'Message', 'Line' ])
                 ->setRows($tableData)
                 ->render();
         } else {
@@ -171,14 +171,14 @@ 
                     ' for the following reasons:',
                     PHP_EOL,
                 ]);
-                $tableData = [];
+                $tableData = [ ];
                 /** @var $reason Reasoning */
                 foreach ($result->getRequirementIterator() as $reason) {
-                    $tableData[] = [$reason['version'], $reason['msg'], $reason['line']];
+                    $tableData[ ] = [ $reason[ 'version' ], $reason[ 'msg' ], $reason[ 'line' ] ];
                 }
 
                 (new Table($out))
-                    ->setHeaders(['Version', 'Message', 'Line'])
+                    ->setHeaders([ 'Version', 'Message', 'Line' ])
                     ->setRows($tableData)
                     ->render();
             }
@@ -213,7 +213,7 @@ 
         ]);
         $out->writeln('');
 
-        $usedVersions = [];
+        $usedVersions = [ ];
         /** @var AnalysisResult $result */
         foreach ($results as $result) {
             $versions = array_keys($result->getRequirements());
@@ -222,12 +222,12 @@ 
             }
         }
 
-        usort($usedVersions, function ($a, $b) {
+        usort($usedVersions, function($a, $b) {
             return version_compare($b, $a);
         });
 
         $table = new Table($out);
-        $table->setHeaders(['Version', 'Message', 'Position']);
+        $table->setHeaders([ 'Version', 'Message', 'Position' ]);
 
         foreach ($usedVersions as $index => $version) {
             /** @var AnalysisResult $result */
@@ -237,8 +237,8 @@ 
                     foreach ($selectedResults as $reason) {
                         $table->addRow([
                             $version,
-                            $reason['msg'],
-                            $this->formatOutputPath($reason['targetId']) . ':' . $reason['line'],
+                            $reason[ 'msg' ],
+                            $this->formatOutputPath($reason[ 'targetId' ]) . ':' . $reason[ 'line' ],
                         ]);
                     }
                 }