Youshido / GraphQL

src/Parser/Ast/Fragment.php

Doc Comments +4 added lines, -4 removed lines

@@ -60,7 +60,7 @@ 
     }
 
     /**
-     * @param mixed $name
+     * @param string $name
      */
     public function setName($name)
     {
@@ -76,7 +76,7 @@ 
     }
 
     /**
-     * @param mixed $model
+     * @param string $model
      */
     public function setModel($model)
     {
@@ -84,7 +84,7 @@ 
     }
 
     /**
-     * @return Field[]|Query[]
+     * @return Query[]
      */
     public function getFields()
     {
@@ -92,7 +92,7 @@ 
     }
 
     /**
-     * @param Field[]|Query[] $fields
+     * @param Field[] $fields
      */
     public function setFields($fields)
     {

src/Parser/Ast/Query.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
     }
 
     /**
-     * @return Field[]|Query[]|FragmentInterface[]
+     * @return Query[]
      */
     public function getFields()
     {

src/Parser/Ast/TypedFragmentReference.php

Doc Comments +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
     }
 
     /**
-     * @return Field[]|Query[]
+     * @return Field[]
      */
     public function getFields()
     {
@@ -36,7 +36,7 @@ 
     }
 
     /**
-     * @param Field[]|Query[] $fields
+     * @param Field[] $fields
      */
     public function setFields($fields)
     {

src/Type/InterfaceType/InterfaceType.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
     {
     }
 
+    /**
+     * @param \Youshido\GraphQL\Type\Object\ObjectType $object
+     */
     public function resolveType($object)
     {
         return $this->getConfig()->resolveType($object);

src/Validator/SchemaValidator/SchemaValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      * @param Field                 $objField
      * @param AbstractInterfaceType $interface
      *
-     * @return bool
+     * @return boolean|null
      *
      * @throws ConfigurationException
      */

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;

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/Type/Object/AbstractObjectType.php

Doc Comments +3 added lines

@@ -87,6 +87,9 @@
         return $this->getConfigValue('interfaces', []);
     }
 
+    /**
+     * @param ObjectType|null $value
+     */
     public function isValidValue($value)
     {
         return is_array($value) || is_null($value) || is_object($value);