Stan fixes

Author: TamasSzigeti

Controller/GraphQL/LoginController.php

@@ -82,6 +82,7 @@ public function login(string $userName, string $password, Request $request): Use
 
         // Fire the login event manually
         $event = new InteractiveLoginEvent($request, $token);
+        // @phpstan-ignore-next-line BC for Symfony4
         $this->eventDispatcher->dispatch($event, 'security.interactive_login');
 
         return $user;

Controller/GraphqliteController.php

@@ -106,18 +106,16 @@ private function handlePsr7Request(ServerRequestInterface $request, Request $sym
             return new JsonResponse($result->toArray($this->debug), $httpCodeDecider->decideHttpStatusCode($result));
         }
         if (is_array($result)) {
-            $finalResult = array_map(function (ExecutionResult $executionResult) {
+            $finalResult = array_map(function (ExecutionResult $executionResult): array {
                 return $executionResult->toArray($this->debug);
             }, $result);
             // Let's return the highest result.
             $statuses = array_map([$httpCodeDecider, 'decideHttpStatusCode'], $result);
             $status = empty($statuses) ? 500 : max($statuses);
+
             return new JsonResponse($finalResult, $status);
         }
-        if ($result instanceof Promise) {
-            throw new RuntimeException('Only SyncPromiseAdapter is supported');
-        }
-        /* @phpstan-ignore-next-line */
-        throw new RuntimeException('Unexpected response from StandardServer::executePsrRequest'); // @codeCoverageIgnore
+
+        throw new RuntimeException('Only SyncPromiseAdapter is supported');
     }
 }

DependencyInjection/GraphqliteCompilerPass.php

