Youshido / GraphQL

Tests/StarWars/Schema/QueryType.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 
 use Youshido\GraphQL\Field\Field;
-use Youshido\GraphQL\Field\FieldFactory;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
 use Youshido\GraphQL\Type\Scalar\IdType;
 

src/Type/TypeFactory.php

Braces +3 added lines, -1 removed lines

@@ -26,7 +26,9 @@
     {
         if (is_object($typeName)) {
             /** freeing memory */
-            if (!($typeName instanceof AbstractScalarType)) throw new ConfigurationException('Configuration problem with type ' . $typeName);
+            if (!($typeName instanceof AbstractScalarType)) {
+                throw new ConfigurationException('Configuration problem with type ' . $typeName);
+            }
             $typeName = $typeName->getName();
         }
         if (TypeService::isScalarType($typeName)) {

src/Execution/Processor.php

Braces +3 added lines, -1 removed lines

@@ -144,7 +144,9 @@
     protected function collectValueForQueryWithType(Query $query, AbstractType $fieldType, $resolvedValue)
     {
         $fieldType = $this->resolveValidator->resolveTypeIfAbstract($fieldType, $resolvedValue);
-        if (is_null($resolvedValue)) return null;
+        if (is_null($resolvedValue)) {
+            return null;
+        }
 
         $value = [];
         if ($fieldType->getKind() == TypeMap::KIND_LIST) {

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;

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/Validator/ResolveValidator/ResolveValidator.php

Braces +6 added lines, -2 removed lines

@@ -59,7 +59,9 @@ 
      */
     public function validateArguments(AbstractField $field, $query, Request $request)
     {
-        if (!count($field->getArguments())) return true;
+        if (!count($field->getArguments())) {
+            return true;
+        }
 
         $requiredArguments = array_filter($field->getArguments(), function (InputField $argument) {
             return $argument->getType()->getKind() == TypeMap::KIND_NON_NULL;
@@ -141,7 +143,9 @@ 
     {
         $unionTypes = $unionType->getTypes();
         $valid      = false;
-        if (empty($unionTypes)) return false;
+        if (empty($unionTypes)) {
+            return false;
+        }
 
         foreach ($unionTypes as $unionType) {
             if ($unionType->getName() == $type->getName()) {

src/Relay/Connection/ArrayConnection.php

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
 
     public static function cursorForObjectInConnection($data, $object)
     {
-        if (!is_array($data)) return null;
+        if (!is_array($data)) {
+            return null;
+        }
 
         $index = array_search($object, $data);
         return $index === false ? null : self::offsetToCursor($index);