Youshido / GraphQL

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;

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Youshido\GraphQL\Type\AbstractType;
 use Youshido\GraphQL\Type\Config\Object\UnionTypeConfig;
 use Youshido\GraphQL\Type\Config\Traits\ConfigCallTrait;
-use Youshido\GraphQL\Type\Config\TypeConfigInterface;
 use Youshido\GraphQL\Type\Traits\AutoNameTrait;
 use Youshido\GraphQL\Type\TypeMap;
 

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/02_blog/schema-bootstrap.php

Spacing +10 added lines, -10 removed lines

@@ -12,16 +12,16 @@
 use Youshido\GraphQL\Type\ListType\ListType;
 use Youshido\GraphQL\Type\Object\ObjectType;
 
-require_once __DIR__ . '/../../vendor/autoload.php';
-require_once __DIR__ . '/Schema/DataProvider.php';
-require_once __DIR__ . '/Schema/PostType.php';
-require_once __DIR__ . '/Schema/PostStatus.php';
-require_once __DIR__ . '/Schema/ContentBlockInterface.php';
-require_once __DIR__ . '/Schema/LikePost.php';
-require_once __DIR__ . '/Schema/BannerType.php';
-require_once __DIR__ . '/Schema/ContentBlockUnion.php';
-require_once __DIR__ . '/Schema/PostInputType.php';
-require_once __DIR__ . '/Schema/BlogSchema.php';
+require_once __DIR__.'/../../vendor/autoload.php';
+require_once __DIR__.'/Schema/DataProvider.php';
+require_once __DIR__.'/Schema/PostType.php';
+require_once __DIR__.'/Schema/PostStatus.php';
+require_once __DIR__.'/Schema/ContentBlockInterface.php';
+require_once __DIR__.'/Schema/LikePost.php';
+require_once __DIR__.'/Schema/BannerType.php';
+require_once __DIR__.'/Schema/ContentBlockUnion.php';
+require_once __DIR__.'/Schema/PostInputType.php';
+require_once __DIR__.'/Schema/BlogSchema.php';
 
 /**
 $rootQueryType = new ObjectType([

examples/02_blog/index.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 use Youshido\GraphQL\Processor;
 use Youshido\GraphQL\Schema;
 
-require_once __DIR__ . '/schema-bootstrap.php';
+require_once __DIR__.'/schema-bootstrap.php';
 /** @var Schema $schema */
 $schema = new BlogSchema();
 
@@ -20,4 +20,4 @@ 
 $payload  = '{ pageContentInterface { title} }';
 $response = $processor->processRequest($payload, [])->getResponseData();
 
-echo json_encode($response) . "\n\n";
+echo json_encode($response)."\n\n";

examples/02_blog/router.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 use Youshido\GraphQL\Processor;
 use Youshido\GraphQL\Schema;
 
-require_once __DIR__ . '/schema-bootstrap.php';
+require_once __DIR__.'/schema-bootstrap.php';
 /** @var Schema $schema */
 $schema = new BlogSchema();
 

examples/02_blog/Schema/BlogSchema.php

Spacing +3 added lines, -3 removed lines

@@ -18,19 +18,19 @@
             'latestPost'           => new PostType(),
             'randomBanner'         => [
                 'type'    => new BannerType(),
-                'resolve' => function () {
+                'resolve' => function() {
                     return DataProvider::getBanner(rand(1, 10));
                 }
             ],
             'pageContentUnion'     => [
                 'type'    => new ListType(new ContentBlockUnion()),
-                'resolve' => function () {
+                'resolve' => function() {
                     return [DataProvider::getPost(1), DataProvider::getBanner(1)];
                 }
             ],
             'pageContentInterface' => [
                 'type'    => new ListType(new ContentBlockInterface()),
-                'resolve' => function () {
+                'resolve' => function() {
                     return [DataProvider::getPost(2), DataProvider::getBanner(3)];
                 }
             ]