Roave / BackwardCompatibilityCheck

src/DetectChanges/BCBreak/ClassBased/MethodChanged.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 
         return array_reduce(
             array_keys($commonMethods),
-            function (Changes $accumulator, string $methodName) use ($methodsFrom, $methodsTo) : Changes {
+            function(Changes $accumulator, string $methodName) use ($methodsFrom, $methodsTo) : Changes {
                 return $accumulator->mergeWith($this->checkMethod->__invoke(
                     $methodsFrom[$methodName],
                     $methodsTo[$methodName]
@@ -49,7 +49,7 @@ 
         $methods = $class->getMethods();
 
         return array_combine(
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 return strtolower($method->getName());
             }, $methods),
             $methods

src/DetectChanges/BCBreak/ClassBased/FinalClassChanged.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     public function __invoke(ReflectionClass $fromClass, ReflectionClass $toClass) : Changes
     {
-        if (! $fromClass->isFinal()) {
+        if (!$fromClass->isFinal()) {
             return Changes::empty();
         }
 

src/DetectChanges/BCBreak/ClassBased/ConstantChanged.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
         return array_reduce(
             array_keys($commonConstants),
-            function (Changes $accumulator, string $constantName) use ($constantsFrom, $constantsTo) : Changes {
+            function(Changes $accumulator, string $constantName) use ($constantsFrom, $constantsTo) : Changes {
                 return $accumulator->mergeWith($this->checkConstant->__invoke(
                     $constantsFrom[$constantName],
                     $constantsTo[$constantName]

src/DetectChanges/BCBreak/ClassBased/ConstantRemoved.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
             $this->accessibleConstants($toClass)
         );
 
-        return Changes::fromList(...array_values(array_map(function (ReflectionClassConstant $constant) use ($fromClass) : Change {
+        return Changes::fromList(...array_values(array_map(function(ReflectionClassConstant $constant) use ($fromClass) : Change {
             return Change::removed(
                 sprintf('Constant %s::%s was removed', $fromClass->getName(), $constant->getName()),
                 true
@@ -34,7 +34,7 @@ 
     /** @return ReflectionClassConstant[] */
     private function accessibleConstants(ReflectionClass $class) : array
     {
-        return array_filter($class->getReflectionConstants(), function (ReflectionClassConstant $constant) : bool {
+        return array_filter($class->getReflectionConstants(), function(ReflectionClassConstant $constant) : bool {
             return $constant->isPublic() || $constant->isProtected();
         });
     }

src/DetectChanges/BCBreak/ClassBased/PropertyChanged.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
         return array_reduce(
             array_keys($commonProperties),
-            function (Changes $accumulator, string $propertyName) use ($propertiesFrom, $propertiesTo) : Changes {
+            function(Changes $accumulator, string $propertyName) use ($propertiesFrom, $propertiesTo) : Changes {
                 return $accumulator->mergeWith($this->checkProperty->__invoke(
                     $propertiesFrom[$propertyName],
                     $propertiesTo[$propertyName]

src/DetectChanges/BCBreak/ClassBased/MethodRemoved.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
             array_change_key_case($this->accessibleMethods($toClass), CASE_UPPER)
         );
 
-        return Changes::fromList(...array_values(array_map(function (ReflectionMethod $method) : Change {
+        return Changes::fromList(...array_values(array_map(function(ReflectionMethod $method) : Change {
             return Change::removed(
                 sprintf('Method %s was removed', $this->formatFunction->__invoke($method)),
                 true
@@ -46,12 +46,12 @@ 
     /** @return ReflectionMethod[] */
     private function accessibleMethods(ReflectionClass $class) : array
     {
-        $methods = array_filter($class->getMethods(), function (ReflectionMethod $method) : bool {
+        $methods = array_filter($class->getMethods(), function(ReflectionMethod $method) : bool {
             return $method->isPublic() || $method->isProtected();
         });
 
         return array_combine(
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 return $method->getName();
             }, $methods),
             $methods

src/DetectChanges/BCBreak/ClassBased/ClassBecameTrait.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 {
     public function __invoke(ReflectionClass $fromClass, ReflectionClass $toClass) : Changes
     {
-        if ($fromClass->isTrait() || ! $toClass->isTrait()) {
+        if ($fromClass->isTrait() || !$toClass->isTrait()) {
             return Changes::empty();
         }
 

src/DetectChanges/BCBreak/FunctionBased/MultipleChecksOnAFunction.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         return array_reduce(
             $this->checks,
-            function (Changes $changes, FunctionBased $check) use ($fromFunction, $toClass) : Changes {
+            function(Changes $changes, FunctionBased $check) use ($fromFunction, $toClass) : Changes {
                 return $changes->mergeWith($check->__invoke($fromFunction, $toClass));
             },
             Changes::empty()

src/DetectChanges/BCBreak/TraitBased/MultipleChecksOnATrait.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     {
         return array_reduce(
             $this->checks,
-            function (Changes $changes, TraitBased $check) use ($fromTrait, $toTrait) : Changes {
+            function(Changes $changes, TraitBased $check) use ($fromTrait, $toTrait) : Changes {
                 return $changes->mergeWith($check->__invoke($fromTrait, $toTrait));
             },
             Changes::empty()