thecodingmachine / discovery

src/PackagesOrderer.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,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/AssetType.php

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/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/Commands/ListAssetTypesCommand.php

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/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/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);
 

src/AssetsBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -53,14 +53,14 @@
         // This has been observed when doing a "composer install" on an empty vendor directory.
         // Let's ensure each package is represented only once.
         $dedupPackages = [];
-        foreach($unorderedPackagesList as $package) {
+        foreach ($unorderedPackagesList as $package) {
             $dedupPackages[$package->getName()] = $package;
         }
         $dedupPackages = array_values($dedupPackages);
 
         $orderedPackageList = PackagesOrderer::reorderPackages($dedupPackages);
 
-        $packages = array_filter($orderedPackageList, function (PackageInterface $package) {
+        $packages = array_filter($orderedPackageList, function(PackageInterface $package) {
             $packageInstallPath = $this->getInstallPath($package);
 
             return file_exists($packageInstallPath.'/discovery.json');

tests/unitTests/DiscoveryTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     {
         $result = Discovery::getInstance()->get('test-asset');
 
-        $this->assertSame([ 'a1', 'a2', 'b' ], $result);
+        $this->assertSame(['a1', 'a2', 'b'], $result);
     }
 
     public function testEmptyValues()

tests/unitTests/AssetsBuilderTest.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         $io = new BufferIO();
 
         $assetsBuilder = new AssetsBuilder($installationManager, $io, '.');
-        $assetTypes = $assetsBuilder->buildAssetTypes([ $packageA ]);
+        $assetTypes = $assetsBuilder->buildAssetTypes([$packageA]);
 
         $this->assertCount(1, $assetTypes);
         $this->assertArrayHasKey('test-asset', $assetTypes);
@@ -36,7 +36,7 @@ 
         $io = new BufferIO();
 
         $assetsBuilder = new AssetsBuilder($installationManager, $io, '.');
-        $assetTypes = $assetsBuilder->buildAssetTypes([ $packageA ]);
+        $assetTypes = $assetsBuilder->buildAssetTypes([$packageA]);
 
         $this->assertCount(0, $assetTypes);
         $this->assertNotEmpty($io->getOutput());