Merge branch '0.14'

Author: Jeremiah VALERIE

.php_cs.dist

@@ -10,6 +10,8 @@ return PhpCsFixer\Config::create()
     ->setRules(
         [
             '@Symfony' => true,
+            '@PHP71Migration' => true,
+            '@PHP74Migration:risky' => true,
             'ordered_imports' => ['imports_order' => ['class', 'function', 'const']],
             'general_phpdoc_annotation_remove' => ['author', 'category', 'copyright', 'created', 'license', 'package', 'since', 'subpackage', 'version'],
             'fully_qualified_strict_types' => true,
@@ -19,7 +21,9 @@ return PhpCsFixer\Config::create()
             'global_namespace_import' => ['import_functions' => true, 'import_classes' => true, 'import_constants' => true],
             'phpdoc_summary' => false,
             'hash_to_slash_comment' => false,
-            'single_line_comment_style' => false
+            'single_line_comment_style' => false,
+            'phpdoc_no_alias_tag' => ['type' => 'var'],
+            'no_mixed_echo_print' => ['use' => 'echo'],
         ]
     )
     ->setFinder($finder)

UPGRADE.md

@@ -12,6 +12,8 @@ UPGRADE FROM 0.13 to 0.14
 - [Replace `overblog_graphql.global_variable` tag](#replace-overblog_graphqlglobal_variable-tag)
 - [Replace `resolver` expression function](#replace-resolver-expression-function)
 - [Rename `ResolverInterface` to `QueryInterface`](#rename-resolverinterface-to-queryinterface)
+- [Remove Argument deprecated method](#remove-argument-deprecated-method)
+- [Remove ConnectionBuilder deprecated class](#remove-connectionbuilder-deprecated-class)
 
 ### Customize the cursor encoder of the edges of a connection
 
@@ -199,6 +201,15 @@ Example:
 }
 ```
 
+### Remove Argument deprecated method
+
+Method `Overblog\GraphQLBundle\Definition\Argument::getRawArguments` is replaced by
+`Overblog\GraphQLBundle\Definition\Argument::getArrayCopy`.
+
+### Remove ConnectionBuilder deprecated class
+
+Class `Overblog\GraphQLBundle\Relay\Connection\Output\ConnectionBuilder` is replaced by
+`Overblog\GraphQLBundle\Relay\Connection\ConnectionBuilder`.
 
 UPGRADE FROM 0.12 to 0.13
 =======================

composer.json

@@ -80,7 +80,7 @@
             "test -f phpbench.phar || wget https://github.com/phpbench/phpbench/releases/download/1.0.0-alpha7/phpbench.phar -O phpbench.phar",
             "@php phpbench.phar run -l dots --ansi -vvv --report='generator: \"table\", cols: [\"benchmark\", \"subject\", \"params\", \"best\", \"mean\", \"mode\", \"worst\", \"diff\"], break: [\"benchmark\"], sort: {mean: \"asc\"}'"
         ],
-        "install-cs": "test -f php-cs-fixer.phar || wget https://github.com/FriendsOfPHP/PHP-CS-Fixer/releases/download/v2.16.4/php-cs-fixer.phar -O php-cs-fixer.phar",
+        "install-cs": "test -f php-cs-fixer.phar || wget https://github.com/FriendsOfPHP/PHP-CS-Fixer/releases/download/v2.18.7/php-cs-fixer.phar -O php-cs-fixer.phar",
         "fix-cs": [
             "@install-cs",
             "@php php-cs-fixer.phar fix --diff -v --allow-risky=yes --ansi"

src/Config/ObjectTypeDefinition.php

@@ -50,9 +50,7 @@ public function getDefinition(): ArrayNodeDefinition
     private function treatFieldsDefaultAccess(ArrayNodeDefinition $node): void
     {
         $node->validate()
-            ->ifTrue(function ($v) {
-                return array_key_exists('fieldsDefaultAccess', $v) && null !== $v['fieldsDefaultAccess'];
-            })
+            ->ifTrue(fn ($v) => array_key_exists('fieldsDefaultAccess', $v) && null !== $v['fieldsDefaultAccess'])
             ->then(function ($v) {
                 foreach ($v['fields'] as &$field) {
                     if (array_key_exists('access', $field) && null !== $field['access']) {

src/Config/Parser/AnnotationParser.php

@@ -15,7 +15,7 @@
 
 class AnnotationParser extends MetadataParser
 {
-    const METADATA_FORMAT = '@%s';
+    public const METADATA_FORMAT = '@%s';
 
     protected static ?AnnotationReader $annotationReader = null;
 

src/Config/Parser/AttributeParser.php

@@ -14,7 +14,7 @@
 
 class AttributeParser extends MetadataParser
 {
-    const METADATA_FORMAT = '#[%s]';
+    public const METADATA_FORMAT = '#[%s]';
 
     public static function getMetadatas(Reflector $reflector): array
     {

src/Config/Parser/MetadataParser/MetadataParser.php

@@ -44,8 +44,8 @@
 
 abstract class MetadataParser implements PreParserInterface
 {
-    const ANNOTATION_NAMESPACE = 'Overblog\GraphQLBundle\Annotation\\';
-    const METADATA_FORMAT = '%s';
+    public const ANNOTATION_NAMESPACE = 'Overblog\GraphQLBundle\Annotation\\';
+    public const METADATA_FORMAT = '%s';
 
     private static ClassesTypesMap $map;
     private static array $typeGuessers = [];
@@ -233,7 +233,7 @@ private static function classMetadatasToGQLConfiguration(
 
         if (null !== $gqlType) {
             if (!$gqlName) {
-                $gqlName = isset($classMetadata->name) ? $classMetadata->name : $reflectionClass->getShortName();
+                $gqlName = !empty($classMetadata->name) ? $classMetadata->name : $reflectionClass->getShortName();
             }
 
             if ($preProcess) {
@@ -346,9 +346,7 @@ private static function graphQLTypeConfigFromAnnotation(ReflectionClass $reflect
 
         $buildersAnnotations = array_merge(self::getMetadataMatching($metadatas, Metadata\FieldsBuilder::class), $typeAnnotation->builders);
         if (!empty($buildersAnnotations)) {
-            $typeConfiguration['builders'] = array_map(function ($fieldsBuilderAnnotation) {
-                return ['builder' => $fieldsBuilderAnnotation->name, 'builderConfig' => $fieldsBuilderAnnotation->config];
-            }, $buildersAnnotations);
+            $typeConfiguration['builders'] = array_map(fn ($fieldsBuilderAnnotation) => ['builder' => $fieldsBuilderAnnotation->name, 'builderConfig' => $fieldsBuilderAnnotation->config], $buildersAnnotations);
         }
 
         if (isset($typeAnnotation->isTypeOf)) {
@@ -593,7 +591,7 @@ private static function getTypeFieldConfigurationFromReflector(ReflectionClass $
             if (is_string($fieldMetadata->argsBuilder)) {
                 $fieldConfiguration['argsBuilder'] = ['builder' => $fieldMetadata->argsBuilder, 'config' => []];
             } elseif (is_array($fieldMetadata->argsBuilder)) {
-                list($builder, $builderConfig) = $fieldMetadata->argsBuilder;
+                [$builder, $builderConfig] = $fieldMetadata->argsBuilder;
                 $fieldConfiguration['argsBuilder'] = ['builder' => $builder, 'config' => $builderConfig];
             } else {
                 throw new InvalidArgumentException(sprintf('The attribute "argsBuilder" on metadata %s defined on "%s" must be a string or an array where first index is the builder name and the second is the config.', static::formatMetadata($fieldMetadataName), $reflector->getName()));
@@ -608,7 +606,7 @@ private static function getTypeFieldConfigurationFromReflector(ReflectionClass $
                 $fieldConfiguration['builder'] = $fieldMetadata->fieldBuilder;
                 $fieldConfiguration['builderConfig'] = [];
             } elseif (is_array($fieldMetadata->fieldBuilder)) {
-                list($builder, $builderConfig) = $fieldMetadata->fieldBuilder;
+                [$builder, $builderConfig] = $fieldMetadata->fieldBuilder;
                 $fieldConfiguration['builder'] = $builder;
                 $fieldConfiguration['builderConfig'] = $builderConfig ?: [];
             } else {
@@ -929,7 +927,7 @@ private static function guessType(ReflectionClass $reflectionClass, Reflector $r
             }
         }
 
-        throw new TypeGuessingException(join("\n", $errors));
+        throw new TypeGuessingException(implode("\n", $errors));
     }
 
     /**

src/Config/Processor/InheritanceProcessor.php

@@ -39,9 +39,7 @@ public static function process(array $configs): array
 
     private static function removedDecorators(array $configs): array
     {
-        return array_filter($configs, function ($config) {
-            return !isset($config['decorator']) || true !== $config['decorator'];
-        });
+        return array_filter($configs, fn ($config) => !isset($config['decorator']) || true !== $config['decorator']);
     }
 
     private static function processConfigsHeirs(array $configs): array

src/Definition/Argument.php

@@ -6,9 +6,6 @@
 
 use function array_key_exists;
 use function count;
-use function sprintf;
-use function trigger_error;
-use const E_USER_DEPRECATED;
 
 class Argument implements ArgumentInterface
 {
@@ -32,23 +29,6 @@ public function getArrayCopy(): array
         return $this->rawArguments;
     }
 
-    /**
-     * @deprecated This method is deprecated since 0.12 and will be removed in 0.13. You should use getArrayCopy method instead.
-     */
-    public function getRawArguments(): array
-    {
-        @trigger_error(
-            sprintf(
-                'This "%s" method is deprecated since 0.12 and will be removed in 0.13. You should use "%s::getArrayCopy" instead.',
-                __METHOD__,
-                __CLASS__
-            ),
-            E_USER_DEPRECATED
-        );
-
-        return $this->getArrayCopy();
-    }
-
     /**
      * @param int|string $offset
      */

src/Definition/Type/CustomScalarType.php

@@ -19,7 +19,7 @@ public function __construct(array $config = [])
         $config['name'] = $config['name'] ?? uniqid('CustomScalar');
         parent::__construct($config);
 
-        $this->config['scalarType'] = isset($this->config['scalarType']) ? $this->config['scalarType'] : null;
+        $this->config['scalarType'] = $this->config['scalarType'] ?? null;
     }
 
     /**