shmax / graphql-php-validation-toolkit

src/Type/Definition/UserErrorsType.php

Spacing +16 added lines, -16 removed lines

@@ -20,18 +20,18 @@ 
      * @param mixed[]  $config
      * @param string[] $path
      */
-    public function __construct(array $config, array $path, $isParentList = false)
+    public function __construct (array $config, array $path, $isParentList = false)
     {
         $finalFields = $config['fields'] ?? [];
 
-        if (! isset($config['type'])) {
+        if (!isset($config['type'])) {
             throw new Exception('You must specify a type for your field');
         }
 
         GraphQL\Utils\Utils::invariant($config['type'] instanceof Type, 'Must provide type.');
 
         if (isset($config['errorCodes'])) {
-            if (! isset($config['validate'])) {
+            if (!isset($config['validate'])) {
                 throw new Exception('If you specify errorCodes, you must also provide a validate callback');
             }
 
@@ -43,7 +43,7 @@ 
                     'values' => $config['errorCodes'],
                 ]), $config),
                 'description' => 'An error code',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     return $value['error'][0] ?? null;
                 },
             ];
@@ -54,7 +54,7 @@ 
             $finalFields['msg'] = [
                 'type' => Type::string(),
                 'description' => 'A natural language description of the issue',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     return $value['error'][1] ?? null;
                 },
             ];
@@ -65,18 +65,18 @@ 
             $fields = [];
             foreach ($type->getFields() as $key => $field) {
                 $newType = static::create(
-                    $field->config + ['typeSetter' => $config['typeSetter'] ?? null],
+                    $field->config+['typeSetter' => $config['typeSetter'] ?? null],
                     array_merge($path, [$key])
                 );
 
-                if (! $newType) {
+                if (!$newType) {
                     continue;
                 }
 
                 $fields[$key] = [
                     'description' => 'Error for ' . $key,
                     'type' => $newType,
-                    'resolve' => static function ($value) use ($key) {
+                    'resolve' => static function($value) use ($key) {
                         return $value[$key] ?? null;
                     },
                 ];
@@ -93,7 +93,7 @@ 
                         'fields' => $fields,
                     ]), $config),
                     'description' => 'Suberrors for ' . ucfirst($path[count($path)-1]),
-                    'resolve' => static function (array $value) {
+                    'resolve' => static function(array $value) {
                         return $value['suberrors'] ?? null;
                     },
                 ];
@@ -117,7 +117,7 @@ 
                 $finalFields['suberrors'] = [
                     'description' => 'Suberrors for the list of ' . $type->ofType . ' items',
                     'type' => $newType,
-                    'resolve' => static function ($value) {
+                    'resolve' => static function($value) {
                         return $value['suberrors'] ?? null;
                     },
                 ];
@@ -131,7 +131,7 @@ 
             $finalFields['path'] = [
                 'type' => Type::listOf(Type::int()),
                 'description' => 'A path describing this items\'s location in the nested array',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     return $value['path'];
                 },
             ];
@@ -149,7 +149,7 @@ 
      *
      * @return mixed
      */
