Merge branch '1.x' into 2.x

* 1.x:
  Flex recipe should always be installed first

Author: fabpot

src/Flex.php

@@ -484,9 +484,10 @@ public function fetchRecipes(array $operations, bool $reset): array
         $data = $this->downloader->getRecipes($operations);
         $manifests = $data['manifests'] ?? [];
         $locks = $data['locks'] ?? [];
-        // symfony/flex and symfony/framework-bundle recipes should always be applied first
+        // symfony/flex recipes should always be applied first
+        $flexRecipe = [];
+        // symfony/framework-bundle recipe should always be applied first after the metapackages
         $recipes = [
-            'symfony/flex' => null,
             'symfony/framework-bundle' => null,
         ];
         $metaRecipes = [];
@@ -530,6 +531,8 @@ public function fetchRecipes(array $operations, bool $reset): array
             if (isset($manifests[$name])) {
                 if ('metapackage' === $package->getType()) {
                     $metaRecipes[$name] = new Recipe($package, $name, $job, $manifests[$name], $locks[$name] ?? []);
+                } elseif ('symfony/flex' === $name) {
+                    $flexRecipe = [$name => new Recipe($package, $name, $job, $manifests[$name], $locks[$name] ?? [])];
                 } else {
                     $recipes[$name] = new Recipe($package, $name, $job, $manifests[$name], $locks[$name] ?? []);
                 }
@@ -557,7 +560,7 @@ public function fetchRecipes(array $operations, bool $reset): array
             }
         }
 
-        return array_merge($metaRecipes, array_filter($recipes));
+        return array_merge($flexRecipe, $metaRecipes, array_filter($recipes));
     }
 
     public function truncatePackages(PrePoolCreateEvent $event)

tests/FlexTest.php

@@ -178,6 +178,41 @@ public function getPackagesForAutoDiscovery(): array
         return $return;
     }
 
+    public function testFetchRecipesOrder()
+    {
+        $packages = [
+            ['name' => 'symfony/console', 'type' => 'library'],
+            ['name' => 'symfony/flex', 'type' => 'composer-plugin'],
+            ['name' => 'symfony/framework-bundle', 'type' => 'library'],
+            ['name' => 'symfony/webapp-meta', 'type' => 'metapackage'],
+        ];
+
+        $io = new BufferIO('', OutputInterface::VERBOSITY_VERBOSE);
+        $rootPackage = $this->mockRootPackage(['symfony' => ['allow-contrib' => true]]);
+
+        $flex = $this->mockFlex($io, $rootPackage, null, [
+            'manifests' => array_reduce($packages, static function (array $manifests, array $packageInfo) {
+                $manifests[$packageInfo['name']] = ['manifest' => []];
+
+                return $manifests;
+            }, []),
+        ]);
+
+        $recipes = $flex->fetchRecipes(array_map(static function (array $packageInfo) {
+            $package = new Package($packageInfo['name'], '1.0.0', '1.0.0');
+            $package->setType($packageInfo['type']);
+
+            return new InstallOperation($package);
+        }, $packages), true);
+
+        $this->assertSame([
+            'symfony/flex',
+            'symfony/webapp-meta',
+            'symfony/framework-bundle',
+            'symfony/console',
+        ], array_keys($recipes));
+    }
+
     public static function getTestPackages(): array
     {
         return [