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)

src/DiscoveryPlugin.php

Unused Use Statements -4 removed lines

@@ -6,17 +6,13 @@
 
 use Composer\Composer;
 use Composer\EventDispatcher\EventSubscriberInterface;
-use Composer\Factory;
 use Composer\IO\IOInterface;
-use Composer\Package\PackageInterface;
 use Composer\Plugin\Capability\CommandProvider;
 use Composer\Plugin\Capable;
 use Composer\Plugin\PluginInterface;
 use Composer\Script\Event;
 use Composer\Script\ScriptEvents;
-use Symfony\Component\Filesystem\Filesystem;
 use TheCodingMachine\Discovery\Commands\CommandProvider as DiscoveryCommandProvider;
-use TheCodingMachine\Discovery\Commands\DumpCommand;
 
 class DiscoveryPlugin implements PluginInterface, EventSubscriberInterface, Capable
 {

Spacing +1 added lines, -1 removed lines

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

src/AssetOperation.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
         $simple = $this->asset->toSimpleArray();
         if ($this->operation === self::REMOVE) {
             if (is_string($simple)) {
-                $simple = [ 'value' => $simple ];
+                $simple = ['value' => $simple];
             }
             $simple['action'] = self::REMOVE;
         }

src/Commands/RemoveAssetCommand.php

Unused Use Statements -4 removed lines

@@ -4,15 +4,11 @@
 namespace TheCodingMachine\Discovery\Commands;
 
 
-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 +3 added lines, -3 removed lines

@@ -22,9 +22,9 @@
     protected function configure()
     {
         $this->setName('discovery:remove')
-             ->setDescription('Remove an asset')
-             ->addArgument('asset-type', InputArgument::REQUIRED, 'The asset type')
-             ->addArgument('value', InputArgument::REQUIRED, 'The asset to remove');
+                ->setDescription('Remove an asset')
+                ->addArgument('asset-type', InputArgument::REQUIRED, 'The asset type')
+                ->addArgument('value', InputArgument::REQUIRED, 'The asset to remove');
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)

src/AssetsBuilder.php

Doc Comments -1 removed lines

@@ -42,7 +42,6 @@
      * Find discovery.json files in the passed repository and builds an asset type.
      *
      * @param RepositoryInterface $repository
-     * @param RootPackageInterface $rootPackage
      * @return array|AssetType[]
      */
     public function findAssetTypes(RepositoryInterface $repository) : array

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
         $orderedPackageList = PackagesOrderer::reorderPackages($unorderedPackagesList);
 
-        $packages = array_filter($orderedPackageList, function (PackageInterface $package) {
+        $packages = array_filter($orderedPackageList, function(PackageInterface $package) {
             $packageInstallPath = $this->getInstallPath($package);
 
             return file_exists($packageInstallPath.'/discovery.json');

src/Dumper.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         $assetTypes = $this->getAssetsBuilder()->findAssetTypes($compositeRepos);
 
         // Let's get an array of values, indexed by asset type (to store in the discovery_values.php file)
-        $values = array_map(function (AssetType $assetType) {
+        $values = array_map(function(AssetType $assetType) {
             return $assetType->getValues();
         }, $assetTypes);
 
@@ -51,7 +51,7 @@ 
 return '.var_export($values, true).";\n");
 
         // Let's get an array of assetTypes, indexed by asset type (to store in the discovery_asset_types.php file)
-        $assetTypes = array_map(function (AssetType $assetType) {
+        $assetTypes = array_map(function(AssetType $assetType) {
             return $assetType->jsonSerialize();
         }, $assetTypes);