maglnet / ComposerRequireChecker

example/test.php

Spacing +2 added lines, -2 removed lines

@@ -10,11 +10,11 @@
 
 // Please run "run-test.sh", and not this file directly.
 
-(function () {
+(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();
 

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/FileLocator/LocateAllFilesByExtension.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 
         /* @var $file \SplFileInfo */
         foreach ($files as $file) {
-            if (! preg_match($extensionMatcher, $file->getBasename())) {
+            if (!preg_match($extensionMatcher, $file->getBasename())) {
                 continue;
             }
 

src/ComposerRequireChecker/FileLocator/LocateComposerPackageDirectDependenciesSourceFiles.php

Spacing +4 added lines, -4 removed lines

@@ -11,18 +11,18 @@
         $packageDir = dirname($composerJsonPath);
 
         $vendorDirs = array_values(array_map(
-            function (string $vendorName) use ($packageDir) {
+            function(string $vendorName) use ($packageDir) {
                 return $packageDir . '/vendor/' . $vendorName;
             },
-            array_keys(json_decode(file_get_contents($composerJsonPath), true)['require'] ?? [])
+            array_keys(json_decode(file_get_contents($composerJsonPath), true)['require'] ? ? [])
         ));
 
         foreach ($vendorDirs as $vendorDir) {
-            if (! file_exists($vendorDir . '/composer.json')) {
+            if (!file_exists($vendorDir . '/composer.json')) {
                 continue;
             }
 
-            yield from (new LocateComposerPackageSourceFiles())->__invoke($vendorDir . '/composer.json');
+            yield from(new LocateComposerPackageSourceFiles())->__invoke($vendorDir . '/composer.json');
         }
     }
 }

src/ComposerRequireChecker/FileLocator/LocateComposerPackageSourceFiles.php

Spacing +6 added lines, -6 removed lines

@@ -12,23 +12,23 @@ 
         $composerData = json_decode(file_get_contents($composerJsonPath), true);
 
         yield from $this->locateFilesInClassmapDefinitions(
-            $this->getFilePaths($composerData['autoload']['classmap'] ?? [], $packageDir)
+            $this->getFilePaths($composerData['autoload']['classmap'] ? ? [], $packageDir)
         );
         yield from $this->locateFilesInFilesInFilesDefinitions(
-            $this->getFilePaths($composerData['autoload']['files'] ?? [], $packageDir)
+            $this->getFilePaths($composerData['autoload']['files'] ? ? [], $packageDir)
         );
         yield from $this->locateFilesInPsr0Definitions(
-            $this->getFilePaths($composerData['autoload']['psr-0'] ?? [], $packageDir)
+            $this->getFilePaths($composerData['autoload']['psr-0'] ? ? [], $packageDir)
         );
         yield from $this->locateFilesInPsr4Definitions(
-            $this->getFilePaths($composerData['autoload']['psr-4'] ?? [], $packageDir)
+            $this->getFilePaths($composerData['autoload']['psr-4'] ? ? [], $packageDir)
         );
     }
 
     private function getFilePaths(array $sourceDirs, string $packageDir) : array
     {
         return array_values(array_map(
-            function (string $sourceDir) use ($packageDir) {
+            function(string $sourceDir) use ($packageDir) {
                 return $packageDir . '/' . ltrim($sourceDir, '/');
             },
             $sourceDirs
@@ -63,6 +63,6 @@ 
 
     private function extractFilesFromDirectory(string $directory) : Generator
     {
-        yield from (new LocateAllFilesByExtension())->__invoke(new \ArrayIterator([$directory]), '.php');
+        yield from(new LocateAllFilesByExtension())->__invoke(new \ArrayIterator([$directory]), '.php');
     }
 }

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/NodeVisitor/UsedSymbolCollector.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
 
     private function recordTraitUsage(Node $node)
     {
-        if (! $node instanceof Node\Stmt\TraitUse) {
+        if (!$node instanceof Node\Stmt\TraitUse) {
             return;
         }
 

src/ComposerRequireChecker/DefinedExtensionsResolver/DefinedExtensionsResolver.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
     public function __invoke(string $composerJson, array $phpCoreExtensions = []) : array
     {
-        $requires = json_decode(file_get_contents($composerJson), true)['require'] ?? [];
+        $requires = json_decode(file_get_contents($composerJson), true)['require'] ? ? [];
 
         $extensions = [];
         foreach ($requires as $require => $version) {

src/ComposerRequireChecker/Cli/CheckCommand.php

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: matthias
- * Date: 28.11.15
- * Time: 16:16
- */
+     * Created by PhpStorm.
+     * User: matthias
+     * Date: 28.11.15
+     * Time: 16:16
+     */
 
 namespace ComposerRequireChecker\Cli;
 

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 +8 added lines, -8 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,16 +109,16 @@ 
     private function getCheckOptions(InputInterface $input) : Options
     {
         $fileName = $input->getOption('config-file');
-        if(!$fileName) {
+        if (!$fileName) {
             return new Options();
         }
 
-        if(!is_readable($fileName)) {
+        if (!is_readable($fileName)) {
             throw new \InvalidArgumentException('unable to read ' . $fileName);
         }
 
         $jsonData = json_decode(file_get_contents($fileName), true);
-        if(false === $jsonData) {
+        if (false === $jsonData) {
             throw new \Exception('error parsing the config file: ' . json_last_error_msg());
         }
 
@@ -133,11 +133,11 @@ 
      */
     private function checkJsonFile(string $jsonFile)
     {
-        if(!is_readable($jsonFile)) {
+        if (!is_readable($jsonFile)) {
             throw new InvalidInputFileException('cannot read ' . $jsonFile);
         }
 
-        if(false == json_decode(file_get_contents($jsonFile))) {
+        if (false == json_decode(file_get_contents($jsonFile))) {
             throw new InvalidInputFileException('error parsing ' . $jsonFile . ': ' . json_last_error_msg());
         }