Merge pull request #51 from moufmouf/autowire_alias

Fixing autowiring on service aliases

Author: moufmouf

DependencyInjection/GraphqliteCompilerPass.php

@@ -304,7 +304,11 @@ private function makePublicInjectedServices(ReflectionClass $refClass, Annotatio
         });
 
         foreach ($services as $service) {
-            $container->getDefinition($service)->setPublic(true);
+            if ($container->hasAlias($service)) {
+                $container->getAlias($service)->setPublic(true);
+            } else {
+                $container->getDefinition($service)->setPublic(true);
+            }
         }
 
     }

Tests/Fixtures/Entities/Contact.php

@@ -37,11 +37,12 @@ public function getName(): string
      * @Field()
      * @Autowire(for="$testService")
      * @Autowire(for="$someService", identifier="someService")
+     * @Autowire(for="$someAlias", identifier="someAlias")
      * @return string
      */
-    public function injectService(TestGraphqlController $testService = null, stdClass $someService = null): string
+    public function injectService(TestGraphqlController $testService = null, stdClass $someService = null, stdClass $someAlias = null): string
     {
-        if (!$testService instanceof TestGraphqlController || $someService === null) {
+        if (!$testService instanceof TestGraphqlController || $someService === null || $someAlias === null) {
             return 'KO';
         }
         return 'OK';

Tests/Fixtures/config/services.yaml

@@ -20,6 +20,9 @@ services:
     someService:
         class: stdClass
 
+    someAlias:
+        alias: someService
+
     Symfony\Component\HttpFoundation\Session\Storage\Handler\NullSessionHandler:
         class: Symfony\Component\HttpFoundation\Session\Storage\Handler\NullSessionHandler
     # controllers are imported separately to make sure services can be injected