minor #40848 [DependencyInjection] Improve autowiring errors when named autowiring aliases exist (nicolas-grekas)

This PR was merged into the 5.3-dev branch.

Discussion
----------

[DependencyInjection] Improve autowiring errors when named autowiring aliases exist

| Q             | A
| ------------- | ---
| Branch?       | 5.x
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Before:
`Cannot autowire service "App\Controller\HelloController": argument "$workflow" of method "__construct()" references interface "Symfony\Component\Workflow\WorkflowInterface" but no such service exists. You should maybe alias this interface to one of these existing services: "state_machine.pull_request", "state_machine.comment".`

After:
`Cannot autowire service "App\Controller\HelloController": argument "$workflow" of method "__construct()" references interface "Symfony\Component\Workflow\WorkflowInterface" but no such service exists. Available autowiring aliases for this interface are: "$pullRequestStateMachine", "$commentStateMachine".`

/cc `@lyrixx` `@weaverryan`

Commits
-------

88eb0b57c1 [DependencyInjection] Improve autowiring errors when named autowiring aliases exist

Author: fabpot

Compiler/AutowirePass.php

@@ -34,6 +34,7 @@ class AutowirePass extends AbstractRecursivePass
 {
     private $types;
     private $ambiguousServiceTypes;
+    private $autowiringAliases;
     private $lastFailure;
     private $throwOnAutowiringException;
     private $decoratedClass;
@@ -344,10 +345,15 @@ private function populateAvailableTypes(ContainerBuilder $container)
     {
         $this->types = [];
         $this->ambiguousServiceTypes = [];
+        $this->autowiringAliases = [];
 
         foreach ($container->getDefinitions() as $id => $definition) {
             $this->populateAvailableType($container, $id, $definition);
         }
+
+        foreach ($container->getAliases() as $id => $alias) {
+            $this->populateAutowiringAlias($id);
+        }
     }
 
     /**
@@ -371,6 +377,8 @@ private function populateAvailableType(ContainerBuilder $container, string $id,
         do {
             $this->set($reflectionClass->name, $id);
         } while ($reflectionClass = $reflectionClass->getParentClass());
+
+        $this->populateAutowiringAlias($id);
     }
 
     /**
@@ -460,6 +468,10 @@ private function createTypeAlternatives(ContainerBuilder $container, TypedRefere
         }
 
         $servicesAndAliases = $container->getServiceIds();
+        if (null !== ($autowiringAliases = $this->autowiringAliases[$type] ?? null) && !isset($autowiringAliases[''])) {
+            return sprintf(' Available autowiring aliases for this %s are: "$%s".', class_exists($type, false) ? 'class' : 'interface', implode('", "$', $autowiringAliases));
+        }
+
         if (!$container->has($type) && false !== $key = array_search(strtolower($type), array_map('strtolower', $servicesAndAliases))) {
             return sprintf(' Did you mean "%s"?', $servicesAndAliases[$key]);
         } elseif (isset($this->ambiguousServiceTypes[$type])) {
@@ -498,4 +510,18 @@ private function getAliasesSuggestionForType(ContainerBuilder $container, string
 
         return null;
     }
+
+    private function populateAutowiringAlias(string $id): void
+    {
+        if (!preg_match('/(?(DEFINE)(?<V>[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*+))^((?&V)(?:\\\\(?&V))*+)(?: \$((?&V)))?$/', $id, $m)) {
+            return;
+        }
+
+        $type = $m[2];
+        $name = $m[3] ?? '';
+
+        if (class_exists($type, false) || interface_exists($type, false)) {
+            $this->autowiringAliases[$type][$name] = $name;
+        }
+    }
 }