Allow service locators to be ordered by priority.

longwave · longwave · commit c9c25c27ab37 · 2024-06-07T10:10:33.000+01:00
diff --git a/src/Symfony/Component/DependencyInjection/Compiler/ServiceLocatorTagPass.php b/src/Symfony/Component/DependencyInjection/Compiler/ServiceLocatorTagPass.php
@@ -131,7 +131,6 @@ public static function map(array $services): array
 
             $services[$k] = new ServiceClosureArgument($v);
         }
-        ksort($services);
 
         return $services;
     }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/ServiceLocatorTagPassTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/ServiceLocatorTagPassTest.php
@@ -214,7 +214,7 @@ public function testDefinitionOrderIsTheSame()
         $locator = $container->getDefinition($locator);
         $factories = $locator->getArguments()[0];
 
-        static::assertSame(['service-1', 'service-2'], array_keys($factories));
+        static::assertSame(['service-2', 'service-1'], array_keys($factories));
     }
 
     public function testBindingsAreProcessed()

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,6 @@ public static function map(array $services): array`
`131`	`131`
`132`	`132`	`$services[$k] = new ServiceClosureArgument($v);`
`133`	`133`	`}`
`134`		`- ksort($services);`
`135`	`134`
`136`	`135`	`return $services;`
`137`	`136`	`}`
Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ public function testDefinitionOrderIsTheSame()`
`214`	`214`	`$locator = $container->getDefinition($locator);`
`215`	`215`	`$factories = $locator->getArguments()[0];`
`216`	`216`
`217`		`- static::assertSame(['service-1', 'service-2'], array_keys($factories));`
	`217`	`+ static::assertSame(['service-2', 'service-1'], array_keys($factories));`
`218`	`218`	`}`
`219`	`219`
`220`	`220`	`public function testBindingsAreProcessed()`