Youshido / GraphQL

src/Config/Schema/SchemaConfig.php

Doc Comments +4 added lines, -1 removed lines

@@ -23,6 +23,9 @@ 
      */
     private $typesList;
 
+    /**
+     * @param \Youshido\GraphQL\Schema\AbstractSchema $contextObject
+     */
     public function __construct(array $configData, $contextObject = null, $finalClass = false)
     {
         $this->typesList = new SchemaTypesList();
@@ -78,7 +81,7 @@ 
     }
 
     /**
-     * @param $query AbstractObjectType
+     * @param ObjectType $query AbstractObjectType
      *
      * @return SchemaConfig
      */

src/Field/AbstractField.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     }
     protected $isFinal = false;
 
-    private $nameCache            = null;
+    private $nameCache = null;
 
     public function __construct(array $config = [])
     {
@@ -34,7 +34,7 @@ 
             $config['type'] = $this->getType();
             $config['name'] = $this->getName();
             if (empty($config['name'])) {
-                $config['name'] =$this->getAutoName();
+                $config['name'] = $this->getAutoName();
             }
         }
 

src/Type/Enum/AbstractEnumType.php

Braces +3 added lines, -1 removed lines

@@ -48,7 +48,9 @@
      */
     public function isValidValue($value)
     {
-        if (is_null($value)) return true;
+        if (is_null($value)) {
+            return true;
+        }
         foreach ($this->getConfig()->get('values') as $item) {
             if ($value === $item['name'] || $value === $item['value']) {
                 return true;

Tests/Library/Type/ScalarTypeTest.php

Spacing +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
     {
         foreach (TypeFactory::getScalarTypesNames() as $typeName) {
             $scalarType     = TypeFactory::getScalarType($typeName);
-            $testDataMethod = 'get' . $typeName . 'TestData';
+            $testDataMethod = 'get'.$typeName.'TestData';
 
             $this->assertNotEmpty($scalarType->getDescription());
             $this->assertEquals($scalarType->getKind(), TypeMap::KIND_SCALAR);
@@ -39,9 +39,9 @@ 
                 $this->assertParse($scalarType, $data, $serialized, $typeName);
 
                 if ($isValid) {
-                    $this->assertTrue($scalarType->isValidValue($data), $typeName . ' validation for :' . serialize($data));
+                    $this->assertTrue($scalarType->isValidValue($data), $typeName.' validation for :'.serialize($data));
                 } else {
-                    $this->assertFalse($scalarType->isValidValue($data), $typeName . ' validation for :' . serialize($data));
+                    $this->assertFalse($scalarType->isValidValue($data), $typeName.' validation for :'.serialize($data));
                 }
             }
         }
@@ -62,7 +62,7 @@ 
 
     private function assertSerialization(AbstractScalarType $object, $input, $expected)
     {
-        $this->assertEquals($expected, $object->serialize($input), $object->getName() . ' serialize for: ' . serialize($input));
+        $this->assertEquals($expected, $object->serialize($input), $object->getName().' serialize for: '.serialize($input));
     }
 
     private function assertParse(AbstractScalarType $object, $input, $expected, $typeName)
@@ -71,7 +71,7 @@ 
         if ($parsed instanceof \DateTime) {
             $expected = \DateTime::createFromFormat($typeName == 'datetime' ? 'Y-m-d H:i:s' : 'D, d M Y H:i:s O', $expected);
         }
-        $this->assertEquals($expected, $parsed, $object->getName() . ' parse for: ' . serialize($input));
+        $this->assertEquals($expected, $parsed, $object->getName().' parse for: '.serialize($input));
     }
 
 }

Tests/Schema/InputObjectDefaultValuesTest.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
                 ],
             ]
         ]);
-        $schema   = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -49,7 +49,7 @@ 
                                 ]
                             ])
                         ],
-                        'resolve'    => function ($source, $args) {
+                        'resolve'    => function($source, $args) {
                             return sprintf('Result with level %s and status %s',
                                 $args['statObject']['level'], $args['statObject']['status']
                             );

Tests/Issues/Issue90/Issue90Schema.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
                         'args'    => [
                             'date' => new DateTimeType('Y-m-d H:ia')
                         ],
-                        'resolve' => function ($value, $args, $info) {
+                        'resolve' => function($value, $args, $info) {
 
                             if (isset($args['date'])) {
                                 return $args['date'];
@@ -42,7 +42,7 @@ 
                         'args'    => [
                             'date' => new DateTimeType('Y-m-d H:ia')
                         ],
-                        'resolve' => function ($value, $args, $info) {
+                        'resolve' => function($value, $args, $info) {
 
                             if (isset($args['date'])) {
                                 return $args['date'];

Tests/Schema/VariablesTest.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
                         'args'    => [
                             'ids' => new ListType(new NonNullType(new IdType()))
                         ],
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'item';
                         }
                     ],
@@ -85,7 +85,7 @@ 
                         'args'    => [
                             'sortOrder' => new StringType(),
                         ],
-                        'resolve' => function ($args) {
+                        'resolve' => function($args) {
                             return sprintf('Result with %s order', empty($args['sortOrder']) ? 'default' : $args['sortOrder']);
                         },
                     ],

Tests/Library/Type/InputObjectTypeTest.php

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
@@ -72,7 +72,7 @@ 
                             ]))
                         ],
                         'type'    => new BooleanType(),
-                        'resolve' => function ($object, $args) {
+                        'resolve' => function($object, $args) {
                             return true;
                         }
                     ]
@@ -98,14 +98,14 @@ 
         );
     }
 
-    public function testNullableInputWithNonNull(){
+    public function testNullableInputWithNonNull() {
         $processor = new Processor(new Schema([
             'query'    => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
@@ -124,7 +124,7 @@ 
                             ])
                         ],
                         'type'    => new BooleanType(),
-                        'resolve' => function ($object, $args) {
+                        'resolve' => function($object, $args) {
                             return true;
                         }
                     ]
@@ -148,7 +148,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                         }
                     ],
                 ]
@@ -171,7 +171,7 @@ 
                                 ]
                             ])
                         ],
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'success message';
                         }
                     ]

src/Config/Traits/FieldsAwareConfigTrait.php

Doc Comments +4 added lines, -1 removed lines

@@ -90,7 +90,7 @@ 
     }
 
     /**
-     * @param $name
+     * @param string $name
      *
      * @return Field
      */
@@ -122,6 +122,9 @@ 
         return $this->fields;
     }
 
+    /**
+     * @param string $name
+     */
     public function removeField($name)
     {
         if ($this->hasField($name)) {