Completed
Push — master ( 200493...fdd355 )
by Alexandr
08:14 queued 04:55
created
src/Validator/ConfigValidator/ConfigValidator.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,9 @@
 block discarded – undo
36 36
 
37 37
     public function validate($data, $rules = [], $extraFieldsAllowed = null)
38 38
     {
39
-        if ($extraFieldsAllowed !== null) $this->setExtraFieldsAllowed($extraFieldsAllowed);
39
+        if ($extraFieldsAllowed !== null) {
40
+            $this->setExtraFieldsAllowed($extraFieldsAllowed);
41
+        }
40 42
 
41 43
         $processedFields = [];
42 44
         foreach ($rules as $fieldName => $fieldRules) {
Please login to merge, or discard this patch.
src/Parser/Ast/Query.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
     }
62 62
 
63 63
     /**
64
-     * @return Field[]|Query[]
64
+     * @return Query[]
65 65
      */
66 66
     public function getFields()
67 67
     {
Please login to merge, or discard this patch.
src/Parser/Ast/Fragment.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
     }
62 62
 
63 63
     /**
64
-     * @return Field[]|Query[]
64
+     * @return Query[]
65 65
      */
66 66
     public function getFields()
67 67
     {
Please login to merge, or discard this patch.
src/Parser/Ast/TypedFragmentReference.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
     }
29 29
 
30 30
     /**
31
-     * @return Field[]|Query[]
31
+     * @return Field[]
32 32
      */
33 33
     public function getFields()
34 34
     {
Please login to merge, or discard this patch.
src/AbstractSchema.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -45,6 +45,9 @@
 block discarded – undo
45 45
     {
46 46
     }
47 47
 
48
+    /**
49
+     * @param string $name
50
+     */
48 51
     public function addQuery($name, AbstractObjectType $query)
49 52
     {
50 53
         $this->getQueryType()->getConfig()->addField($name, $query);
Please login to merge, or discard this patch.
src/Processor.php 2 patches
Doc Comments   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -353,6 +353,9 @@  discard block
 block discarded – undo
353 353
         throw new \Exception(sprintf('Property "%s" not found in resolve result', $astField->getName()));
354 354
     }
355 355
 
356
+    /**
357
+     * @param string $path
358
+     */
356 359
     protected function getPropertyValue($data, $path)
357 360
     {
358 361
         if (is_object($data)) {
@@ -398,8 +401,8 @@  discard block
 block discarded – undo
398 401
     }
399 402
 
400 403
     /**
401
-     * @param $field     Field
402
-     * @param $query     Query
404
+     * @param Field $field     Field
405
+     * @param Query $query     Query
403 406
      *
404 407
      * @return array
405 408
      */
@@ -418,7 +421,7 @@  discard block
 block discarded – undo
418 421
     }
419 422
 
420 423
     /**
421
-     * @param $query         Query
424
+     * @param Query $query         Query
422 425
      * @param $queryType     ObjectType|TypeInterface|Field
423 426
      * @param $resolvedValue mixed
424 427
      * @param $value         array
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -81,7 +81,9 @@  discard block
 block discarded – undo
81 81
 
82 82
     public function processRequest($payload, $variables = [])
83 83
     {
84
-        if ($this->hasErrors()) return $this;
84
+        if ($this->hasErrors()) {
85
+            return $this;
86
+        }
85 87
 
86 88
         $this->data = [];
87 89
 
@@ -157,7 +159,9 @@  discard block
 block discarded – undo
157 159
      */
158 160
     protected function executeMutation(Mutation $mutation, $currentLevelSchema)
159 161
     {
160
-        if (!$currentLevelSchema) throw new ConfigurationException('There is no mutation ' . $mutation->getName());
162
+        if (!$currentLevelSchema) {
163
+            throw new ConfigurationException('There is no mutation ' . $mutation->getName());
164
+        }
161 165
 
162 166
         if (!$this->resolveValidator->checkFieldExist($currentLevelSchema, $mutation)) {
163 167
             return null;
Please login to merge, or discard this patch.
src/Type/Object/AbstractInterfaceType.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -55,6 +55,9 @@
 block discarded – undo
55 55
         return $this;
56 56
     }
57 57
 
58
+    /**
59
+     * @param AbstractObjectType $value
60
+     */
58 61
     public function isValidValue($value)
59 62
     {
60 63
         if ($value instanceof AbstractObjectType) {
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,9 @@
 block discarded – undo
59 59
     {
60 60
         if ($value instanceof AbstractObjectType) {
61 61
             foreach($value->getInterfaces() as $interface) {
62
-                if ($interface instanceof $this) return true;
62
+                if ($interface instanceof $this) {
63
+                    return true;
64
+                }
63 65
             }
64 66
         }
65 67
         return false;
Please login to merge, or discard this patch.
src/Validator/SchemaValidator/SchemaValidator.php 2 patches
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -51,7 +51,8 @@
 block discarded – undo
51 51
     /**
52 52
      * @param Field $intField
53 53
      * @param Field $objField
54
-     * @return bool
54
+     * @param \Youshido\GraphQL\Type\Object\AbstractInterfaceType $interface
55
+     * @return boolean|null
55 56
      * @throws ConfigurationException
56 57
      */
57 58
     protected function assertFieldsIdentical($intField, $objField, $interface)
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,9 @@
 block discarded – undo
39 39
 
40 40
     protected function assertInterfaceImplementationCorrect(AbstractObjectType $type)
41 41
     {
42
-        if (!$type->getInterfaces()) return true;
42
+        if (!$type->getInterfaces()) {
43
+            return true;
44
+        }
43 45
 
44 46
         foreach ($type->getInterfaces() as $interface) {
45 47
             foreach ($interface->getConfig()->getFields() as $intField) {
Please login to merge, or discard this patch.
src/Introspection/Traits/TypeCollectorTrait.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,9 @@
 block discarded – undo
27 27
         if (!$type) {
28 28
             return;
29 29
         }
30
-        if (is_object($type) && array_key_exists($type->getName(), $this->types)) return;
30
+        if (is_object($type) && array_key_exists($type->getName(), $this->types)) {
31
+            return;
32
+        }
31 33
 
32 34
         switch ($type->getKind()) {
33 35
             case TypeMap::KIND_INTERFACE:
Please login to merge, or discard this patch.