-    protected function _set(Type $type, array $config)
+    protected function _set (Type $type, array $config)
     {
         if (is_callable($config['typeSetter'] ?? null)) {
             $config['typeSetter']($type);
@@ -166,14 +166,14 @@ 
      *
      * @throws Exception
      */
-    public static function create(array $config, array $path, bool $isParentList = false, $name = '') : ?self
+    public static function create (array $config, array $path, bool $isParentList = false, $name = '') : ?self
     {
         $config['fields'] = $config['fields'] ?? [];
         if (isset($config['validate']) && is_callable($config['validate'])) {
             $config['fields']['code'] = $config['fields']['code'] ?? [
                 'type' => Type::int(),
                 'description' => 'A numeric error code. 0 on success, non-zero on failure.',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     $error = $value['error'] ?? null;
                     switch (gettype($error)) {
                         case 'integer':
@@ -186,7 +186,7 @@ 
             $config['fields']['msg'] = $config['fields']['msg'] ?? [
                 'type' => Type::string(),
                 'description' => 'An error message.',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     $error = $value['error'] ?? null;
                     switch (gettype($error)) {
                         case 'integer':
@@ -211,7 +211,7 @@ 
     /**
      * @param string[] $path
      */
-    protected function _nameFromPath(array $path) : string
+    protected function _nameFromPath (array $path) : string
     {
         return implode('_', array_map('ucfirst', $path));
     }

src/Type/Definition/ValidatedFieldDefinition.php

Doc Comments +1 added lines, -1 removed lines

@@ -183,7 +183,7 @@
     }
 
     /**
-     * @return mixed|string|string[]|null
+     * @return string
      *
      * @throws ReflectionException
      */

Spacing +14 added lines, -14 removed lines

@@ -24,7 +24,7 @@ 
     /**
      * @param mixed[] $config
      */
-    public function __construct($config)
+    public function __construct ($config)
     {
         $args = $config['args'];
         $name = $config['name'] ?? lcfirst($this->tryInferName());
@@ -37,14 +37,14 @@ 
                 'result' => [
                     'type' => $config['type'],
                     'description' => 'The payload, if any',
-                    'resolve' => static function ($value) {
+                    'resolve' => static function($value) {
                         return $value['result'] ?? null;
                     },
                 ],
                 'valid' => [
                     'type' => Type::nonNull(Type::boolean()),
                     'description' => 'Whether all validation passed. True for yes, false for no.',
-                    'resolve' => static function ($value) {
+                    'resolve' => static function($value) {
                         return $value['valid'];
                     },
                 ],
@@ -61,17 +61,17 @@ 
             'type' => $type,
             'args' => $args,
             'name' => $name,
-            'resolve' => function ($value, $args1, $context, $info) use ($config, $args) {
+            'resolve' => function($value, $args1, $context, $info) use ($config, $args) {
                 // validate inputs
-                $config['type']  = new InputObjectType([
+                $config['type'] = new InputObjectType([
                     'name'=>'',
                     'fields' => $args,
                 ]);
                 $errors          = $this->_validate($config, $args1);
                 $result          = $errors;
-                $result['valid'] = ! $errors;
+                $result['valid'] = !$errors;
 
-                if (! isset($result['error']) && ! isset($result['suberrors'])) {
+                if (!isset($result['error']) && !isset($result['suberrors'])) {
                     $result['result'] = $config['resolve']($value, $args1, $context, $info);
                 }
 
@@ -83,12 +83,12 @@ 
     /**
      * @param   mixed[] $arr
      */
-    protected function _isAssoc(array $arr) : bool
+    protected function _isAssoc (array $arr) : bool
     {
         if ($arr === []) {
             return false;
         }
-        return array_keys($arr) !== range(0, count($arr) - 1);
+        return array_keys($arr) !== range(0, count($arr)-1);
     }
 
     /**
@@ -97,16 +97,16 @@ 
      *
      * @throws  ValidateItemsError
      */
-    protected function _validateItems(array $value, array $path, callable $validate)
+    protected function _validateItems (array $value, array $path, callable $validate)
     {
         foreach ($value as $idx => $subValue) {
-            if (is_array($subValue) && ! $this->_isAssoc($subValue)) {
+            if (is_array($subValue) && !$this->_isAssoc($subValue)) {
                 $path[count($path)-1] = $idx;
                 $newPath              = $path;
                 $newPath[]            = 0;
                 $this->_validateItems($subValue, $newPath, $validate);
             } else {
-                $path[count($path) - 1] = $idx;
+                $path[count($path)-1] = $idx;
                 $err                    = $validate($subValue);
 
                 if ($err) {
@@ -119,7 +119,7 @@ 
     /**
      * @return mixed[]
      */
-    protected function _validate($arg, $value)
+    protected function _validate ($arg, $value)
     {
         $res = [];
 
@@ -187,7 +187,7 @@ 
      *
      * @throws ReflectionException
      */
-    protected function tryInferName()
+    protected function tryInferName ()
     {
         // If class is extended - infer name from className
         // QueryType -> Type

tests/Type/ValidatedFieldDefinition/ScalarValidationTest.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     /** @var Schema */
     protected $schema;
 
-    protected function setUp()
+    protected function setUp ()
     {
         $this->personType = new ObjectType([
             'name' => 'Person',
@@ -55,13 +55,13 @@ 
             'fields' => [
                 'title' => [
                     'type' => Type::string(),
-                    'resolve' => static function ($book) {
+                    'resolve' => static function($book) {
                         return $book['title'];
                     },
                 ],
                 'author' => [
                     'type' => $this->personType,
-                    'resolve' => static function ($book) {
+                    'resolve' => static function($book) {
                         return $book['author'];
                     },
                 ],
@@ -74,7 +74,7 @@ 
             'query' => $this->query,
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => function () {
+                'fields' => function() {
                     return [
                         'updateBook' => new ValidatedFieldDefinition([
                             'name' => 'updateBook',
@@ -83,7 +83,7 @@ 
                                 'bookId' => [
                                     'type' => Type::id(),
                                     'errorCodes' => ['bookNotFound'],
-                                    'validate' => function ($bookId) {
+                                    'validate' => function($bookId) {
                                         if (isset($this->data['books'][$bookId])) {
                                             return 0;
                                         }
@@ -92,7 +92,7 @@ 
                                     },
                                 ],
                             ],
-                            'resolve' => static function ($value) : bool {
+                            'resolve' => static function($value) : bool {
                                 return !!$value;
                             },
                         ]),
@@ -102,7 +102,7 @@ 
         ]);
     }
 
-    public function testNullableScalarValidationOnNullValueSuccess()
+    public function testNullableScalarValidationOnNullValueSuccess ()
     {
         $res = GraphQL::executeQuery(
             $this->schema,

tests/Type/ValidatedFieldDefinition/ListOfInputObjectValidationTest.php

Spacing +11 added lines, -11 removed lines

@@ -42,7 +42,7 @@ 
     /** @var Schema */
     protected $schema;
 
-    protected function setUp()
+    protected function setUp ()
     {
         $this->personType = new ObjectType([
             'name' => 'Person',
@@ -62,7 +62,7 @@ 
                 'title' => [
                     'type' => Type::string(),
                     'description' => 'Enter a book title, no more than 10 characters in length',
-                    'validate' => static function (string $title) {
+                    'validate' => static function(string $title) {
                         if (strlen($title) > 10) {
                             return [1, 'book title must be less than 10 chaacters'];
                         }
@@ -76,8 +76,8 @@ 
                         'unknownAuthor',
                         'authorDeceased',
                     ],
-                    'validate' => function (string $authorId) {
-                        if (! isset($this->data['people'][$authorId])) {
+                    'validate' => function(string $authorId) {
+                        if (!isset($this->data['people'][$authorId])) {
                             return ['unknownAuthor', 'We have no record of that author'];
                         }
                         return 0;
@@ -92,7 +92,7 @@ 
             'query' => $this->query,
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => function () {
+                'fields' => function() {
                     return [
                         'updateBooks' => new ValidatedFieldDefinition([
                             'name' => 'updateBooks',
@@ -100,15 +100,15 @@ 
                             'args' => [
                                 'bookAttributes' => [
                                     'type' => Type::listOf($this->bookAttributesInputType),
-                                    'validate' => static function ($var) {
-                                        return $var ? 0: 1;
+                                    'validate' => static function($var) {
+                                        return $var ? 0 : 1;
                                     },
-                                    'validateItem' => static function ($book) {
-                                        return isset($book['author']) || isset($book['title']) ? 0: 1;
+                                    'validateItem' => static function($book) {
+                                        return isset($book['author']) || isset($book['title']) ? 0 : 1;
                                     },
                                 ],
                             ],
-                            'resolve' => static function ($value) : bool {
+                            'resolve' => static function($value) : bool {
                                 return !!$value;
                             },
                         ]),
@@ -118,7 +118,7 @@ 
         ]);
     }
 
-    public function testValidationFail()
+    public function testValidationFail ()
     {
         $res = GraphQL::executeQuery(
             $this->schema,

tests/Type/ValidatedFieldDefinition/BasicTest.php

Spacing +6 added lines, -6 removed lines

@@ -9,7 +9,7 @@ 
 
 final class BasicTest extends TestCase
 {
-    public function testInferName()
+    public function testInferName ()
     {
         $def = new NamelessDef([
             'type' => Type::boolean(),
@@ -17,12 +17,12 @@ 
                 'bookId' => [
                     'type' => Type::id(),
                     'errorCodes' => ['bookNotFound'],
-                    'validate' => static function ($bookId) {
+                    'validate' => static function($bookId) {
                         return $bookId ? 0 : 1;
                     },
                 ],
             ],
-            'resolve' => static function ($value) : bool {
+            'resolve' => static function($value) : bool {
                 return !!$value;
             },
         ]);
@@ -30,7 +30,7 @@ 
         static::assertEquals('namelessDef', $def->name);
     }
 
-    public function testInferNameNameAlreadyExists()
+    public function testInferNameNameAlreadyExists ()
     {
         $def = new NamelessDef([
             'name' => 'FooDef',
@@ -39,12 +39,12 @@ 
                 'bookId' => [
                     'type' => Type::id(),
                     'errorCodes' => ['bookNotFound'],
-                    'validate' => static function ($bookId) {
+                    'validate' => static function($bookId) {
                         return $bookId ? 0 : 1;
                     },
                 ],
             ],
-            'resolve' => static function ($value) : bool {
+            'resolve' => static function($value) : bool {
                 return !!$value;
             },
         ]);