thecodingmachine / discovery

src/Utils/JsonException.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\Discovery\Utils;
 

src/AssetType.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
      * Similar to a usort except that if 2 priorities are equal, order is preserved.
      * Shamelessly copied from http://php.net/manual/fr/function.usort.php#38827
      *
-     * @param $array
+     * @param Asset[] $array
      * @param string $cmp_function
      */
     private function stableSort(&$array, $cmp_function = 'strcmp')

Spacing +4 added lines, -4 removed lines

@@ -36,7 +36,7 @@ 
             $this->assets[] = $operation->getAsset();
         } else {
             // This is a remove!
-            $this->assets = array_values(array_filter($this->assets, function (Asset $asset) use ($operation) {
+            $this->assets = array_values(array_filter($this->assets, function(Asset $asset) use ($operation) {
                 return $asset->getValue() !== $operation->getAsset()->getValue();
             }));
         }
@@ -54,7 +54,7 @@ 
         }
 
         // Let's order assets by priority.
-        $this->stableSort($this->assets, function (Asset $asset1, Asset $asset2) {
+        $this->stableSort($this->assets, function(Asset $asset1, Asset $asset2) {
             return $asset2->getPriority() <=> $asset1->getPriority();
         });
 
@@ -129,7 +129,7 @@ 
      */
     public function getValues() : array
     {
-        return array_map(function (Asset $asset) {
+        return array_map(function(Asset $asset) {
             return $asset->getValue();
         }, $this->getAssets());
     }
@@ -143,7 +143,7 @@ 
      */
     public function jsonSerialize()
     {
-        return array_map(function (Asset $asset) {
+        return array_map(function(Asset $asset) {
             return $asset->jsonSerialize();
         }, $this->getAssets());
     }

src/PackagesOrderer.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\Discovery;
 
@@ -27,7 +27,7 @@ 
     {
         // The very first step is to reorder the packages alphabetically.
         // This is to ensure the same order every time, even between packages that are unrelated.
-        usort($unorderedPackagesList, function (PackageInterface $packageA, PackageInterface $packageB) {
+        usort($unorderedPackagesList, function(PackageInterface $packageA, PackageInterface $packageB) {
             return strcmp($packageA->getName(), $packageB->getName());
         });
 

src/ImmutableAssetType.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      */
     public static function fromArray(string $name, array $assetsArray) : ImmutableAssetType
     {
-        $assets = array_map(function (array $assetArray) {
+        $assets = array_map(function(array $assetArray) {
             return Asset::fromArray($assetArray);
         }, $assetsArray);
         return new self($name, $assets);

src/Utils/IOException.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\Discovery\Utils;
 

src/DiscoveryInterface.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace TheCodingMachine\Discovery;
 

src/Commands/ListAssetTypesCommand.php

Indentation +3 added lines, -3 removed lines

@@ -18,9 +18,9 @@
     protected function configure()
     {
         $this->setName('discovery:list')
-             ->setDescription('List the values of the asset type passed in parameter (or list all asset types if no parameter is passed)')
-             ->addArgument('asset-type', InputArgument::OPTIONAL, 'The asset type to list')
-             ->addOption('format', null, InputOption::VALUE_REQUIRED, 'The output format (txt or json)', 'txt');
+                ->setDescription('List the values of the asset type passed in parameter (or list all asset types if no parameter is passed)')
+                ->addArgument('asset-type', InputArgument::OPTIONAL, 'The asset type to list')
+                ->addOption('format', null, InputOption::VALUE_REQUIRED, 'The output format (txt or json)', 'txt');
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
                 $output->writeln(sprintf('<error>Could not find the "%s" asset type.</error>', $assetTypeName));
                 return;
             }
-            $assetTypes = [ $assetTypes[$assetTypeName] ];
+            $assetTypes = [$assetTypes[$assetTypeName]];
         }
 
         switch ($input->getOption('format')) {

src/Commands/AddAssetCommand.php

Unused Use Statements -3 removed lines

@@ -5,15 +5,12 @@
 
 
 use Composer\Command\BaseCommand;
-use Composer\Factory;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use TheCodingMachine\Discovery\Asset;
 use TheCodingMachine\Discovery\AssetOperation;
-use TheCodingMachine\Discovery\AssetsBuilder;
-use TheCodingMachine\Discovery\AssetType;
 use TheCodingMachine\Discovery\DiscoveryFileLoader;
 use TheCodingMachine\Discovery\Dumper;
 

Indentation +4 added lines, -4 removed lines

@@ -22,10 +22,10 @@
     protected function configure()
     {
         $this->setName('discovery:add')
-             ->setDescription('Add an asset')
-             ->addArgument('asset-type', InputArgument::REQUIRED, 'The asset type')
-             ->addArgument('value', InputArgument::REQUIRED, 'The asset to add')
-             ->addOption('priority', 'p', InputOption::VALUE_REQUIRED, 'The priority', 0.0);
+                ->setDescription('Add an asset')
+                ->addArgument('asset-type', InputArgument::REQUIRED, 'The asset type')
+                ->addArgument('value', InputArgument::REQUIRED, 'The asset to add')
+                ->addOption('priority', 'p', InputOption::VALUE_REQUIRED, 'The priority', 0.0);
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)

src/Commands/DumpCommand.php

Unused Use Statements -5 removed lines

@@ -5,13 +5,8 @@
 
 
 use Composer\Command\BaseCommand;
-use Composer\Factory;
-use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
-use TheCodingMachine\Discovery\AssetsBuilder;
-use TheCodingMachine\Discovery\AssetType;
 use TheCodingMachine\Discovery\Dumper;
 
 class DumpCommand extends BaseCommand

Indentation +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     protected function configure()
     {
         $this->setName('discovery:dump')
-             ->setDescription('Regenerates the Discovery class (in the .discovery folder). This is automatically triggered on install and update.');
+                ->setDescription('Regenerates the Discovery class (in the .discovery folder). This is automatically triggered on install and update.');
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)