Merge pull request #135 from mcg-web/optional-proxy-resolver-method-param

Adjusted tagged proxy resolver method param to be optional

Author: mcg-web

DependencyInjection/Compiler/ResolverTaggedServiceMappingPass.php

@@ -27,7 +27,7 @@ protected function checkRequirements($id, array $tag)
     {
         parent::checkRequirements($id, $tag);
 
-        if (!isset($tag['method']) || !is_string($tag['method'])) {
+        if (isset($tag['method']) && !is_string($tag['method'])) {
             throw new \InvalidArgumentException(
                 sprintf('Service tagged "%s" must have valid "method" argument.', $id)
             );

Resolver/AbstractProxyResolver.php

@@ -42,7 +42,7 @@ public function resolve($input)
         }
 
         $options = $this->getSolutionOptions($alias);
-        $func = [$solution, $options['method']];
+        $func = isset($options['method']) ? [$solution, $options['method']] : $solution;
 
         return call_user_func_array($func, $funcArgs);
     }

Tests/Functional/app/Exception/ExampleException.php

@@ -13,7 +13,7 @@
 
 class ExampleException
 {
-    public function throwException()
+    public function __invoke()
     {
         throw new \InvalidArgumentException('Invalid argument exception');
     }

Tests/Functional/app/config/exception/services.yml

@@ -2,4 +2,4 @@ services:
     overblog_graphql.test.exception:
         class: Overblog\GraphQLBundle\Tests\Functional\app\Exception\ExampleException
         tags:
-            - { name: "overblog_graphql.resolver", alias: "example_exception", method: "throwException" }
+            - { name: "overblog_graphql.resolver", alias: "example_exception" }