Youshido / GraphQL

Tests/Parser/VariableTest.php

Spacing +6 added lines, -6 removed lines

@@ -14,7 +14,7 @@ 
      */
     public function testGetValue($actual, $expected)
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setValue($actual);
         $this->assertEquals($var->getValue(), $expected);
     }
@@ -25,13 +25,13 @@ 
      */
     public function testGetNullValueException()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->getValue();
     }
 
     public function testGetValueReturnsDefaultValueIfNoValueSet()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setDefaultValue('default-value');
 
         $this->assertEquals(
@@ -42,7 +42,7 @@ 
 
     public function testGetValueReturnsSetValueEvenWithDefaultValue()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setValue('real-value');
         $var->setDefaultValue('default-value');
 
@@ -54,7 +54,7 @@ 
 
     public function testIndicatesDefaultValuePresent()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setDefaultValue('default-value');
 
         $this->assertTrue(
@@ -64,7 +64,7 @@ 
 
     public function testHasNoDefaultValue()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
 
         $this->assertFalse(
             $var->hasDefaultValue()

src/Execution/Processor.php

Doc Comments +6 added lines

@@ -149,6 +149,9 @@ 
         return [$this->getAlias($query) => $value];
     }
 
+    /**
+     * @return AstFieldInterface
+     */
     protected function resolveField(FieldInterface $field, AstFieldInterface $ast, $parentValue = null, $fromObject = false)
     {
         try {
@@ -220,6 +223,9 @@ 
         }
     }
 
+    /**
+     * @param \Youshido\GraphQL\Parser\Ast\Interfaces\ValueInterface $argumentValue
+     */
     private function prepareArgumentValue($argumentValue, AbstractType $argumentType, Request $request)
     {
         switch ($argumentType->getKind()) {

Indentation +15 added lines, -15 removed lines

@@ -93,15 +93,15 @@ 
 
             // If the processor found any deferred results, resolve them now.
             if (!empty($this->data) && $this->deferredResults) {
-              try {
-                  while ($deferredResolver = array_shift($this->deferredResults)) {
-                      $deferredResolver->resolve();
-                  }
-              } catch (\Exception $e) {
-                  $this->executionContext->addError($e);
-              } finally {
-                  $this->data = static::unpackDeferredResults($this->data);
-              }
+                try {
+                    while ($deferredResolver = array_shift($this->deferredResults)) {
+                        $deferredResolver->resolve();
+                    }
+                } catch (\Exception $e) {
+                    $this->executionContext->addError($e);
+                } finally {
+                    $this->data = static::unpackDeferredResults($this->data);
+                }
             }
 
         } catch (\Exception $e) {
@@ -429,9 +429,9 @@ 
             }
 
             $fakeField = new Field([
-              'name' => $field->getName(),
-              'type' => $itemType,
-              'args' => $field->getArguments(),
+                'name' => $field->getName(),
+                'type' => $itemType,
+                'args' => $field->getArguments(),
             ]);
 
             $result = [];
@@ -526,9 +526,9 @@ 
             }
 
             $fakeField = new Field([
-              'name' => $field->getName(),
-              'type' => $resolvedType,
-              'args' => $field->getArguments(),
+                'name' => $field->getName(),
+                'type' => $resolvedType,
+                'args' => $field->getArguments(),
             ]);
 
             return $this->resolveObject($fakeField, $ast, $resolvedValue, true);

Spacing +5 added lines, -5 removed lines

@@ -412,7 +412,7 @@ 
         /** @var AstQuery $ast */
         $resolvedValue = $this->doResolve($field, $ast, $parentValue);
 
-        return $this->deferredResolve($resolvedValue, function ($resolvedValue) use ($field, $ast, $parentValue) {
+        return $this->deferredResolve($resolvedValue, function($resolvedValue) use ($field, $ast, $parentValue) {
             $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue);
 
             if (null === $resolvedValue) {
@@ -437,7 +437,7 @@ 
             $result = [];
             foreach ($resolvedValue as $resolvedValueItem) {
                 try {
-                    $fakeField->getConfig()->set('resolve', function () use ($resolvedValueItem) {
+                    $fakeField->getConfig()->set('resolve', function() use ($resolvedValueItem) {
                         return $resolvedValueItem;
                     });
 
@@ -483,7 +483,7 @@ 
             $resolvedValue = $this->doResolve($field, $ast, $parentValue);
         }
 
-        return $this->deferredResolve($resolvedValue, function ($resolvedValue) use ($field, $ast, $parentValue) {
+        return $this->deferredResolve($resolvedValue, function($resolvedValue) use ($field, $ast, $parentValue) {
             $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue);
 
             if (null === $resolvedValue) {
@@ -504,7 +504,7 @@ 
     {
         /** @var AstQuery $ast */
         $resolvedValue = $this->doResolve($field, $ast, $parentValue);
-        return $this->deferredResolve($resolvedValue, function ($resolvedValue) use ($field, $ast, $parentValue) {
+        return $this->deferredResolve($resolvedValue, function($resolvedValue) use ($field, $ast, $parentValue) {
             $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue);
 
             if (null === $resolvedValue) {
@@ -512,7 +512,7 @@ 
             }
 
             /** @var AbstractUnionType $type */
-            $type         = $field->getType()->getNullableType();
+            $type = $field->getType()->getNullableType();
             $resolveInfo = new ResolveInfo(
                 $field,
                 $ast instanceof AstQuery ? $ast->getFields() : [],

examples/04_bookstore/schema-bootstrap.php

Spacing +10 added lines, -10 removed lines

@@ -1,13 +1,13 @@
 <?php
 
-if (is_file(__DIR__ . '/../../../../../vendor/autoload.php')) {
-    require_once __DIR__ . '/../../../../../vendor/autoload.php';
+if (is_file(__DIR__.'/../../../../../vendor/autoload.php')) {
+    require_once __DIR__.'/../../../../../vendor/autoload.php';
 }
-require_once __DIR__ . '/../../vendor/autoload.php';
-require_once __DIR__ . '/Schema/Type/BookType.php';
-require_once __DIR__ . '/Schema/Type/AuthorType.php';
-require_once __DIR__ . '/Schema/Type/CategoryType.php';
-require_once __DIR__ . '/Schema/Field/Book/RecentBooksField.php';
-require_once __DIR__ . '/Schema/Field/CategoriesField.php';
-require_once __DIR__ . '/DataProvider.php';
-require_once __DIR__ . '/Schema/BookStoreSchema.php';
+require_once __DIR__.'/../../vendor/autoload.php';
+require_once __DIR__.'/Schema/Type/BookType.php';
+require_once __DIR__.'/Schema/Type/AuthorType.php';
+require_once __DIR__.'/Schema/Type/CategoryType.php';
+require_once __DIR__.'/Schema/Field/Book/RecentBooksField.php';
+require_once __DIR__.'/Schema/Field/CategoriesField.php';
+require_once __DIR__.'/DataProvider.php';
+require_once __DIR__.'/Schema/BookStoreSchema.php';

examples/04_bookstore/Schema/Field/CategoriesField.php

Spacing +3 added lines, -3 removed lines

@@ -24,12 +24,12 @@
 
     public function resolve($value, array $args, ResolveInfo $info)
     {
-        return new DeferredResolver(function () use ($value) {
-            $id = empty($value['id']) ? "1" : $value['id'] . ".1";
+        return new DeferredResolver(function() use ($value) {
+            $id = empty($value['id']) ? "1" : $value['id'].".1";
 
             return [
                 'id'       => $id,
-                'title'    => 'Category ' . $id,
+                'title'    => 'Category '.$id,
                 'authors'  => [
                     [
                         'id'        => 'author1',

examples/04_bookstore/Schema/BookStoreSchema.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         $config->getQuery()->addFields([
             'authors' => [
                 'type'    => new ListType(new AuthorType()),
-                'resolve' => function () {
+                'resolve' => function() {
                     return DataProvider::getAuthors();
                 }
             ],

Tests/Schema/DeferredTest.php

Indentation +95 added lines, -95 removed lines

@@ -67,7 +67,7 @@ 
     {
         if (!array_key_exists($resultId, $this->results)) {
             $query = array_unique(
-              array_reduce($this->buffer, 'array_merge', [])
+                array_reduce($this->buffer, 'array_merge', [])
             );
             sort($query);
             $result = $this->database->query($query);
@@ -101,43 +101,43 @@ 
     public function build($config)
     {
         $config->addField(
-          new Field(
+            new Field(
             [
-              'name' => 'name',
-              'type' => new StringType(),
-              'resolve' => function ($value) {
-                  return $value['name'];
-              },
+                'name' => 'name',
+                'type' => new StringType(),
+                'resolve' => function ($value) {
+                    return $value['name'];
+                },
             ]
-          )
+            )
         );
 
         $config->addField(
-          new Field(
+            new Field(
             [
-              'name' => 'friends',
-              'type' => new ListType(new DeferredUserType($this->database)),
-              'resolve' => function ($value) {
-                  return new DeferredResolver(
+                'name' => 'friends',
+                'type' => new ListType(new DeferredUserType($this->database)),
+                'resolve' => function ($value) {
+                    return new DeferredResolver(
                     $this->database->add($value['friends'])
-                  );
-              },
+                    );
+                },
             ]
-          )
+            )
         );
 
         $config->addField(
-          new Field(
+            new Field(
             [
-              'name' => 'foes',
-              'type' => new ListType(new DeferredUserType($this->database)),
-              'resolve' => function ($value) {
-                  return new DeferredResolver(
+                'name' => 'foes',
+                'type' => new ListType(new DeferredUserType($this->database)),
+                'resolve' => function ($value) {
+                    return new DeferredResolver(
                     $this->database->add($value['foes'])
-                  );
-              },
+                    );
+                },
             ]
-          )
+            )
         );
     }
 }
@@ -148,30 +148,30 @@ 
     public function __construct(DeferredQueryBuffer $buffer)
     {
         $usersField = new Field(
-          [
+            [
             'name' => 'users',
             'type' => new ListType(new DeferredUserType($buffer)),
             'resolve' => function ($value, $args) use ($buffer) {
                 return new DeferredResolver($buffer->add($args['ids']));
             },
-          ]
+            ]
         );
 
         $usersField->addArgument(
-          'ids',
-          [
+            'ids',
+            [
             'type' => new ListType(new StringType()),
-          ]
+            ]
         );
         parent::__construct(
-          [
+            [
             'query' => new ObjectType(
-              [
+                [
                 'name' => 'RootQuery',
                 'fields' => [$usersField],
-              ]
+                ]
             ),
-          ]
+            ]
         );
     }
 
@@ -216,28 +216,28 @@ 
         $database = $this->prophesize(DeferredDatabase::class);
 
         $database->query(['a', 'b'])->willReturn(
-          [
+            [
             'a' => ['id' => 'a', 'name' => 'User A'],
             'b' => ['id' => 'b', 'name' => 'User B'],
-          ]
+            ]
         )->shouldBeCalledTimes(1);
 
         $result = $this->query(
-          $query,
-          new DeferredQueryBuffer($database->reveal())
+            $query,
+            new DeferredQueryBuffer($database->reveal())
         );
 
         $database->checkProphecyMethodsPredictions();
 
         $this->assertEquals(
-          [
+            [
             'users' => [
-              ['name' => 'User A'],
-              ['name' => 'User B'],
+                ['name' => 'User A'],
+                ['name' => 'User B'],
+            ],
             ],
-          ],
-          $result['data'],
-          'Retrieved correct data.'
+            $result['data'],
+            'Retrieved correct data.'
         );
     }
 
@@ -259,30 +259,30 @@ 
         $database = $this->prophesize(DeferredDatabase::class);
 
         $database->query(['a', 'b'])->willReturn(
-          [
+            [
             'a' => ['id' => 'a', 'name' => 'User A'],
             'b' => ['id' => 'b', 'name' => 'User B'],
-          ]
+            ]
         )->shouldBeCalledTimes(1);
 
         $result = $this->query(
-          $query,
-          new DeferredQueryBuffer($database->reveal())
+            $query,
+            new DeferredQueryBuffer($database->reveal())
         );
 
         $database->checkProphecyMethodsPredictions();
 
         $this->assertEquals(
-          [
+            [
             'a' => [
-              ['name' => 'User A'],
+                ['name' => 'User A'],
             ],
             'b' => [
-              ['name' => 'User B'],
+                ['name' => 'User B'],
+            ],
             ],
-          ],
-          $result['data'],
-          'Retrieved correct data.'
+            $result['data'],
+            'Retrieved correct data.'
         );
 
     }
@@ -305,39 +305,39 @@ 
         $database = $this->prophesize(DeferredDatabase::class);
 
         $database->query(['a'])->willReturn(
-          [
+            [
             'a' => ['id' => 'a', 'name' => 'User A', 'friends' => ['b', 'c']],
-          ]
+            ]
         )->shouldBeCalledTimes(1);
 
         $database->query(['b', 'c'])->willReturn(
-          [
+            [
             'b' => ['id' => 'b', 'name' => 'User B'],
             'c' => ['id' => 'c', 'name' => 'User C'],
-          ]
+            ]
         );
 
         $result = $this->query(
-          $query,
-          new DeferredQueryBuffer($database->reveal())
+            $query,
+            new DeferredQueryBuffer($database->reveal())
         );
 
         $database->checkProphecyMethodsPredictions();
 
         $this->assertEquals(
-          [
+            [
             'a' => [
-              [
+                [
                 'name' => 'User A',
                 'friends' => [
-                  ['name' => 'User B'],
-                  ['name' => 'User C'],
+                    ['name' => 'User B'],
+                    ['name' => 'User C'],
+                ],
                 ],
-              ],
             ],
-          ],
-          $result['data'],
-          'Retrieved correct data.'
+            ],
+            $result['data'],
+            'Retrieved correct data.'
         );
     }
 
@@ -371,68 +371,68 @@ 
         $database = $this->prophesize(DeferredDatabase::class);
 
         $database->query(['a', 'b'])->willReturn(
-          [
+            [
             'a' => [
-              'id' => 'a',
-              'name' => 'User A',
-              'friends' => ['b', 'c'],
-              'foes' => ['d', 'e'],
+                'id' => 'a',
+                'name' => 'User A',
+                'friends' => ['b', 'c'],
+                'foes' => ['d', 'e'],
             ],
             'b' => [
-              'id' => 'b',
-              'name' => 'User B',
-              'friends' => ['a'],
-              'foes' => ['c'],
+                'id' => 'b',
+                'name' => 'User B',
+                'friends' => ['a'],
+                'foes' => ['c'],
             ],
-          ]
+            ]
         )->shouldBeCalledTimes(1);
 
         $database->query(['a', 'b', 'c', 'd', 'e'])->willReturn(
-          [
+            [
             'a' => ['id' => 'a', 'name' => 'User A'],
             'b' => ['id' => 'b', 'name' => 'User B'],
             'c' => ['id' => 'c', 'name' => 'User C'],
             'd' => ['id' => 'd', 'name' => 'User D'],
             'e' => ['id' => 'e', 'name' => 'User E'],
-          ]
+            ]
         )->shouldBeCalledTimes(1);
 
         $result = $this->query(
-          $query,
-          new DeferredQueryBuffer($database->reveal())
+            $query,
+            new DeferredQueryBuffer($database->reveal())
         );
 
         $database->checkProphecyMethodsPredictions();
 
         $this->assertEquals(
-          [
+            [
             'a' => [
-              [
+                [
                 'name' => 'User A',
                 'friends' => [
-                  ['name' => 'User B'],
-                  ['name' => 'User C'],
+                    ['name' => 'User B'],
+                    ['name' => 'User C'],
                 ],
                 'foes' => [
-                  ['name' => 'User D'],
-                  ['name' => 'User E'],
+                    ['name' => 'User D'],
+                    ['name' => 'User E'],
+                ],
                 ],
-              ],
             ],
             'b' => [
-              [
+                [
                 'name' => 'User B',
                 'friends' => [
-                  ['name' => 'User A'],
+                    ['name' => 'User A'],
                 ],
                 'foes' => [
-                  ['name' => 'User C'],
+                    ['name' => 'User C'],
+                ],
                 ],
-              ],
             ],
-          ],
-          $result['data'],
-          'Retrieved data is correct.'
+            ],
+            $result['data'],
+            'Retrieved data is correct.'
         );
     }
 }
\ No newline at end of file

Spacing +5 added lines, -5 removed lines

@@ -58,7 +58,7 @@ 
         $key = md5(serialize($ids));
         $this->buffer[$key] = $ids;
 
-        return function () use ($key) {
+        return function() use ($key) {
             return $this->fetch($key);
         };
     }
@@ -105,7 +105,7 @@ 
             [
               'name' => 'name',
               'type' => new StringType(),
-              'resolve' => function ($value) {
+              'resolve' => function($value) {
                   return $value['name'];
               },
             ]
@@ -117,7 +117,7 @@ 
             [
               'name' => 'friends',
               'type' => new ListType(new DeferredUserType($this->database)),
-              'resolve' => function ($value) {
+              'resolve' => function($value) {
                   return new DeferredResolver(
                     $this->database->add($value['friends'])
                   );
@@ -131,7 +131,7 @@ 
             [
               'name' => 'foes',
               'type' => new ListType(new DeferredUserType($this->database)),
-              'resolve' => function ($value) {
+              'resolve' => function($value) {
                   return new DeferredResolver(
                     $this->database->add($value['foes'])
                   );
@@ -151,7 +151,7 @@ 
           [
             'name' => 'users',
             'type' => new ListType(new DeferredUserType($buffer)),
-            'resolve' => function ($value, $args) use ($buffer) {
+            'resolve' => function($value, $args) use ($buffer) {
                 return new DeferredResolver($buffer->add($args['ids']));
             },
           ]

src/Execution/DeferredResolver.php

Indentation +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
     }
 
     public function resolve() {
-      return call_user_func($this->resolver);
+        return call_user_func($this->resolver);
     }
 }
\ No newline at end of file

src/Execution/DeferredResolverInterface.php

Indentation +5 added lines, -5 removed lines

@@ -16,10 +16,10 @@
  */
 interface DeferredResolverInterface {
 
-  /**
-   * @return mixed
-   *   The actual result value.
-   */
-  public function resolve();
+    /**
+     * @return mixed
+     *   The actual result value.
+     */
+    public function resolve();
 
 }

Tests/Library/Field/InputFieldTest.php

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 
     public function testFieldWithInputFieldArgument()
     {
-        $schema    = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [