maglnet / ComposerRequireChecker

src/ComposerRequireChecker/DefinedSymbolsLocator/LocateDefinedSymbolsFromExtensions.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
     public function __invoke(array $extensionNames) : array
     {
         $definedSymbols = [];
-        foreach($extensionNames as $extensionName) {
-            try{
+        foreach ($extensionNames as $extensionName) {
+            try {
                 $extensionReflection = new \ReflectionExtension($extensionName);
                 $definedSymbols = array_merge(
                     $definedSymbols,

src/ComposerRequireChecker/NodeVisitor/DefinedSymbolCollector.php

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
      */
     private function recordDefinitionOf(Node $node)
     {
-        if (! isset($node->namespacedName)) {
+        if (!isset($node->namespacedName)) {
             throw new \UnexpectedValueException(sprintf(
                 'Given node of type "%s" (defined at line %s)does not have an assigned "namespacedName" property: '
                 . 'did you pass it through a name resolver visitor?',

src/ComposerRequireChecker/Cli/CheckCommand.php

Unused Use Statements -1 removed lines

@@ -21,7 +21,6 @@
 use PhpParser\ParserFactory;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\Table;
-use Symfony\Component\Console\Helper\TableCell;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;

Spacing +12 added lines, -12 removed lines

@@ -52,7 +52,7 @@ 
     protected function execute(InputInterface $input, OutputInterface $output) : int
     {
 
-        if(!$output->isQuiet()) {
+        if (!$output->isQuiet()) {
             $output->writeln($this->getApplication()->getLongVersion());
         }
 
@@ -61,7 +61,7 @@ 
 
         $getPackageSourceFiles = new LocateComposerPackageSourceFiles();
 
-        $sourcesASTs  = new LocateASTFromFiles((new ParserFactory())->create(ParserFactory::PREFER_PHP7));
+        $sourcesASTs = new LocateASTFromFiles((new ParserFactory())->create(ParserFactory::PREFER_PHP7));
 
         $definedVendorSymbols = (new LocateDefinedSymbolsFromASTRoots())->__invoke($sourcesASTs(
             (new ComposeGenerators())->__invoke(
@@ -96,7 +96,7 @@ 
         $guesser = new DependencyGuesser();
         foreach ($unknownSymbols as $unknownSymbol) {
             $guessedDependencies = [];
-            foreach($guesser($unknownSymbol) as $guessedDependency) {
+            foreach ($guesser($unknownSymbol) as $guessedDependency) {
                 $guessedDependencies[] = $guessedDependency;
             }
             $table->addRow([$unknownSymbol, implode("\n", $guessedDependencies)]);
@@ -109,17 +109,17 @@ 
     private function getCheckOptions(InputInterface $input) : Options
     {
         $fileName = $input->getOption('config-file');
-        if(!$fileName) {
+        if (!$fileName) {
             return new Options();
         }
 
-        if(!is_readable($fileName)) {
-            throw new \InvalidArgumentException('unable to read ' . $fileName);
+        if (!is_readable($fileName)) {
+            throw new \InvalidArgumentException('unable to read '.$fileName);
         }
 
         $jsonData = json_decode(file_get_contents($fileName), true);
-        if(false === $jsonData) {
-            throw new \Exception('error parsing the config file: ' . json_last_error_msg());
+        if (false === $jsonData) {
+            throw new \Exception('error parsing the config file: '.json_last_error_msg());
         }
 
         return new Options($jsonData);
@@ -133,12 +133,12 @@ 
      */
     private function checkJsonFile(string $jsonFile)
     {
-        if(!is_readable($jsonFile)) {
-            throw new InvalidInputFileException('cannot read ' . $jsonFile);
+        if (!is_readable($jsonFile)) {
+            throw new InvalidInputFileException('cannot read '.$jsonFile);
         }
 
-        if(false == json_decode(file_get_contents($jsonFile))) {
-            throw new InvalidInputFileException('error parsing ' . $jsonFile . ': ' . json_last_error_msg());
+        if (false == json_decode(file_get_contents($jsonFile))) {
+            throw new InvalidInputFileException('error parsing '.$jsonFile.': '.json_last_error_msg());
         }
 
     }

src/ComposerRequireChecker/DependencyGuesser/DependencyGuesser.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 
     public function __invoke($symbolName) : \Generator
     {
-        foreach($this->guessers as $guesser) {
+        foreach ($this->guessers as $guesser) {
             yield from $guesser($symbolName);
         }
     }

example/test.php

Spacing +4 added lines, -4 removed lines

@@ -10,16 +10,16 @@
 
 // Please run "run-test.sh", and not this file directly.
 
-(function () {
-    require_once  __DIR__ . '/../vendor/autoload.php';
+(function() {
+    require_once  __DIR__.'/../vendor/autoload.php';
 
     $whitelistSymbols = ['null', 'true', 'false', 'static', 'self', 'parent'];
-    $defaultExtensions = ["Core","standard"]; // "bcmath","calendar","ctype","date","filter","hash","iconv","json","mcrypt","SPL","pcre","Reflection","session","standard","mysqlnd","tokenizer","zip","zlib","libxml","dom","PDO","Phar","SimpleXML","xml","wddx","xmlreader","xmlwriter"];
+    $defaultExtensions = ["Core", "standard"]; // "bcmath","calendar","ctype","date","filter","hash","iconv","json","mcrypt","SPL","pcre","Reflection","session","standard","mysqlnd","tokenizer","zip","zlib","libxml","dom","PDO","Phar","SimpleXML","xml","wddx","xmlreader","xmlwriter"];
 
     $getPackageSourceFiles = new LocateComposerPackageSourceFiles();
 
     $sourcesASTs  = new LocateASTFromFiles((new ParserFactory())->create(ParserFactory::PREFER_PHP7));
-    $composerJson = __DIR__ . '/test-data/zend-feed/composer.json';
+    $composerJson = __DIR__.'/test-data/zend-feed/composer.json';
 
     $definedVendorSymbols = (new LocateDefinedSymbolsFromASTRoots())->__invoke($sourcesASTs(
         (new ComposeGenerators())->__invoke(

src/ComposerRequireChecker/Cli/Application.php

Spacing +4 added lines, -4 removed lines

@@ -25,17 +25,17 @@
     {
         $version = null;
         $pharFile = \Phar::running();
-        if($pharFile) {
+        if ($pharFile) {
             $metadata = (new \Phar($pharFile))->getMetadata();
             $version = $metadata['version'] ?? null;
         }
 
-        if(!$version) {
+        if (!$version) {
             $pwd = getcwd();
-            chdir(realpath(__DIR__ . '/../../../'));
+            chdir(realpath(__DIR__.'/../../../'));
             $gitVersion = @exec('git describe --tags --dirty=-dev --always 2>&1', $output, $returnValue);
             chdir($pwd);
-            if($returnValue === 0) {
+            if ($returnValue === 0) {
                 $version = $gitVersion;
             }
         }

src/ComposerRequireChecker/Cli/Options.php

Spacing +2 added lines, -2 removed lines

@@ -25,8 +25,8 @@
     {
         foreach ($options as $key => $option) {
             $methodName = 'set'.$this->getCamelCase($key);
-            if(!method_exists($this, $methodName)) {
-                throw new \InvalidArgumentException($key . ' is not a known option - there is no method ' . $methodName);
+            if (!method_exists($this, $methodName)) {
+                throw new \InvalidArgumentException($key.' is not a known option - there is no method '.$methodName);
             }
             $this->$methodName($option);
         }

src/ComposerRequireChecker/DependencyGuesser/GuessFromLoadedExtensions.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         foreach ($this->loadedExtensions as $extensionName) {
             $extensionSymbols = $definedSymbolsFromExtensions([$extensionName]);
             if (in_array($symbolName, $extensionSymbols)) {
-                yield 'ext-' . $extensionName;
+                yield 'ext-'.$extensionName;
             }
         }
     }

src/ComposerRequireChecker/FileLocator/LocateAllFilesByExtension.php

Spacing +2 added lines, -2 removed lines

@@ -21,11 +21,11 @@
 
     private function filterFilesByExtension(Traversable $files, string $fileExtension) : Traversable
     {
-        $extensionMatcher = '/.*' . preg_quote($fileExtension) . '$/';
+        $extensionMatcher = '/.*'.preg_quote($fileExtension).'$/';
 
         /* @var $file \SplFileInfo */
         foreach ($files as $file) {
-            if (! preg_match($extensionMatcher, $file->getBasename())) {
+            if (!preg_match($extensionMatcher, $file->getBasename())) {
                 continue;
             }