Youshido / GraphQL

Tests/Library/Relay/MutationTest.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     {
         $mutation = RelayMutation::buildMutation('ship', [
             'name' => new StringType()
-        ],[
+        ], [
             'id' => new IdType(),
             'name' => new StringType()
         ], function($source, $args, $info) {

Tests/Library/Relay/CallableFetcherTest.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 {
     public function testMethods()
     {
-        $fetcher = new CallableFetcher(function ($type, $id) { return ['name' => $type . ' Name', 'id' => $id]; }, function ($object) { return $object; });
+        $fetcher = new CallableFetcher(function($type, $id) { return ['name' => $type.' Name', 'id' => $id]; }, function($object) { return $object; });
         $this->assertEquals([
             'name' => 'User Name',
             'id'   => 12

Tests/DataProvider/TestSchema.php

Spacing +3 added lines, -3 removed lines

@@ -22,13 +22,13 @@ 
         $config->getQuery()->addFields([
             'me'     => [
                 'type'    => new TestObjectType(),
-                'resolve' => function ($value, $args, ResolveInfo $info) {
+                'resolve' => function($value, $args, ResolveInfo $info) {
                     return $info->getReturnType()->getData();
                 }
             ],
             'status' => [
                 'type'    => new TestEnumType(),
-                'resolve' => function () {
+                'resolve' => function() {
                     return $this->testStatusValue;
                 }
             ],
@@ -36,7 +36,7 @@ 
         $config->getMutation()->addFields([
             'updateStatus' => [
                 'type'    => new TestEnumType(),
-                'resolve' => function () {
+                'resolve' => function() {
                     return $this->testStatusValue;
                 },
                 'args'    => [

Unused Use Statements -3 removed lines

@@ -12,9 +12,6 @@
 use Youshido\GraphQL\Config\Schema\SchemaConfig;
 use Youshido\GraphQL\Execution\ResolveInfo;
 use Youshido\GraphQL\Schema\AbstractSchema;
-use Youshido\GraphQL\Type\NonNullType;
-use Youshido\GraphQL\Type\Scalar\IntType;
-use Youshido\GraphQL\Type\Scalar\StringType;
 
 class TestSchema extends AbstractSchema
 {

src/Type/Scalar/StringType.php

Indentation +2 added lines, -2 removed lines

@@ -42,8 +42,8 @@
     public function getDescription()
     {
         return 'The `String` scalar type represents textual data, represented as UTF-8 ' .
-               'character sequences. The String type is most often used by GraphQL to ' .
-               'represent free-form human-readable text.';
+                'character sequences. The String type is most often used by GraphQL to ' .
+                'represent free-form human-readable text.';
     }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -27,11 +27,11 @@ 
             return null;
         }
 
-        if(is_array($value)) {
+        if (is_array($value)) {
             return '';
         }
 
-        return (string) $value;
+        return (string)$value;
     }
 
     public function isValidValue($value)
@@ -41,8 +41,8 @@ 
 
     public function getDescription()
     {
-        return 'The `String` scalar type represents textual data, represented as UTF-8 ' .
-               'character sequences. The String type is most often used by GraphQL to ' .
+        return 'The `String` scalar type represents textual data, represented as UTF-8 '.
+               'character sequences. The String type is most often used by GraphQL to '.
                'represent free-form human-readable text.';
     }
 

src/Type/Scalar/IdType.php

Indentation +4 added lines, -4 removed lines

@@ -30,9 +30,9 @@
     public function getDescription()
     {
         return 'The `ID` scalar type represents a unique identifier, often used to ' .
-               'refetch an object or as key for a cache. The ID type appears in a JSON ' .
-               'response as a String; however, it is not intended to be human-readable. ' .
-               'When expected as an input type, any string (such as `"4"`) or integer ' .
-               '(such as `4`) input value will be accepted as an ID.';
+                'refetch an object or as key for a cache. The ID type appears in a JSON ' .
+                'response as a String; however, it is not intended to be human-readable. ' .
+                'When expected as an input type, any string (such as `"4"`) or integer ' .
+                '(such as `4`) input value will be accepted as an ID.';
     }
 }

Spacing +4 added lines, -4 removed lines

@@ -29,10 +29,10 @@
 
     public function getDescription()
     {
-        return 'The `ID` scalar type represents a unique identifier, often used to ' .
-               'refetch an object or as key for a cache. The ID type appears in a JSON ' .
-               'response as a String; however, it is not intended to be human-readable. ' .
-               'When expected as an input type, any string (such as `"4"`) or integer ' .
+        return 'The `ID` scalar type represents a unique identifier, often used to '.
+               'refetch an object or as key for a cache. The ID type appears in a JSON '.
+               'response as a String; however, it is not intended to be human-readable. '.
+               'When expected as an input type, any string (such as `"4"`) or integer '.
                '(such as `4`) input value will be accepted as an ID.';
     }
 }

src/Config/Object/InterfaceTypeConfig.php

Spacing +1 added lines, -1 removed lines

@@ -45,6 +45,6 @@
             return $this->contextObject->resolveType($object);
         }
 
-        throw new ConfigurationException('There is no valid resolveType for ' . $this->getName());
+        throw new ConfigurationException('There is no valid resolveType for '.$this->getName());
     }
 }

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']);

Doc Comments +3 added lines

@@ -98,6 +98,9 @@
         return $this->arguments;
     }
 
+    /**
+     * @param string $name
+     */
     public function removeArgument($name)
     {
         if ($this->hasArgument($name)) {

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) {

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     public function assertValidConfig(AbstractConfig $config)
     {
         if (!$this->isValidConfig($config)) {
-            throw new ConfigurationException('Config is not valid for ' . ($config->getContextObject() ? get_class($config->getContextObject()) : null) . "\n" . implode("\n", $this->getErrorsArray(false)));
+            throw new ConfigurationException('Config is not valid for '.($config->getContextObject() ? get_class($config->getContextObject()) : null)."\n".implode("\n", $this->getErrorsArray(false)));
         }
     }