Issue #3186009 by longwave, ankithashetty, klausi: MimeTypePass has undefined variables

core/lib/Drupal/Core/DependencyInjection/Compiler/MimeTypePass.php

@@ -55,16 +55,7 @@ public function process(ContainerBuilder $container) {
     // Add a method call for each handler to the consumer service
     // definition.
     foreach ($handlers as $id => $priority) {
-      $arguments = [];
-      $arguments[0] = new Reference($id);
-      if (isset($priority_pos)) {
-        $arguments[$priority_pos] = $priority;
-      }
-      if (isset($id_pos)) {
-        $arguments[$id_pos] = $id;
-      }
-      // Sort the arguments by position.
-      ksort($arguments);
+      $arguments = [new Reference($id), $priority];
       if (is_subclass_of($interfaces[$id], $interface)) {
         $consumer->addMethodCall('addMimeTypeGuesser', $arguments);
       }

core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/MimeTypePassTest.php

@@ -34,15 +34,17 @@ public function testProcessLegacy() {
 
     $container
       ->register('handler1', __NAMESPACE__ . '\NewMimeTypeGuesser')
-      ->addTag('mime_type_guesser');
+      ->addTag('mime_type_guesser', ['priority' => 200]);
     $container
       ->register('handler2', __NAMESPACE__ . '\LegacyMimeTypeGuesser')
-      ->addTag('mime_type_guesser');
+      ->addTag('mime_type_guesser', ['priority' => 100]);
 
     $handler_pass = new MimeTypePass();
     $handler_pass->process($container);
     $method_calls = $container->getDefinition('file.mime_type.guesser')->getMethodCalls();
     $this->assertCount(2, $method_calls);
+    $this->assertSame(200, $method_calls[0][1][1]);
+    $this->assertSame(100, $method_calls[1][1][1]);
   }
 
 }