Youshido / GraphQL

src/Config/Traits/ArgumentsAwareConfigTrait.php

Braces +3 added lines, -1 removed lines

@@ -18,7 +18,9 @@
 
     public function buildArguments()
     {
-        if ($this->_isArgumentsBuilt) return true;
+        if ($this->_isArgumentsBuilt) {
+            return true;
+        }
 
         if (!empty($this->data['args'])) {
             $this->addArguments($this->data['args']);

src/Validator/ConfigValidator/Rules/TypeValidationRule.php

Braces +21 added lines, -7 removed lines

@@ -28,7 +28,9 @@ 
 
     public function validate($data, $ruleInfo)
     {
-        if (!is_string($ruleInfo)) return false;
+        if (!is_string($ruleInfo)) {
+            return false;
+        }
 
         switch ($ruleInfo) {
             case TypeService::TYPE_ANY:
@@ -98,7 +100,9 @@ 
 
     private function isEnumValues($data)
     {
-        if (!is_array($data) || empty($data)) return false;
+        if (!is_array($data) || empty($data)) {
+            return false;
+        }
 
         foreach ($data as $item) {
             if (!is_array($item) || !array_key_exists('name', $item) || !is_string($item['name']) || !preg_match('/^[_a-zA-Z][_a-zA-Z0-9]*$/', $item['name'])) {
@@ -115,7 +119,9 @@ 
 
     private static function isArrayOfInterfaces($data)
     {
-        if (!is_array($data)) return false;
+        if (!is_array($data)) {
+            return false;
+        }
 
         foreach ($data as $item) {
             if (!TypeService::isInterface($item)) {
@@ -128,10 +134,14 @@ 
 
     private function isArrayOfFields($data)
     {
-        if (!is_array($data) || empty($data)) return false;
+        if (!is_array($data) || empty($data)) {
+            return false;
+        }
 
         foreach ($data as $name => $item) {
-            if (!$this->isField($item, $name)) return false;
+            if (!$this->isField($item, $name)) {
+                return false;
+            }
         }
 
         return true;
@@ -157,10 +167,14 @@ 
 
     private function isArrayOfInputFields($data)
     {
-        if (!is_array($data)) return false;
+        if (!is_array($data)) {
+            return false;
+        }
 
         foreach ($data as $name => $item) {
-            if (!$this->isInputField($item)) return false;
+            if (!$this->isInputField($item)) {
+                return false;
+            }
         }
 
         return true;

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 namespace Youshido\GraphQL\Validator\ConfigValidator\Rules;
 
 
-use Youshido\GraphQL\Field\AbstractInputField;
 use Youshido\GraphQL\Field\FieldInterface;
 use Youshido\GraphQL\Field\InputFieldInterface;
 use Youshido\GraphQL\Type\AbstractType;

src/Validator/ConfigValidator/ConfigValidator.php

Braces +6 added lines, -2 removed lines

@@ -36,7 +36,9 @@ 
 
     public function validate($data, $rules = [], $extraFieldsAllowed = null)
     {
-        if ($extraFieldsAllowed !== null) $this->setExtraFieldsAllowed($extraFieldsAllowed);
+        if ($extraFieldsAllowed !== null) {
+            $this->setExtraFieldsAllowed($extraFieldsAllowed);
+        }
 
         $processedFields = [];
         foreach ($rules as $fieldName => $fieldRules) {
@@ -54,7 +56,9 @@ 
             } elseif (!array_key_exists($fieldName, $data)) {
                 continue;
             }
-            if (!empty($fieldRules['final'])) unset($fieldRules['final']);
+            if (!empty($fieldRules['final'])) {
+                unset($fieldRules['final']);
+            }
 
             /** Validation of all other rules*/
             foreach ($fieldRules as $ruleName => $ruleInfo) {

src/Field/AbstractInputField.php

Unused Use Statements -2 removed lines

@@ -9,8 +9,6 @@
 
 
 use Youshido\GraphQL\Config\Field\InputFieldConfig;
-use Youshido\GraphQL\Config\Traits\ResolvableObjectTrait;
-use Youshido\GraphQL\Type\InputObject\AbstractInputObjectType;
 use Youshido\GraphQL\Type\InputTypeInterface;
 use Youshido\GraphQL\Type\Traits\AutoNameTrait;
 use Youshido\GraphQL\Type\Traits\FieldsArgumentsAwareObjectTrait;

src/Type/SchemaTypesList.php

Braces +6 added lines, -2 removed lines

@@ -42,7 +42,9 @@ 
     public function addType(TypeInterface $type)
     {
         $typeName = $this->getTypeName($type);
-        if ($this->isTypeNameRegistered($typeName)) return $this;
+        if ($this->isTypeNameRegistered($typeName)) {
+            return $this;
+        }
 
         $this->typesList[$typeName] = $type;
         return $this;
@@ -54,7 +56,9 @@ 
     }
 
     private function getTypeName($type) {
-        if (is_string($type)) return $type;
+        if (is_string($type)) {
+            return $type;
+        }
         if (is_object($type) && $type instanceof AbstractType) {
             return $type->getName();
         }

Tests/Schema/FragmentsTest.php

Unused Use Statements -4 removed lines

@@ -2,14 +2,10 @@
 
 namespace Youshido\Tests\Schema;
 
-use Youshido\GraphQL\Config\Object\InterfaceTypeConfig;
-use Youshido\GraphQL\Config\Object\ObjectTypeConfig;
 use Youshido\GraphQL\Execution\Processor;
 use Youshido\GraphQL\Schema\Schema;
 use Youshido\GraphQL\Type\InterfaceType\AbstractInterfaceType;
-use Youshido\GraphQL\Type\InterfaceType\InterfaceType;
 use Youshido\GraphQL\Type\ListType\ListType;
-use Youshido\GraphQL\Type\NonNullType;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\Scalar\IdType;

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;

examples/02_blog/Schema/BlogSchema.php

Braces +3 added lines, -1 removed lines

@@ -53,7 +53,9 @@
                     // code for creating a new post goes here
                     // we simple use our DataProvider for now
                     $post = DataProvider::getPost(10);
-                    if (!empty($args['post']['title'])) $post['title'] = $args['post']['title'];
+                    if (!empty($args['post']['title'])) {
+                        $post['title'] = $args['post']['title'];
+                    }
 
                     return $post;
                 }

src/Introspection/Traits/TypeCollectorTrait.php

Braces +3 added lines, -1 removed lines

@@ -20,7 +20,9 @@
 
     protected function collectTypes(AbstractType $type)
     {
-        if (is_object($type) && array_key_exists($type->getName(), $this->types)) return;
+        if (is_object($type) && array_key_exists($type->getName(), $this->types)) {
+            return;
+        }
 
         switch ($type->getKind()) {
             case TypeMap::KIND_INTERFACE: