xprt64 / code-analysis

src/Gica/CodeAnalysis/PhpClassInFileInspector.php

Braces +3 added lines, -2 removed lines

@@ -80,8 +80,9 @@
         }
 
         $namespace = $m['namespace'];
-        if ($namespace)
-            $namespace = '\\' . $namespace;
+        if ($namespace) {
+                    $namespace = '\\' . $namespace;
+        }
 
 
         $fqn = $namespace . '\\' . $unqualifiedClassName;

src/Gica/CodeAnalysis/MethodListenerDiscovery.php

Braces +3 added lines, -2 removed lines

@@ -194,8 +194,9 @@
 
     private function isValidListenerMethod(\ReflectionMethod $reflectionMethod)
     {
-        if ($reflectionMethod->getNumberOfParameters() == 0)
-            return false;
+        if ($reflectionMethod->getNumberOfParameters() == 0) {
+                    return false;
+        }
 
         return true;
     }

Indentation +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
             }
         }
 
-         return $this->allEventsListeners;
+            return $this->allEventsListeners;
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
     protected function filterFiles(\Iterator $files)
     {
-        return new \CallbackFilterIterator($files, function ($file) {
+        return new \CallbackFilterIterator($files, function($file) {
             return $this->isListenerFileName($file);
         });
     }

src/Gica/CodeAnalysis/MethodListenerDiscovery/MapCodeGenerator/MapCodeGeneratorBase.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
     protected function generateLevel1Item(array $eventListenerMethods)
     {
-        return array_map(function (ListenerMethod $listener) {
+        return array_map(function(ListenerMethod $listener) {
             return $this->spaces(self::SPACES_AT_HANDLERS) .
                 '[' . $this->prependSlash($this->getLevel1FirstItem($listener)) . '::class' . ', \'' . $this->getLevel1SecondItem($listener) . '\'],';
         }, $eventListenerMethods);

src/Gica/CodeAnalysis/AggregateEventHandlersValidator.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
     protected function filterFiles(\Iterator $files)
     {
-        return new \CallbackFilterIterator($files, function ($file) {
+        return new \CallbackFilterIterator($files, function($file) {
             return $this->isListenerFileName($file);
         });
     }

src/Gica/CodeAnalysis/Shared/ClassComparison/SubclassComparator.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
         return $parent->isInterface() ? $child->implementsInterface($parentClass) : $child->isSubclassOf($parentClass);
     }
 
+    /**
+     * @param \ReflectionClass $object
+     */
     public function isASubClassOrSameClass($object, string $parentClass)
     {
         return $this->getObjectClass($object) === $parentClass || $this->isASubClass($object, $parentClass);

src/Gica/CodeAnalysis/Shared/ClassSorter/ByConstructorDependencySorter.php

Doc Comments +3 added lines

@@ -68,6 +68,9 @@
         return $dependencies;
     }
 
+    /**
+     * @param string[] $classes
+     */
     private function isParentClassOfAny(\ReflectionClass $parentClass, $classes): bool
     {
         $comparator = new SubclassComparator();

Spacing +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
     {
         $comparator = new SubclassComparator();
 
-        $isASubClassOrSameClass = function (\ReflectionClass $class) use ($parentClass, $comparator) {
+        $isASubClassOrSameClass = function(\ReflectionClass $class) use ($parentClass, $comparator) {
             return $comparator->isASubClassOrSameClass($class, $parentClass->name);
         };
 
@@ -93,11 +93,11 @@ 
      */
     private function classNameFromParameters(array $parameters)
     {
-        $strings = array_map(function (\ReflectionParameter $parameter) {
+        $strings = array_map(function(\ReflectionParameter $parameter) {
             return $this->classNameFromParameter($parameter);
         }, $parameters);
 
-        return array_filter($strings, function ($s) {
+        return array_filter($strings, function($s) {
             return !!$s;
         });
     }

src/Gica/CodeAnalysis/Shared/BubbleArraySorter.php

Spacing +4 added lines, -4 removed lines

@@ -8,10 +8,10 @@
 {
     public function sort($arr, callable $isBefore) {
         $size = count($arr);
-        for ($i=0; $i<$size; $i++) {
-            for ($j=0; $j<$size-1-$i; $j++) {
-                if ($isBefore($arr[$j+1], $arr[$j])) {
-                    $this->swap($arr, $j, $j+1);
+        for ($i = 0; $i < $size; $i++) {
+            for ($j = 0; $j < $size - 1 - $i; $j++) {
+                if ($isBefore($arr[$j + 1], $arr[$j])) {
+                    $this->swap($arr, $j, $j + 1);
                 }
             }
         }

src/Gica/CodeAnalysis/MethodListenerDiscovery/MapGrouper/GrouperByEvent.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         $classSorter = new ByConstructorDependencySorter();
 
-        return (new BubbleArraySorter)->sort($listeners, function (ListenerMethod $a, ListenerMethod $b) use ($classSorter) {
+        return (new BubbleArraySorter)->sort($listeners, function(ListenerMethod $a, ListenerMethod $b) use ($classSorter) {
             return $classSorter->__invoke($a->getClass(), $b->getClass());
         });
     }

src/Gica/CodeAnalysis/ClassDiscovery.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
 
     protected function filterFiles(\Iterator $files)
     {
-        return new \CallbackFilterIterator($files, function ($file) {
+        return new \CallbackFilterIterator($files, function($file) {
             return $this->isListenerFileName($file);
         });
     }