thecodingmachine / discovery

src/DiscoveryPlugin.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 use Composer\IO\IOInterface;
 use Composer\Package\PackageInterface;
 use Composer\Plugin\PluginInterface;
-use Composer\Repository\InstalledFilesystemRepository;
 use Composer\Script\Event;
 use Composer\Script\ScriptEvents;
 use Seld\JsonLint\JsonParser;

Spacing +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
             foreach ($json as $key => $values) {
                 $existingValues = $array[$key] ?? [];
                 if (!is_array($values)) {
-                    $values = [ $values ];
+                    $values = [$values];
                 }
                 $existingValues = array_merge($existingValues, $values);
                 $array[$key] = $existingValues;

src/PackagesOrderer.php

Indentation +48 added lines, -48 removed lines

@@ -17,61 +17,61 @@
      * @param PackageInterface[] $unorderedPackagesList
      * @return array|PackageInterface[]
      */
-	public static function reorderPackages(array $unorderedPackagesList) {
-		// 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) {
-			return strcmp($packageA->getName(), $packageB->getName());
-		});
+    public static function reorderPackages(array $unorderedPackagesList) {
+        // 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) {
+            return strcmp($packageA->getName(), $packageB->getName());
+        });
 		
-		$orderedPackagesList = array();
-		foreach ($unorderedPackagesList as $package) {
-			$orderedPackagesList = self::walkPackagesList($package, $orderedPackagesList, $unorderedPackagesList);
-		}
+        $orderedPackagesList = array();
+        foreach ($unorderedPackagesList as $package) {
+            $orderedPackagesList = self::walkPackagesList($package, $orderedPackagesList, $unorderedPackagesList);
+        }
 	
-		return $orderedPackagesList;
-	}
+        return $orderedPackagesList;
+    }
 	
-	/**
-	 * Function used to sort packages by dependencies (packages depending from no other package in front of others)
-	 * Invariant hypothesis for this function: $orderedPackagesList is already ordered and the package we add
-	 * has all its dependencies already accounted for. If not, we add the dependencies first.
-	 *
-	 * @param PackageInterface $package
-	 * @param PackageInterface[] $orderedPackagesList The list of sorted packages
-	 * @param PackageInterface[] $availablePackages The list of all packages not yet sorted
-	 * @return PackageInterface[]
-	 */
-	private static function walkPackagesList(PackageInterface $package, array $orderedPackagesList, array &$availablePackages) {
-		// First, let's check that the package we want to add is not already in our list.
-		foreach ($orderedPackagesList as $includedPackage) {
-			if ($includedPackage->equals($package)) {
-				return $orderedPackagesList;
-			}
-		}
+    /**
+     * Function used to sort packages by dependencies (packages depending from no other package in front of others)
+     * Invariant hypothesis for this function: $orderedPackagesList is already ordered and the package we add
+     * has all its dependencies already accounted for. If not, we add the dependencies first.
+     *
+     * @param PackageInterface $package
+     * @param PackageInterface[] $orderedPackagesList The list of sorted packages
+     * @param PackageInterface[] $availablePackages The list of all packages not yet sorted
+     * @return PackageInterface[]
+     */
+    private static function walkPackagesList(PackageInterface $package, array $orderedPackagesList, array &$availablePackages) {
+        // First, let's check that the package we want to add is not already in our list.
+        foreach ($orderedPackagesList as $includedPackage) {
+            if ($includedPackage->equals($package)) {
+                return $orderedPackagesList;
+            }
+        }
 	
-		// We need to make sure there is no loop (if a package A requires a package B that requires the package A)...
-		// We do that by removing the package from the list of all available packages.
-		$key = array_search($package, $availablePackages);
-		unset($availablePackages[$key]);
+        // We need to make sure there is no loop (if a package A requires a package B that requires the package A)...
+        // We do that by removing the package from the list of all available packages.
+        $key = array_search($package, $availablePackages);
+        unset($availablePackages[$key]);
 	
-		// Now, let's see if there are dependencies.
-		foreach ($package->getRequires() as $require) {
-			/* @var $require Link */
-			foreach ($availablePackages as $iterPackage) {
-				if ($iterPackage->getName() == $require->getTarget()) {
-					$orderedPackagesList = self::walkPackagesList($iterPackage, $orderedPackagesList, $availablePackages);
-					break;
-				}
-			}
-		}
+        // Now, let's see if there are dependencies.
+        foreach ($package->getRequires() as $require) {
+            /* @var $require Link */
+            foreach ($availablePackages as $iterPackage) {
+                if ($iterPackage->getName() == $require->getTarget()) {
+                    $orderedPackagesList = self::walkPackagesList($iterPackage, $orderedPackagesList, $availablePackages);
+                    break;
+                }
+            }
+        }
 	
-		// FIXME: manage dev-requires and "provides"
+        // FIXME: manage dev-requires and "provides"
 	
-		// Finally, let's add the package once all dependencies have been added.
-		$orderedPackagesList[] = $package;
+        // Finally, let's add the package once all dependencies have been added.
+        $orderedPackagesList[] = $package;
 	
-		return $orderedPackagesList;
-	}
+        return $orderedPackagesList;
+    }
 
 }
\ No newline at end of file