Youshido / GraphQL

src/Type/TypeService.php

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
         if (is_object($data)) {
             $getter = $path;
             if (substr($path, 0, 2) != 'is') {
-                $getter = 'get' . self::classify($path);
+                $getter = 'get'.self::classify($path);
             }
 
             return is_callable([$data, $getter]) ? $data->$getter() : null;

src/Type/InputObject/AbstractInputObjectType.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         }
 
         $typeConfig     = $this->getConfig();
-        $requiredFields = array_filter($typeConfig->getFields(), function (FieldInterface $field) {
+        $requiredFields = array_filter($typeConfig->getFields(), function(FieldInterface $field) {
             return $field->getType()->getKind() == TypeMap::KIND_NON_NULL;
         });
 

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/ConfigCallTrait.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
             return $this->config->get($propertyName);
         }
 
-        throw new \Exception('Call to undefined method ' . $method);
+        throw new \Exception('Call to undefined method '.$method);
     }
 
 }

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

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     public function assertValidateConfig(AbstractConfig $config)
     {
         if (!$this->validate($config->getData(), $this->getConfigFinalRules($config), $config->isExtraFieldsAllowed())) {
-            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)));
         }
     }
 

src/Validator/ResolveValidator/ResolveValidator.php

Spacing +4 added lines, -4 removed lines

@@ -61,11 +61,11 @@ 
     {
         if (!count($field->getArguments())) return true;
 
-        $requiredArguments = array_filter($field->getArguments(), function (InputField $argument) {
+        $requiredArguments = array_filter($field->getArguments(), function(InputField $argument) {
             return $argument->getType()->getKind() == TypeMap::KIND_NON_NULL;
         });
 
-        $withDefaultArguments = array_filter($field->getArguments(), function (InputField $argument) {
+        $withDefaultArguments = array_filter($field->getArguments(), function(InputField $argument) {
             return $argument->getConfig()->get('default') !== null;
         });
 
@@ -133,7 +133,7 @@ 
     public function assertTypeImplementsInterface(AbstractType $type, AbstractInterfaceType $interface)
     {
         if (!$interface->isValidValue($type)) {
-            throw new ResolveException('Type ' . $type->getName() . ' does not implement ' . $interface->getName());
+            throw new ResolveException('Type '.$type->getName().' does not implement '.$interface->getName());
         }
     }
 
@@ -152,7 +152,7 @@ 
         }
 
         if (!$valid) {
-            throw new ResolveException('Type ' . $type->getName() . ' not exist in types of ' . $unionType->getName());
+            throw new ResolveException('Type '.$type->getName().' not exist in types of '.$unionType->getName());
         }
     }
 

Doc Comments +3 added lines, -2 removed lines

@@ -42,8 +42,8 @@ 
 
     /**
      * @param AbstractObjectType      $objectType
-     * @param Mutation|Query|AstField $field
-     * @return null
+     * @param AstField $field
+     * @return boolean
      */
     public function objectHasField($objectType, $field)
     {
@@ -58,6 +58,7 @@ 
 
     /**
      * @inheritdoc
+     * @param Query $query
      */
     public function validateArguments(AbstractField $field, $query, Request $request)
     {

Braces +3 added lines, -1 removed lines

@@ -149,7 +149,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()) {

Indentation +3 added lines, -3 removed lines

@@ -191,12 +191,12 @@
     {
         $fieldType = $field->getType();
         if (is_null($value)) {
-          if ($fieldType->getKind() == TypeMap::KIND_NON_NULL) {
+            if ($fieldType->getKind() == TypeMap::KIND_NON_NULL) {
             $this->executionContext->addError(new ResolveException(sprintf('Cannot return null for non-nullable field %s', $field->getName())));
             return false;
-          }
+            }
 
-          return true;
+            return true;
         }
 
         $fieldType = $this->resolveTypeIfAbstract($fieldType->getNullableType(), $value);

src/Relay/Connection/ArrayConnection.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public static function offsetToCursor($offset)
     {
-        return base64_encode(self::PREFIX . $offset);
+        return base64_encode(self::PREFIX.$offset);
     }
 
     /**

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

src/Relay/Connection/Connection.php

Spacing +5 added lines, -5 removed lines

@@ -52,13 +52,13 @@ 
         $edgeFields = !empty($config['edgeFields']) ? $config['edgeFields'] : [];
 
         $edgeType = new ObjectType([
-            'name'        => $name . 'Edge',
+            'name'        => $name.'Edge',
             'description' => 'An edge in a connection.',
             'fields'      => array_merge([
                 'node'   => [
                     'type'        => $type,
                     'description' => 'The item at the end of the edge',
-                    'resolve'     => function ($value) {
+                    'resolve'     => function($value) {
                         return $value['node'];
                     }
                 ],
@@ -86,20 +86,20 @@ 
         $connectionFields = !empty($config['connectionFields']) ? $config['connectionFields'] : [];
 
         $connectionType = new ObjectType([
-            'name'        => $name . 'Connection',
+            'name'        => $name.'Connection',
             'description' => 'A connection to a list of items.',
             'fields'      => array_merge([
                 'pageInfo' => [
                     'type'        => new NonNullType(self::getPageInfoType()),
                     'description' => 'Information to aid in pagination.',
-                    'resolve'     => function ($value) {
+                    'resolve'     => function($value) {
                         return isset($value['pageInfo']) ? $value['pageInfo'] : null;
                     }
                 ],
                 'edges'    => [
                     'type'        => new ListType(self::edgeDefinition($type, $name, $config)),
                     'description' => 'A list of edges.',
-                    'resolve'     => function ($value) {
+                    'resolve'     => function($value) {
                         return isset($value['edges']) ? $value['edges'] : null;
                     }
                 ]