Roave / BackwardCompatibilityCheck

test/unit/Comparator/BackwardsCompatibility/ClassBased/ConstantVisibilityReducedTest.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -29,7 +29,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -43,11 +43,11 @@ 
         return [
             'RoaveTestAsset\\ClassWithConstantVisibilitiesBeingChanged' => [
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/old/ClassWithConstantVisibilitiesBeingChanged.php',
+                    __DIR__.'/../../../../asset/api/old/ClassWithConstantVisibilitiesBeingChanged.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantVisibilitiesBeingChanged'),
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/new/ClassWithConstantVisibilitiesBeingChanged.php',
+                    __DIR__.'/../../../../asset/api/new/ClassWithConstantVisibilitiesBeingChanged.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantVisibilitiesBeingChanged'),
                 [

test/unit/Comparator/BackwardsCompatibility/ClassBased/ConstantValueChangedTest.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -29,7 +29,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -43,11 +43,11 @@ 
         return [
             'RoaveTestAsset\\ClassWithConstantValuesBeingChanged' => [
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/old/ClassWithConstantValuesBeingChanged.php',
+                    __DIR__.'/../../../../asset/api/old/ClassWithConstantValuesBeingChanged.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantValuesBeingChanged'),
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/new/ClassWithConstantValuesBeingChanged.php',
+                    __DIR__.'/../../../../asset/api/new/ClassWithConstantValuesBeingChanged.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantValuesBeingChanged'),
                 [

test/unit/Comparator/BackwardsCompatibility/ClassBased/ConstantRemovedTest.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -29,7 +29,7 @@ 
 
         self::assertSame(
             $expectedMessages,
-            array_map(function (Change $change) : string {
+            array_map(function(Change $change) : string {
                 return $change->__toString();
             }, iterator_to_array($changes))
         );
@@ -43,11 +43,11 @@ 
         return [
             'RoaveTestAsset\\ClassWithConstantsBeingRemoved' => [
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/old/ClassWithConstantsBeingRemoved.php',
+                    __DIR__.'/../../../../asset/api/old/ClassWithConstantsBeingRemoved.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantsBeingRemoved'),
                 (new ClassReflector(new SingleFileSourceLocator(
-                    __DIR__ . '/../../../../asset/api/new/ClassWithConstantsBeingRemoved.php',
+                    __DIR__.'/../../../../asset/api/new/ClassWithConstantsBeingRemoved.php',
                     $locator
                 )))->reflect('RoaveTestAsset\\ClassWithConstantsBeingRemoved'),
                 [

src/Comparator/BackwardsCompatibility/ClassBased/ConstantRemoved.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -21,7 +21,7 @@ 
             $this->accessibleConstants($toClass)
         );
 
-        return Changes::fromArray(array_values(array_map(function (ReflectionClassConstant $constant) use ($fromClass) : Change {
+        return Changes::fromArray(array_values(array_map(function(ReflectionClassConstant $constant) use ($fromClass) : Change {
             return Change::removed(
                 sprintf('Constant %s::%s was removed', $fromClass->getName(), $constant->getName()),
                 true
@@ -32,7 +32,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/Comparator/BackwardsCompatibility/ClassBased/ConstantVisibilityReduced.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -30,19 +30,19 @@ 
             array_combine(
                 $sharedKeys,
                 array_map(
-                    function (string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
+                    function(string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
                         return [$visibilitiesFrom[$propertyName], $visibilitiesTo[$propertyName]];
                     },
                     $sharedKeys
                 )
             ),
-            function (array $visibilities) : bool {
+            function(array $visibilities) : bool {
                 // Note: works because public, protected and private are (luckily) sortable
                 return $visibilities[0] > $visibilities[1];
             }
         );
 
-        return Changes::fromArray(array_values(array_map(function (string $propertyName, array $visibilities) use (
+        return Changes::fromArray(array_values(array_map(function(string $propertyName, array $visibilities) use (
             $fromClass
         ) : Change {
             return Change::changed(
@@ -61,7 +61,7 @@ 
     /** @return string[] indexed by constant name */
     private function constantVisibilities(ReflectionClass $class) : array
     {
-        return array_map(function (ReflectionClassConstant $constant) : string {
+        return array_map(function(ReflectionClassConstant $constant) : string {
             if ($constant->isPublic()) {
                 return self::VISIBILITY_PUBLIC;
             }

src/Comparator/BackwardsCompatibility/ClassBased/ConstantValueChanged.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -19,13 +19,13 @@ 
         $fromValues = $this->accessibleConstantValues($fromClass);
         $toValues   = $this->accessibleConstantValues($toClass);
 
-        $changedConstants = array_keys(array_filter($fromValues, function ($constantValue, string $constantName) use (
+        $changedConstants = array_keys(array_filter($fromValues, function($constantValue, string $constantName) use (
             $toValues
         ) : bool {
             return array_key_exists($constantName, $toValues) && $constantValue !== $toValues[$constantName];
         }, \ARRAY_FILTER_USE_BOTH));
 
-        return Changes::fromArray(array_values(array_map(function (string $constantName) use ($fromClass) : Change {
+        return Changes::fromArray(array_values(array_map(function(string $constantName) use ($fromClass) : Change {
             return Change::changed(
                 sprintf('Value of constant %s::%s changed', $fromClass->getName(), $constantName),
                 true
@@ -38,12 +38,12 @@ 
     {
         $accessibleConstants = array_filter(
             $class->getReflectionConstants(),
-            function (ReflectionClassConstant $constant) : bool {
+            function(ReflectionClassConstant $constant) : bool {
                 return $constant->isPublic() || $constant->isProtected();
             }
         );
 
-        return array_map(function (ReflectionClassConstant $constant) {
+        return array_map(function(ReflectionClassConstant $constant) {
             return $constant->getValue();
         }, $accessibleConstants);
     }

src/Comparator/BackwardsCompatibility/ClassBased/MethodVisibilityReduced.php

Spacing +6 added lines, -6 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -30,19 +30,19 @@ 
             array_combine(
                 $sharedKeys,
                 array_map(
-                    function (string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
+                    function(string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
                         return [$visibilitiesFrom[$propertyName], $visibilitiesTo[$propertyName]];
                     },
                     $sharedKeys
                 )
             ),
-            function (array $visibilities) : bool {
+            function(array $visibilities) : bool {
                 // Note: works because public, protected and private are (luckily) sortable
                 return $visibilities[0] > $visibilities[1];
             }
         );
 
-        return Changes::fromArray(array_values(array_map(function (string $methodName, array $visibilities) use (
+        return Changes::fromArray(array_values(array_map(function(string $methodName, array $visibilities) use (
             $fromClass
         ) : Change {
             return Change::changed(
@@ -64,10 +64,10 @@ 
         $methods = $class->getMethods();
 
         return array_combine(
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 return $method->getName();
             }, $methods),
-            array_map(function (ReflectionMethod $method) : string {
+            array_map(function(ReflectionMethod $method) : string {
                 if ($method->isPublic()) {
                     return self::VISIBILITY_PUBLIC;
                 }

src/Comparator/BackwardsCompatibility/ClassBased/PropertyVisibilityReduced.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased;
 
@@ -30,19 +30,19 @@ 
             array_combine(
                 $sharedKeys,
                 array_map(
-                    function (string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
+                    function(string $propertyName) use ($visibilitiesFrom, $visibilitiesTo) : array {
                         return [$visibilitiesFrom[$propertyName], $visibilitiesTo[$propertyName]];
                     },
                     $sharedKeys
                 )
             ),
-            function (array $visibilities) : bool {
+            function(array $visibilities) : bool {
                 // Note: works because public, protected and private are (luckily) sortable
                 return $visibilities[0] > $visibilities[1];
             }
         );
 
-        return Changes::fromArray(array_values(array_map(function (string $propertyName, array $visibilities) use (
+        return Changes::fromArray(array_values(array_map(function(string $propertyName, array $visibilities) use (
             $fromClass
         ) : Change {
             return Change::changed(
@@ -61,7 +61,7 @@ 
     /** @return string[] */
     private function propertyVisibilities(ReflectionClass $class) : array
     {
-        return array_map(function (ReflectionProperty $property) : string {
+        return array_map(function(ReflectionProperty $property) : string {
             if ($property->isPublic()) {
                 return self::VISIBILITY_PUBLIC;
             }