@@ -333,7 +333,7 @@ private function mapAdderToTag(string $tag, string $methodName, ContainerBuilder
      */
     private function makePublicInjectedServices(ReflectionClass $refClass, AnnotationReader $reader, ContainerBuilder $container, bool $isController): void
     {
-        $services = $this->getCodeCache()->get($refClass, function() use ($refClass, $reader, $container, $isController) {
+        $services = $this->getCodeCache()->get($refClass, function() use ($refClass, $reader, $container, $isController): array {
             $services = [];
             foreach ($refClass->getMethods(ReflectionMethod::IS_PUBLIC) as $method) {
                 $field = $reader->getRequestAnnotation($method, Field::class) ?? $reader->getRequestAnnotation($method, Query::class) ?? $reader->getRequestAnnotation($method, Mutation::class);
@@ -347,6 +347,7 @@ private function makePublicInjectedServices(ReflectionClass $refClass, Annotatio
                     }
                 }
             }
+
             return $services;
         });
 
@@ -485,7 +486,7 @@ private function getClassList(string $namespace, int $globTtl = 2, bool $recursi
                 // The autoloader might trigger errors if the file does not respect PSR-4 or if the
                 // Symfony DebugAutoLoader is installed. (see https://github.com/thecodingmachine/graphqlite/issues/216)
                 require_once $phpFile;
-                // Does it exists now?
+                // @phpstan-ignore-next-line Does it exists now?
                 if (! class_exists($className, false)) {
                     continue;
                 }

DependencyInjection/GraphqliteExtension.php

@@ -42,7 +42,12 @@ public function load(array $configs, ContainerBuilder $container): void
             if (!is_array($controllers)) {
                 $controllers = [ $controllers ];
             }
-            $namespaceController = array_map(function($namespace) { return rtrim($namespace, '\\') . '\\'; }, $controllers);
+            $namespaceController = array_map(
+                function($namespace): string {
+                    return rtrim($namespace, '\\') . '\\';
+                },
+                $controllers
+            );
         } else {
             $namespaceController = [];
         }
@@ -51,7 +56,12 @@ public function load(array $configs, ContainerBuilder $container): void
             if (!is_array($types)) {
                 $types = [ $types ];
             }
-            $namespaceType = array_map(function($namespace) { return rtrim($namespace, '\\') . '\\'; }, $types);
+            $namespaceType = array_map(
+                function($namespace): string {
+                    return rtrim($namespace, '\\') . '\\';
+                },
+                $types
+            );
         } else {
             $namespaceType = [];
         }
@@ -84,20 +94,6 @@ public function load(array $configs, ContainerBuilder $container): void
             ->addTag('graphql.root_type_mapper_factory');
     }
 
-    private function getNamespaceDir(string $namespace): string
-    {
-        $classNameMapper = ClassNameMapper::createFromComposerFile(null, null, true);
-
-        $possibleFileNames = $classNameMapper->getPossibleFileNames($namespace.'Xxx');
-        if (count($possibleFileNames) > 1) {
-            throw new \RuntimeException(sprintf('According to your composer.json, classes belonging to the "%s" namespace can be located in several directories: %s. This is an issue for the GraphQLite lib. Please make sure that a namespace can only be resolved to one PHP file.', $namespace, implode(", ", $possibleFileNames)));
-        } elseif (empty($possibleFileNames)) {
-            throw new \RuntimeException(sprintf('Files in namespace "%s" cannot be autoloaded by Composer. Please set up a PSR-4 autoloader in Composer or change the namespace configured in "graphqlite.namespace.controllers" and "graphqlite.namespace.types"', $namespace));
-        }
-
-        return substr($possibleFileNames[0], 0, -8);
-    }
-
     /**
      * @param array<string, int> $debug
      * @return int

DependencyInjection/OverblogGraphiQLEndpointWiringPass.php

@@ -2,8 +2,6 @@
 
 namespace TheCodingMachine\Graphqlite\Bundle\DependencyInjection;
 
-use Overblog\GraphiQLBundle\Config\GraphiQLControllerEndpoint;
-use Overblog\GraphiQLBundle\Config\GraphqlEndpoint\RootResolver;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;

GraphiQL/EndpointResolver.php

@@ -3,10 +3,15 @@
 namespace TheCodingMachine\Graphqlite\Bundle\GraphiQL;
 
 use Overblog\GraphiQLBundle\Config\GraphiQLControllerEndpoint;
+use Overblog\GraphiQLBundle\Config\GraphQLEndpoint\GraphQLEndpointInvalidSchemaException;
 use Symfony\Component\HttpFoundation\RequestStack;
+use Webmozart\Assert\Assert;
 
 final class EndpointResolver implements GraphiQLControllerEndpoint
 {
+    /**
+     * @var RequestStack
+     */
     protected $requestStack;
 
     public function __construct(RequestStack $requestStack)
@@ -18,6 +23,7 @@ public function getBySchema($name)
     {
         if ('default' === $name) {
             $request = $this->requestStack->getCurrentRequest();
+            Assert::notNull($request);
 
             return $request->getBaseUrl().'/graphql';
         }

Mappers/RequestParameterMiddleware.php

@@ -6,6 +6,7 @@
 
 use phpDocumentor\Reflection\DocBlock;
 use phpDocumentor\Reflection\Type;
+use ReflectionNamedType;
 use ReflectionParameter;
 use Symfony\Component\HttpFoundation\Request;
 use TheCodingMachine\GraphQLite\Annotations\ParameterAnnotations;
@@ -15,13 +16,13 @@
 
 class RequestParameterMiddleware implements ParameterMiddlewareInterface
 {
-
     public function mapParameter(ReflectionParameter $parameter, DocBlock $docBlock, ?Type $paramTagType, ParameterAnnotations $parameterAnnotations, ParameterHandlerInterface $next): ParameterInterface
     {
         $parameterType = $parameter->getType();
-        if ($parameterType && $parameterType->getName() === Request::class) {
+        if ($parameterType instanceof ReflectionNamedType && $parameterType->getName() === Request::class) {
             return new RequestParameter();
         }
+
         return $next->mapParameter($parameter, $docBlock, $paramTagType, $parameterAnnotations);
     }
 }

Server/ServerConfig.php

@@ -4,6 +4,9 @@
 namespace TheCodingMachine\Graphqlite\Bundle\Server;
 
 use GraphQL\Error\InvariantViolation;
+use GraphQL\GraphQL;
+use GraphQL\Language\AST\DocumentNode;
+use GraphQL\Server\OperationParams;
 use GraphQL\Utils\Utils;
 use GraphQL\Validator\DocumentValidator;
 use GraphQL\Validator\Rules\ValidationRule;
@@ -27,7 +30,15 @@ class ServerConfig extends \GraphQL\Server\ServerConfig
      */
     public function setValidationRules($validationRules)
     {
-        parent::setValidationRules(array_merge(DocumentValidator::defaultRules(), $validationRules));
+        parent::setValidationRules(
+            function (OperationParams $params, DocumentNode $doc, string $operationType) use ($validationRules): array {
+                $validationRules = is_callable($validationRules)
+                    ? $validationRules($params, $doc, $operationType)
+                    : $validationRules;
+
+                return array_merge(DocumentValidator::defaultRules(), $validationRules);
+            }
+        );
 
         return $this;
     }

composer.json

@@ -38,7 +38,8 @@
     "php-coveralls/php-coveralls": "^2.1.0",
     "symfony/phpunit-bridge": "^5.3",
     "thecodingmachine/phpstan-strict-rules": "^v0.12.1",
-    "composer/package-versions-deprecated": "^1.8"
+    "composer/package-versions-deprecated": "^1.8",
+    "phpstan/phpstan-webmozart-assert": "^0.12.12"
   },
   "conflict": {
     "mouf/classname-mapper": "<1.0.2",