Completed
Pull Request — master (#34)
by Matthias
04:58
created
DefinedSymbolsLocator/LocateDefinedSymbolsFromExtensions.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -16,8 +16,8 @@
 block discarded – undo
16 16
     public function __invoke(array $extensionNames) : array
17 17
     {
18 18
         $definedSymbols = [];
19
-        foreach($extensionNames as $extensionName) {
20
-            try{
19
+        foreach ($extensionNames as $extensionName) {
20
+            try {
21 21
                 $extensionReflection = new \ReflectionExtension($extensionName);
22 22
                 $definedSymbols = array_merge(
23 23
                     $definedSymbols,
Please login to merge, or discard this patch.
src/ComposerRequireChecker/FileLocator/LocateAllFilesByExtension.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
 
26 26
         /* @var $file \SplFileInfo */
27 27
         foreach ($files as $file) {
28
-            if (! preg_match($extensionMatcher, $file->getBasename())) {
28
+            if (!preg_match($extensionMatcher, $file->getBasename())) {
29 29
                 continue;
30 30
             }
31 31
 
Please login to merge, or discard this patch.
src/ComposerRequireChecker/Cli/Options.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,8 +18,8 @@
 block discarded – undo
18 18
     public function __construct(array $options = [])
19 19
     {
20 20
         foreach ($options as $key => $option) {
21
-            $methodName = 'set'.$this->getCamelCase($key);
22
-            if(!method_exists($this, $methodName)) {
21
+            $methodName = 'set' . $this->getCamelCase($key);
22
+            if (!method_exists($this, $methodName)) {
23 23
                 throw new \InvalidArgumentException($key . ' is not a known option - there is no method ' . $methodName);
24 24
             }
25 25
             $this->$methodName($option);
Please login to merge, or discard this patch.
src/ComposerRequireChecker/DependencyGuesser/DependencyGuesser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
 
19 19
     public function __invoke($symbolName) : \Generator
20 20
     {
21
-        foreach($this->guessers as $guesser) {
21
+        foreach ($this->guessers as $guesser) {
22 22
             yield from $guesser($symbolName);
23 23
         }
24 24
     }
Please login to merge, or discard this patch.
test/ComposerRequireCheckerTest/DependencyGuesser/DependencyGuesserTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
 
21 21
     public function testGuessExtJson()
22 22
     {
23
-        if(!extension_loaded('json')) {
23
+        if (!extension_loaded('json')) {
24 24
             $this->markTestSkipped('extension json is not available');
25 25
         }
26 26
         $result = $this->guesser->__invoke('json_decode');
Please login to merge, or discard this patch.
src/ComposerRequireChecker/FileLocator/LocateComposerPackageSourceFiles.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,13 +29,13 @@
 block discarded – undo
29 29
     {
30 30
         $flattened = array_reduce(
31 31
             $sourceDirs,
32
-            function (array $sourceDirs, $sourceDir) {
32
+            function(array $sourceDirs, $sourceDir) {
33 33
                 return array_merge($sourceDirs, (array)$sourceDir);
34 34
             },
35 35
             []
36 36
         );
37 37
         return array_values(array_map(
38
-            function (string $sourceDir) use ($packageDir) {
38
+            function(string $sourceDir) use ($packageDir) {
39 39
                 return $this->normalizePath($packageDir . '/' . ltrim($sourceDir, '/'));
40 40
             },
41 41
             $flattened
Please login to merge, or discard this patch.
FileLocator/LocateComposerPackageDirectDependenciesSourceFiles.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -11,14 +11,14 @@
 block discarded – undo
11 11
         $packageDir = dirname($composerJsonPath);
12 12
 
13 13
         $vendorDirs = array_values(array_map(
14
-            function (string $vendorName) use ($packageDir) {
14
+            function(string $vendorName) use ($packageDir) {
15 15
                 return $packageDir . '/vendor/' . $vendorName;
16 16
             },
17 17
             array_keys(json_decode(file_get_contents($composerJsonPath), true)['require'] ?? [])
18 18
         ));
19 19
 
20 20
         foreach ($vendorDirs as $vendorDir) {
21
-            if (! file_exists($vendorDir . '/composer.json')) {
21
+            if (!file_exists($vendorDir . '/composer.json')) {
22 22
                 continue;
23 23
             }
24 24
 
Please login to merge, or discard this patch.
src/ComposerRequireChecker/Cli/Application.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -25,17 +25,17 @@
 block discarded – undo
25 25
     {
26 26
         $version = null;
27 27
         $pharFile = \Phar::running();
28
-        if($pharFile) {
28
+        if ($pharFile) {
29 29
             $metadata = (new \Phar($pharFile))->getMetadata();
30 30
             $version = $metadata['version'] ?? null;
31 31
         }
32 32
 
33
-        if(!$version) {
33
+        if (!$version) {
34 34
             $pwd = getcwd();
35 35
             chdir(realpath(__DIR__ . '/../../../'));
36 36
             $gitVersion = @exec('git describe --tags --dirty=-dev --always 2>&1', $output, $returnValue);
37 37
             chdir($pwd);
38
-            if($returnValue === 0) {
38
+            if ($returnValue === 0) {
39 39
                 $version = $gitVersion;
40 40
             }
41 41
         }
Please login to merge, or discard this patch.
src/ComposerRequireChecker/Cli/CheckCommand.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
     protected function execute(InputInterface $input, OutputInterface $output) : int
46 46
     {
47 47
 
48
-        if(!$output->isQuiet()) {
48
+        if (!$output->isQuiet()) {
49 49
             $output->writeln($this->getApplication()->getLongVersion());
50 50
         }
51 51
 
52 52
         $composerJson = realpath($input->getArgument('composer-json'));
53
-        if(false === $composerJson) {
53
+        if (false === $composerJson) {
54 54
             throw new \InvalidArgumentException('file not found: [' . $input->getArgument('composer-json') . ']');
55 55
         }
56 56
         $this->checkJsonFile($composerJson);
@@ -96,20 +96,20 @@  discard block
 block discarded – undo
96 96
         $guesser = new DependencyGuesser();
97 97
         foreach ($unknownSymbols as $unknownSymbol) {
98 98
             $guessedDependencies = [];
99
-            foreach($guesser($unknownSymbol) as $guessedDependency) {
99
+            foreach ($guesser($unknownSymbol) as $guessedDependency) {
100 100
                 $guessedDependencies[] = $guessedDependency;
101 101
             }
102 102
             $table->addRow([$unknownSymbol, implode("\n", $guessedDependencies)]);
103 103
         }
104 104
         $table->render();
105 105
 
106
-        return ((int) (bool) $unknownSymbols);
106
+        return ((int)(bool)$unknownSymbols);
107 107
     }
108 108
 
109 109
     private function getCheckOptions(InputInterface $input) : Options
110 110
     {
111 111
         $fileName = $input->getOption('config-file');
112
-        if(!$fileName) {
112
+        if (!$fileName) {
113 113
             return new Options();
114 114
         }
115 115
         return new Options((new JsonLoader($fileName))->getData());
Please login to merge, or discard this patch.