Youshido / GraphQL

src/Type/Object/AbstractInputObjectType.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
             return false;
         }
 
-        $requiredFields = array_filter($this->getConfig()->getFields(), function (Field $field) {
+        $requiredFields = array_filter($this->getConfig()->getFields(), function(Field $field) {
             return $field->getConfig()->isRequired();
         });
 

examples/02_blog_inline/inline-schema.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace BlogTest;
 
-use Youshido\GraphQL\Type\NonNullType;
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\Scalar\BooleanType;
 use Youshido\GraphQL\Type\Scalar\IntType;

Spacing +7 added lines, -7 removed lines

@@ -17,16 +17,16 @@ 
             'fields'  => [
                 // here you have a complex field with a lot of options
                 'title'     => [
-                    'type'              => new StringType(),                    // string type
-                    'description'       => 'This field contains a post title',  // description
-                    'isDeprecated'      => true,                                // marked as deprecated
-                    'deprecationReason' => 'field title is now deprecated',     // explain the reason
+                    'type'              => new StringType(), // string type
+                    'description'       => 'This field contains a post title', // description
+                    'isDeprecated'      => true, // marked as deprecated
+                    'deprecationReason' => 'field title is now deprecated', // explain the reason
                     'args'              => [
                         'truncated' => new BooleanType()                        // add an optional argument
                     ],
-                    'resolve'           => function ($value, $args) {
+                    'resolve'           => function($value, $args) {
                         // used argument to modify a field value
-                        return (!empty($args['truncated'])) ? explode(' ', $value)[0] . '...' : $value;
+                        return (!empty($args['truncated'])) ? explode(' ', $value)[0].'...' : $value;
                     }
                 ],
                 // if field just has a type, you can use a short declaration syntax like this
@@ -38,7 +38,7 @@ 
                 'id' => new IntType()
             ],
             // resolve function for the query
-            'resolve' => function ($value, $args, $type) {
+            'resolve' => function($value, $args, $type) {
                 return [
                     'title'   => 'Title for the latest Post',
                     'summary' => 'Post summary',

src/AbstractSchema.php

Doc Comments +3 added lines

@@ -45,6 +45,9 @@
     {
     }
 
+    /**
+     * @param string $name
+     */
     public function addQuery($name, AbstractObjectType $query)
     {
         $this->getQueryType()->getConfig()->addField($name, $query);

Spacing +2 added lines, -2 removed lines

@@ -22,10 +22,10 @@
     public function __construct($config = [])
     {
         if (!array_key_exists('query', $config)) {
-            $config['query'] = new ObjectType(['name' => $this->getName() . 'Query']);
+            $config['query'] = new ObjectType(['name' => $this->getName().'Query']);
         }
         if (!array_key_exists('mutation', $config)) {
-            $config['mutation'] = new ObjectType(['name' => $this->getName() . 'Mutation']);
+            $config['mutation'] = new ObjectType(['name' => $this->getName().'Mutation']);
         }
 
         $this->config = new SchemaConfig($config, $this);

src/Processor.php

Doc Comments +6 added lines, -3 removed lines

@@ -353,6 +353,9 @@ 
         throw new \Exception(sprintf('Property "%s" not found in resolve result', $astField->getName()));
     }
 
+    /**
+     * @param string $path
+     */
     protected function getPropertyValue($data, $path)
     {
         if (is_object($data)) {
@@ -398,8 +401,8 @@ 
     }
 
     /**
-     * @param $field     Field
-     * @param $query     Query
+     * @param Field $field     Field
+     * @param Query $query     Query
      *
      * @return array
      */
@@ -418,7 +421,7 @@ 
     }
 
     /**
-     * @param $query         Query
+     * @param Query $query         Query
      * @param $queryType     ObjectType|TypeInterface|Field
      * @param $resolvedValue mixed
      * @param $value         array

Spacing +3 added lines, -3 removed lines

@@ -157,7 +157,7 @@ 
      */
     protected function executeMutation(Mutation $mutation, $currentLevelSchema)
     {
-        if (!$currentLevelSchema) throw new ConfigurationException('There is no mutation '. $mutation->getName());
+        if (!$currentLevelSchema) throw new ConfigurationException('There is no mutation '.$mutation->getName());
 
         if (!$this->resolveValidator->checkFieldExist($currentLevelSchema, $mutation)) {
             return null;
@@ -245,7 +245,7 @@ 
                 }
             } elseif ($field->getType()->getKind() == TypeMap::KIND_NON_NULL) {
                 if (!$field->getType()->isValidValue($preResolvedValue)) {
-                    $this->resolveValidator->addError(new ResolveException(sprintf('Cannot return null for non-nullable field %s', $astField->getName() . '.' . $field->getName())));
+                    $this->resolveValidator->addError(new ResolveException(sprintf('Cannot return null for non-nullable field %s', $astField->getName().'.'.$field->getName())));
                 } elseif (!$field->getType()->getNullableType()->isValidValue($preResolvedValue)) {
                     $this->resolveValidator->addError(new ResolveException(sprintf('Not valid value for %s field %s', $field->getType()->getNullableType()->getKind(), $field->getName())));
                     $value = null;
@@ -356,7 +356,7 @@ 
     protected function getPropertyValue($data, $path)
     {
         if (is_object($data)) {
-            $getter = 'get' . $this->classify($path);
+            $getter = 'get'.$this->classify($path);
 
             return is_callable([$data, $getter]) ? $data->$getter() : null;
         } elseif (is_array($data)) {

Braces +6 added lines, -2 removed lines

@@ -81,7 +81,9 @@ 
 
     public function processRequest($payload, $variables = [])
     {
-        if ($this->hasErrors()) return $this;
+        if ($this->hasErrors()) {
+            return $this;
+        }
 
         $this->data = [];
 
@@ -157,7 +159,9 @@ 
      */
     protected function executeMutation(Mutation $mutation, $currentLevelSchema)
     {
-        if (!$currentLevelSchema) throw new ConfigurationException('There is no mutation '. $mutation->getName());
+        if (!$currentLevelSchema) {
+            throw new ConfigurationException('There is no mutation '. $mutation->getName());
+        }
 
         if (!$this->resolveValidator->checkFieldExist($currentLevelSchema, $mutation)) {
             return null;

src/Type/Config/Schema/SchemaConfig.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 use Youshido\GraphQL\Type\Config\Config;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
-use Youshido\GraphQL\Type\Object\InputObjectType;
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\TypeMap;
 

src/Type/Object/AbstractInterfaceType.php

Doc Comments +3 added lines

@@ -55,6 +55,9 @@
         return $this;
     }
 
+    /**
+     * @param AbstractObjectType $value
+     */
     public function isValidValue($value)
     {
         if ($value instanceof AbstractObjectType) {

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     public function isValidValue($value)
     {
         if ($value instanceof AbstractObjectType) {
-            foreach($value->getInterfaces() as $interface) {
+            foreach ($value->getInterfaces() as $interface) {
                 if ($interface instanceof $this) return true;
             }
         }

Braces +3 added lines, -1 removed lines

@@ -59,7 +59,9 @@
     {
         if ($value instanceof AbstractObjectType) {
             foreach($value->getInterfaces() as $interface) {
-                if ($interface instanceof $this) return true;
+                if ($interface instanceof $this) {
+                    return true;
+                }
             }
         }
         return false;

src/Validator/ConfigValidator/Rules/TypeValidationRule.php

Unused Use Statements -5 removed lines

@@ -11,15 +11,10 @@
 
 use Youshido\GraphQL\Type\AbstractType;
 use Youshido\GraphQL\Type\Config\Field\FieldConfig;
-use Youshido\GraphQL\Type\Config\Field\InputFieldConfig;
 use Youshido\GraphQL\Type\Field\Field;
 use Youshido\GraphQL\Type\Field\InputField;
-use Youshido\GraphQL\Type\Object\AbstractEnumType;
 use Youshido\GraphQL\Type\Object\AbstractInputObjectType;
-use Youshido\GraphQL\Type\Object\AbstractInterfaceType;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
-use Youshido\GraphQL\Type\Object\AbstractUnionType;
-use Youshido\GraphQL\Type\Scalar\AbstractScalarType;
 use Youshido\GraphQL\Type\TypeMap;
 use Youshido\GraphQL\Validator\Exception\ConfigurationException;
 

Braces +21 added lines, -7 removed lines

@@ -84,7 +84,9 @@ 
 
     private function isArrayOfValues($data)
     {
-        if (!is_array($data)) return false;
+        if (!is_array($data)) {
+            return false;
+        }
 
         foreach ($data as $item) {
             if (!array_key_exists('value', $item)) {
@@ -97,7 +99,9 @@ 
 
     private static function isArrayOfInterfaces($data)
     {
-        if (!is_array($data)) return false;
+        if (!is_array($data)) {
+            return false;
+        }
 
         foreach ($data as $item) {
             if (!TypeMap::isInterface($item)) {
@@ -110,10 +114,14 @@ 
 
     private function isArrayOfFields($data)
     {
-        if (!is_array($data)) return false;
+        if (!is_array($data)) {
+            return false;
+        }
 
         foreach ($data as $name => $item) {
-            if (!$this->isField($item, $name)) return false;
+            if (!$this->isField($item, $name)) {
+                return false;
+            }
         }
 
         return true;
@@ -127,7 +135,9 @@ 
 
         try {
             /** @todo need to change it to optimize performance */
-            if (empty($data['name'])) $data['name'] = $name;
+            if (empty($data['name'])) {
+                $data['name'] = $name;
+            }
 
             $config = new FieldConfig($data);
 
@@ -141,10 +151,14 @@ 
 
     private function isArrayOfInputs($data)
     {
-        if (!is_array($data)) return false;
+        if (!is_array($data)) {
+            return false;
+        }
 
         foreach ($data as $name => $item) {
-            if (!$this->isInputField($item, $name)) return false;
+            if (!$this->isInputField($item, $name)) {
+                return false;
+            }
         }
 
         return true;

src/Validator/SchemaValidator/SchemaValidator.php

Doc Comments +2 added lines, -1 removed lines

@@ -51,7 +51,8 @@
     /**
      * @param Field $intField
      * @param Field $objField
-     * @return bool
+     * @param \Youshido\GraphQL\Type\Object\AbstractInterfaceType $interface
+     * @return boolean|null
      * @throws ConfigurationException
      */
     protected function assertFieldsIdentical($intField, $objField, $interface)

Braces +3 added lines, -1 removed lines

@@ -39,7 +39,9 @@
 
     protected function assertInterfaceImplementationCorrect(AbstractObjectType $type)
     {
-        if (!$type->getInterfaces()) return true;
+        if (!$type->getInterfaces()) {
+            return true;
+        }
 
         foreach ($type->getInterfaces() as $interface) {
             foreach ($interface->getConfig()->getFields() as $intField) {

examples/01_sandbox/index.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\Scalar\StringType;
 
-require_once __DIR__ . '/../../vendor/autoload.php';
+require_once __DIR__.'/../../vendor/autoload.php';
 
 $processor = new Processor();
 $processor->setSchema(new Schema([
@@ -24,4 +24,4 @@ 
 ]));
 
 $res = $processor->processRequest('{ currentTime }')->getResponseData();
-echo json_encode($res) . "\n\n";
+echo json_encode($res)."\n\n";