@@ -1,5 +1,5 @@ discard block |
||
1 | 1 | <?php |
2 | -declare(strict_types=1); |
|
2 | +declare(strict_types = 1); |
|
3 | 3 | |
4 | 4 | namespace Roave\ApiCompare\Command; |
5 | 5 | |
@@ -113,14 +113,14 @@ discard block |
||
113 | 113 | $toRevision = $this->parseRevision->fromStringForRepository($input->getOption('to'), $sourceRepo); |
114 | 114 | $sourcesPath = $input->getArgument('sources-path'); |
115 | 115 | |
116 | - $output->writeln(sprintf('Comparing from %s to %s...', (string)$fromRevision, (string)$toRevision)); |
|
116 | + $output->writeln(sprintf('Comparing from %s to %s...', (string) $fromRevision, (string) $toRevision)); |
|
117 | 117 | |
118 | 118 | $fromPath = $this->git->checkout($sourceRepo, $fromRevision); |
119 | 119 | $toPath = $this->git->checkout($sourceRepo, $toRevision); |
120 | 120 | |
121 | 121 | try { |
122 | - $fromSources = $fromPath . '/' . $sourcesPath; |
|
123 | - $toSources = $toPath . '/' . $sourcesPath; |
|
122 | + $fromSources = $fromPath.'/'.$sourcesPath; |
|
123 | + $toSources = $toPath.'/'.$sourcesPath; |
|
124 | 124 | |
125 | 125 | Assert::that($fromSources)->directory(); |
126 | 126 | Assert::that($toSources)->directory(); |
@@ -134,8 +134,8 @@ discard block |
||
134 | 134 | new MethodVisibilityReduced() |
135 | 135 | ) |
136 | 136 | ))->compare( |
137 | - $this->reflectorFactory->__invoke((string)$fromPath . '/' . $sourcesPath), |
|
138 | - $this->reflectorFactory->__invoke((string)$toPath . '/' . $sourcesPath) |
|
137 | + $this->reflectorFactory->__invoke((string) $fromPath.'/'.$sourcesPath), |
|
138 | + $this->reflectorFactory->__invoke((string) $toPath.'/'.$sourcesPath) |
|
139 | 139 | ) |
140 | 140 | ); |
141 | 141 | } finally { |
@@ -153,7 +153,7 @@ discard block |
||
153 | 153 | private function parseRevisionFromInput(InputInterface $input, CheckedOutRepository $repository) : Revision |
154 | 154 | { |
155 | 155 | return $this->parseRevision->fromStringForRepository( |
156 | - (string)$input->getOption('from'), |
|
156 | + (string) $input->getOption('from'), |
|
157 | 157 | $repository |
158 | 158 | ); |
159 | 159 | } |
@@ -1,6 +1,6 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | |
3 | -declare(strict_types=1); |
|
3 | +declare(strict_types = 1); |
|
4 | 4 | |
5 | 5 | namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased; |
6 | 6 | |
@@ -21,7 +21,7 @@ discard block |
||
21 | 21 | { |
22 | 22 | return array_reduce( |
23 | 23 | $this->checks, |
24 | - function (Changes $changes, ClassBased $check) use ($fromClass, $toClass) : Changes { |
|
24 | + function(Changes $changes, ClassBased $check) use ($fromClass, $toClass) : Changes { |
|
25 | 25 | return $changes->mergeWith($check->compare($fromClass, $toClass)); |
26 | 26 | }, |
27 | 27 | Changes::new() |
@@ -1,6 +1,6 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | |
3 | -declare(strict_types=1); |
|
3 | +declare(strict_types = 1); |
|
4 | 4 | |
5 | 5 | namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased; |
6 | 6 | |
@@ -21,7 +21,7 @@ discard block |
||
21 | 21 | array_change_key_case($this->accessibleMethods($toClass), \CASE_UPPER) |
22 | 22 | ); |
23 | 23 | |
24 | - return Changes::fromArray(array_values(array_map(function (ReflectionMethod $method) use ($fromClass) : Change { |
|
24 | + return Changes::fromArray(array_values(array_map(function(ReflectionMethod $method) use ($fromClass) : Change { |
|
25 | 25 | return Change::removed( |
26 | 26 | sprintf('Method %s#%s() was removed', $fromClass->getName(), $method->getName()), |
27 | 27 | true |
@@ -32,12 +32,12 @@ discard block |
||
32 | 32 | /** @return ReflectionMethod[] */ |
33 | 33 | private function accessibleMethods(ReflectionClass $class) : array |
34 | 34 | { |
35 | - $methods = array_filter($class->getMethods(), function (ReflectionMethod $method) : bool { |
|
35 | + $methods = array_filter($class->getMethods(), function(ReflectionMethod $method) : bool { |
|
36 | 36 | return $method->isPublic() || $method->isProtected(); |
37 | 37 | }); |
38 | 38 | |
39 | 39 | return array_combine( |
40 | - array_map(function (ReflectionMethod $method) : string { |
|
40 | + array_map(function(ReflectionMethod $method) : string { |
|
41 | 41 | return $method->getName(); |
42 | 42 | }, $methods), |
43 | 43 | $methods |
@@ -1,6 +1,6 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | |
3 | -declare(strict_types=1); |
|
3 | +declare(strict_types = 1); |
|
4 | 4 | |
5 | 5 | namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased; |
6 | 6 | |
@@ -29,20 +29,20 @@ discard block |
||
29 | 29 | array_combine( |
30 | 30 | array_keys(array_intersect_key($visibilitiesFrom, $visibilitiesTo)), |
31 | 31 | array_map( |
32 | - function (string $visibilityFrom, string $visibilityTo) : array { |
|
32 | + function(string $visibilityFrom, string $visibilityTo) : array { |
|
33 | 33 | return [$visibilityFrom, $visibilityTo]; |
34 | 34 | }, |
35 | 35 | array_intersect_key($visibilitiesFrom, $visibilitiesTo), |
36 | 36 | array_intersect_key($visibilitiesTo, $visibilitiesFrom) |
37 | 37 | ) |
38 | 38 | ), |
39 | - function (array $visibilities) : bool { |
|
39 | + function(array $visibilities) : bool { |
|
40 | 40 | // Note: works because public, protected and private are (luckily) sortable |
41 | 41 | return $visibilities[0] > $visibilities[1]; |
42 | 42 | } |
43 | 43 | ); |
44 | 44 | |
45 | - return Changes::fromArray(array_values(array_map(function (string $methodName, array $visibilities) use ( |
|
45 | + return Changes::fromArray(array_values(array_map(function(string $methodName, array $visibilities) use ( |
|
46 | 46 | $fromClass |
47 | 47 | ) : Change { |
48 | 48 | return Change::changed( |
@@ -64,10 +64,10 @@ discard block |
||
64 | 64 | $methods = $class->getMethods(); |
65 | 65 | |
66 | 66 | return array_combine( |
67 | - array_map(function (ReflectionMethod $method) : string { |
|
67 | + array_map(function(ReflectionMethod $method) : string { |
|
68 | 68 | return $method->getName(); |
69 | 69 | }, $methods), |
70 | - array_map(function (ReflectionMethod $method) : string { |
|
70 | + array_map(function(ReflectionMethod $method) : string { |
|
71 | 71 | if ($method->isPublic()) { |
72 | 72 | return self::VISIBILITY_PUBLIC; |
73 | 73 | } |
@@ -1,6 +1,6 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | |
3 | -declare(strict_types=1); |
|
3 | +declare(strict_types = 1); |
|
4 | 4 | |
5 | 5 | namespace Roave\ApiCompare\Comparator\BackwardsCompatibility\ClassBased; |
6 | 6 | |
@@ -29,20 +29,20 @@ discard block |
||
29 | 29 | array_combine( |
30 | 30 | array_keys(array_intersect_key($visibilitiesFrom, $visibilitiesTo)), |
31 | 31 | array_map( |
32 | - function (string $visibilityFrom, string $visibilityTo) : array { |
|
32 | + function(string $visibilityFrom, string $visibilityTo) : array { |
|
33 | 33 | return [$visibilityFrom, $visibilityTo]; |
34 | 34 | }, |
35 | 35 | array_intersect_key($visibilitiesFrom, $visibilitiesTo), |
36 | 36 | array_intersect_key($visibilitiesTo, $visibilitiesFrom) |
37 | 37 | ) |
38 | 38 | ), |
39 | - function (array $visibilities) : bool { |
|
39 | + function(array $visibilities) : bool { |
|
40 | 40 | // Note: works because public, protected and private are (luckily) sortable |
41 | 41 | return $visibilities[0] > $visibilities[1]; |
42 | 42 | } |
43 | 43 | ); |
44 | 44 | |
45 | - return Changes::fromArray(array_values(array_map(function (string $propertyName, array $visibilities) use ( |
|
45 | + return Changes::fromArray(array_values(array_map(function(string $propertyName, array $visibilities) use ( |
|
46 | 46 | $fromClass |
47 | 47 | ) : Change { |
48 | 48 | return Change::changed( |
@@ -61,7 +61,7 @@ discard block |
||
61 | 61 | /** @return string[] */ |
62 | 62 | private function propertyVisibilities(ReflectionClass $class) : array |
63 | 63 | { |
64 | - return array_map(function (ReflectionProperty $property) : string { |
|
64 | + return array_map(function(ReflectionProperty $property) : string { |
|
65 | 65 | if ($property->isPublic()) { |
66 | 66 | return self::VISIBILITY_PUBLIC; |
67 | 67 | } |