Completed
Push — master ( 29ef50...718f9d )
by Alexandr
03:01
created
src/Execution/Processor.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,6 @@
 block discarded – undo
24 24
 use Youshido\GraphQL\Parser\Ast\Interfaces\FieldInterface as AstFieldInterface;
25 25
 use Youshido\GraphQL\Parser\Ast\Mutation as AstMutation;
26 26
 use Youshido\GraphQL\Parser\Ast\Query as AstQuery;
27
-use Youshido\GraphQL\Parser\Ast\Query;
28 27
 use Youshido\GraphQL\Parser\Ast\TypedFragmentReference;
29 28
 use Youshido\GraphQL\Parser\Parser;
30 29
 use Youshido\GraphQL\Schema\AbstractSchema;
Please login to merge, or discard this patch.
src/Execution/Request.php 4 patches
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -223,6 +223,9 @@  discard block
 block discarded – undo
223 223
         return $this;
224 224
     }
225 225
 
226
+    /**
227
+     * @param string $name
228
+     */
226 229
     public function getVariable($name)
227 230
     {
228 231
         return $this->hasVariable($name) ? $this->variables[$name] : null;
@@ -266,7 +269,7 @@  discard block
 block discarded – undo
266 269
     }
267 270
 
268 271
     /**
269
-     * @return array|VariableReference[]
272
+     * @return VariableReference[]
270 273
      */
271 274
     public function getVariableReferences()
272 275
     {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -8,7 +8,6 @@
 block discarded – undo
8 8
 namespace Youshido\GraphQL\Execution;
9 9
 
10 10
 
11
-use Youshido\GraphQL\Parser\Ast\ArgumentValue\Literal;
12 11
 use Youshido\GraphQL\Parser\Ast\ArgumentValue\Variable;
13 12
 use Youshido\GraphQL\Parser\Ast\ArgumentValue\VariableReference;
14 13
 use Youshido\GraphQL\Parser\Ast\Fragment;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -208,7 +208,7 @@
 block discarded – undo
208 208
         }
209 209
 
210 210
         $this->variables = $variables;
211
-        foreach($this->variableReferences as $reference) {
211
+        foreach ($this->variableReferences as $reference) {
212 212
             /** invalid request with no variable */
213 213
             if (!$reference->getVariable()) continue;
214 214
             $variableName = $reference->getVariable()->getName();
Please login to merge, or discard this patch.
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -210,11 +210,15 @@
 block discarded – undo
210 210
         $this->variables = $variables;
211 211
         foreach($this->variableReferences as $reference) {
212 212
             /** invalid request with no variable */
213
-            if (!$reference->getVariable()) continue;
213
+            if (!$reference->getVariable()) {
214
+                continue;
215
+            }
214 216
             $variableName = $reference->getVariable()->getName();
215 217
 
216 218
             /** no variable was set at the time */
217
-            if (!array_key_exists($variableName, $variables)) continue;
219
+            if (!array_key_exists($variableName, $variables)) {
220
+                continue;
221
+            }
218 222
 
219 223
             $reference->getVariable()->setValue($variables[$variableName]);
220 224
             $reference->setValue($variables[$variableName]);
Please login to merge, or discard this patch.
Tests/Issues/Issue109/Issue109Schema.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
                                 ]
41 41
                             ]
42 42
                         ]),
43
-                        'resolve' => function ($source, array $args, ResolveInfo $info) {
43
+                        'resolve' => function($source, array $args, ResolveInfo $info) {
44 44
                             $internalArgs = [
45 45
                                 'comment_id' => 200
46 46
                             ];
Please login to merge, or discard this patch.