@@ -1,7 +1,7 @@ |
||
| 1 | 1 | <?php |
| 2 | 2 | |
| 3 | -require_once __DIR__ . '/../../vendor/autoload.php'; |
|
| 4 | -require_once __DIR__ . '/Schema/FactionType.php'; |
|
| 5 | -require_once __DIR__ . '/Schema/ShipType.php'; |
|
| 6 | -require_once __DIR__ . '/Schema/TestDataProvider.php'; |
|
| 7 | -require_once __DIR__ . '/Schema/StarWarsRelaySchema.php'; |
|
| 3 | +require_once __DIR__.'/../../vendor/autoload.php'; |
|
| 4 | +require_once __DIR__.'/Schema/FactionType.php'; |
|
| 5 | +require_once __DIR__.'/Schema/ShipType.php'; |
|
| 6 | +require_once __DIR__.'/Schema/TestDataProvider.php'; |
|
| 7 | +require_once __DIR__.'/Schema/StarWarsRelaySchema.php'; |
|
@@ -41,7 +41,7 @@ |
||
| 41 | 41 | public function getValue() |
| 42 | 42 | { |
| 43 | 43 | if (!$this->value) { |
| 44 | - throw new \LogicException('Value is not set for variable "' . $this->name . '"'); |
|
| 44 | + throw new \LogicException('Value is not set for variable "'.$this->name.'"'); |
|
| 45 | 45 | } |
| 46 | 46 | |
| 47 | 47 | return $this->value; |
@@ -30,21 +30,21 @@ |
||
| 30 | 30 | ], |
| 31 | 31 | ])) |
| 32 | 32 | ->addField('location', [ |
| 33 | - 'type' => new ObjectType( |
|
| 34 | - [ |
|
| 35 | - 'name' => 'Location', |
|
| 36 | - 'fields' => [ |
|
| 37 | - 'address' => new StringType() |
|
| 38 | - ] |
|
| 39 | - ] |
|
| 40 | - ), |
|
| 41 | - 'args' => [ |
|
| 42 | - 'noop' => new IntType() |
|
| 43 | - ], |
|
| 44 | - 'resolve' => function ($value, $args, $info) { |
|
| 45 | - return ['address' => '1234 Street']; |
|
| 46 | - } |
|
| 47 | - ] |
|
| 33 | + 'type' => new ObjectType( |
|
| 34 | + [ |
|
| 35 | + 'name' => 'Location', |
|
| 36 | + 'fields' => [ |
|
| 37 | + 'address' => new StringType() |
|
| 38 | + ] |
|
| 39 | + ] |
|
| 40 | + ), |
|
| 41 | + 'args' => [ |
|
| 42 | + 'noop' => new IntType() |
|
| 43 | + ], |
|
| 44 | + 'resolve' => function ($value, $args, $info) { |
|
| 45 | + return ['address' => '1234 Street']; |
|
| 46 | + } |
|
| 47 | + ] |
|
| 48 | 48 | ) |
| 49 | 49 | ->addField( |
| 50 | 50 | 'echo', [ |
@@ -41,7 +41,7 @@ discard block |
||
| 41 | 41 | 'args' => [ |
| 42 | 42 | 'noop' => new IntType() |
| 43 | 43 | ], |
| 44 | - 'resolve' => function ($value, $args, $info) { |
|
| 44 | + 'resolve' => function($value, $args, $info) { |
|
| 45 | 45 | return ['address' => '1234 Street']; |
| 46 | 46 | } |
| 47 | 47 | ] |
@@ -52,7 +52,7 @@ discard block |
||
| 52 | 52 | 'args' => [ |
| 53 | 53 | 'value' => new NonNullType(new StringType()) |
| 54 | 54 | ], |
| 55 | - 'resolve' => function ($value, $args, $info) { |
|
| 55 | + 'resolve' => function($value, $args, $info) { |
|
| 56 | 56 | return $args['value']; |
| 57 | 57 | } |
| 58 | 58 | ] |
@@ -61,6 +61,6 @@ |
||
| 61 | 61 | $arrayData = (new TestObjectType())->getData(); |
| 62 | 62 | |
| 63 | 63 | $this->assertEquals('John', TypeService::getPropertyValue($arrayData, 'name')); |
| 64 | - $this->assertEquals('John', TypeService::getPropertyValue((object) $arrayData, 'name')); |
|
| 64 | + $this->assertEquals('John', TypeService::getPropertyValue((object)$arrayData, 'name')); |
|
| 65 | 65 | } |
| 66 | 66 | } |
@@ -24,7 +24,7 @@ |
||
| 24 | 24 | } |
| 25 | 25 | $this->memo += $cost ?: $this->defaultScore; |
| 26 | 26 | if ($this->memo > $this->maxScore) { |
| 27 | - throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore); |
|
| 27 | + throw new \Exception('query exceeded max allowed complexity of '.$this->maxScore); |
|
| 28 | 28 | } |
| 29 | 29 | return $this->memo; |
| 30 | 30 | } |
@@ -16,40 +16,40 @@ |
||
| 16 | 16 | |
| 17 | 17 | class MaxComplexityQueryVisitor extends AbstractQueryVisitor { |
| 18 | 18 | |
| 19 | - /** |
|
| 20 | - * @var int max score allowed before throwing an exception (causing processing to stop) |
|
| 21 | - */ |
|
| 22 | - public $maxScore; |
|
| 23 | - |
|
| 24 | - /** |
|
| 25 | - * @var int default score for nodes without explicit cost functions |
|
| 26 | - */ |
|
| 27 | - protected $defaultScore = 1; |
|
| 28 | - |
|
| 29 | - /** |
|
| 30 | - * MaxComplexityQueryVisitor constructor. |
|
| 31 | - * |
|
| 32 | - * @param int $max max allowed complexity score |
|
| 33 | - */ |
|
| 34 | - public function __construct($max) { |
|
| 19 | + /** |
|
| 20 | + * @var int max score allowed before throwing an exception (causing processing to stop) |
|
| 21 | + */ |
|
| 22 | + public $maxScore; |
|
| 23 | + |
|
| 24 | + /** |
|
| 25 | + * @var int default score for nodes without explicit cost functions |
|
| 26 | + */ |
|
| 27 | + protected $defaultScore = 1; |
|
| 28 | + |
|
| 29 | + /** |
|
| 30 | + * MaxComplexityQueryVisitor constructor. |
|
| 31 | + * |
|
| 32 | + * @param int $max max allowed complexity score |
|
| 33 | + */ |
|
| 34 | + public function __construct($max) { |
|
| 35 | 35 | parent::__construct(); |
| 36 | 36 | |
| 37 | 37 | $this->maxScore = $max; |
| 38 | - } |
|
| 38 | + } |
|
| 39 | 39 | |
| 40 | - /** |
|
| 41 | - * {@inheritdoc} |
|
| 42 | - */ |
|
| 43 | - public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) { |
|
| 40 | + /** |
|
| 41 | + * {@inheritdoc} |
|
| 42 | + */ |
|
| 43 | + public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) { |
|
| 44 | 44 | $cost = $fieldConfig->get('cost'); |
| 45 | 45 | if (is_callable($cost)) { |
| 46 | - $cost = $cost($args, $fieldConfig, $childScore); |
|
| 46 | + $cost = $cost($args, $fieldConfig, $childScore); |
|
| 47 | 47 | } |
| 48 | 48 | $cost = $cost ?: $this->defaultScore; |
| 49 | 49 | $this->memo += $cost; |
| 50 | 50 | if ($this->memo > $this->maxScore) { |
| 51 | - throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore); |
|
| 51 | + throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore); |
|
| 52 | 52 | } |
| 53 | 53 | return $cost; |
| 54 | - } |
|
| 54 | + } |
|
| 55 | 55 | } |
| 56 | 56 | \ No newline at end of file |
@@ -35,13 +35,13 @@ |
||
| 35 | 35 | if (empty(self::$objectsHash[$type])) { |
| 36 | 36 | $name = ucfirst($type); |
| 37 | 37 | |
| 38 | - $className = 'Youshido\GraphQL\Type\Scalar\\' . $name . 'Type'; |
|
| 38 | + $className = 'Youshido\GraphQL\Type\Scalar\\'.$name.'Type'; |
|
| 39 | 39 | self::$objectsHash[$type] = new $className(); |
| 40 | 40 | } |
| 41 | 41 | |
| 42 | 42 | return self::$objectsHash[$type]; |
| 43 | 43 | } else { |
| 44 | - throw new ConfigurationException('Configuration problem with type ' . $type); |
|
| 44 | + throw new ConfigurationException('Configuration problem with type '.$type); |
|
| 45 | 45 | } |
| 46 | 46 | } |
| 47 | 47 | |
@@ -27,13 +27,13 @@ |
||
| 27 | 27 | 'title' => new StringType(), |
| 28 | 28 | ] |
| 29 | 29 | ]); |
| 30 | - $processor = new Processor(new Schema([ |
|
| 30 | + $processor = new Processor(new Schema([ |
|
| 31 | 31 | 'query' => new ObjectType([ |
| 32 | 32 | 'name' => 'RootQueryType', |
| 33 | 33 | 'fields' => [ |
| 34 | 34 | 'latestReport' => [ |
| 35 | 35 | 'type' => $reportType, |
| 36 | - 'resolve' => function () { |
|
| 36 | + 'resolve' => function() { |
|
| 37 | 37 | return [ |
| 38 | 38 | 'title' => 'Accident #1', |
| 39 | 39 | 'time' => '13:30:12', |
@@ -168,7 +168,9 @@ |
||
| 168 | 168 | $value = []; |
| 169 | 169 | |
| 170 | 170 | if ($fieldType->getKind() == TypeMap::KIND_LIST) { |
| 171 | - if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) return null; |
|
| 171 | + if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) { |
|
| 172 | + return null; |
|
| 173 | + } |
|
| 172 | 174 | foreach ($resolvedValue as $resolvedValueItem) { |
| 173 | 175 | $value[] = []; |
| 174 | 176 | $index = count($value) - 1; |
@@ -111,7 +111,7 @@ discard block |
||
| 111 | 111 | } |
| 112 | 112 | |
| 113 | 113 | /** |
| 114 | - * @param Query|Field $query |
|
| 114 | + * @param Query $query |
|
| 115 | 115 | * @param AbstractObjectType $currentLevelSchema |
| 116 | 116 | * @return array|bool|mixed |
| 117 | 117 | */ |
@@ -260,6 +260,9 @@ discard block |
||
| 260 | 260 | } |
| 261 | 261 | } |
| 262 | 262 | |
| 263 | + /** |
|
| 264 | + * @param AbstractField $field |
|
| 265 | + */ |
|
| 263 | 266 | protected function createResolveInfo($field, $fields) { |
| 264 | 267 | return new ResolveInfo($field, $fields, $this->executionContext); |
| 265 | 268 | } |
@@ -322,7 +325,7 @@ discard block |
||
| 322 | 325 | } |
| 323 | 326 | |
| 324 | 327 | /** |
| 325 | - * @param $query Query|FragmentInterface |
|
| 328 | + * @param Query $query Query|FragmentInterface |
|
| 326 | 329 | * @param $queryType AbstractObjectType|TypeInterface|Field|AbstractType |
| 327 | 330 | * @param $resolvedValue mixed |
| 328 | 331 | * @param $value array |
@@ -494,7 +497,7 @@ discard block |
||
| 494 | 497 | * Fragments (anonymous and named), and Fields. The core of the function is simple: recurse until we hit the base |
| 495 | 498 | * case of a Field and yield that back up to the visitor up in `doVisit`. |
| 496 | 499 | * |
| 497 | - * @param Query|Field|FragmentInterface $queryNode |
|
| 500 | + * @param Query $queryNode |
|
| 498 | 501 | * @param AbstractField $currentLevelAST |
| 499 | 502 | * |
| 500 | 503 | * @return \Generator |
@@ -29,38 +29,38 @@ |
||
| 29 | 29 | |
| 30 | 30 | abstract class AbstractQueryVisitor { |
| 31 | 31 | |
| 32 | - /** |
|
| 33 | - * @var int initial value of $this->memo |
|
| 34 | - */ |
|
| 35 | - protected $initialValue = 0; |
|
| 32 | + /** |
|
| 33 | + * @var int initial value of $this->memo |
|
| 34 | + */ |
|
| 35 | + protected $initialValue = 0; |
|
| 36 | 36 | |
| 37 | - /** |
|
| 38 | - * @var mixed the accumulator |
|
| 39 | - */ |
|
| 40 | - protected $memo; |
|
| 37 | + /** |
|
| 38 | + * @var mixed the accumulator |
|
| 39 | + */ |
|
| 40 | + protected $memo; |
|
| 41 | 41 | |
| 42 | - /** |
|
| 43 | - * AbstractQueryVisitor constructor. |
|
| 44 | - */ |
|
| 45 | - public function __construct() { |
|
| 42 | + /** |
|
| 43 | + * AbstractQueryVisitor constructor. |
|
| 44 | + */ |
|
| 45 | + public function __construct() { |
|
| 46 | 46 | $this->memo = $this->initialValue; |
| 47 | - } |
|
| 47 | + } |
|
| 48 | 48 | |
| 49 | - /** |
|
| 50 | - * @return mixed getter for the memo, in case callers want to inspect it after a process run |
|
| 51 | - */ |
|
| 52 | - public function getMemo() { |
|
| 49 | + /** |
|
| 50 | + * @return mixed getter for the memo, in case callers want to inspect it after a process run |
|
| 51 | + */ |
|
| 52 | + public function getMemo() { |
|
| 53 | 53 | return $this->memo; |
| 54 | - } |
|
| 54 | + } |
|
| 55 | 55 | |
| 56 | - /** |
|
| 57 | - * Visit a query node. See class docstring. |
|
| 58 | - * |
|
| 59 | - * @param array $args |
|
| 60 | - * @param FieldConfig $fieldConfig |
|
| 61 | - * @param int $childScore |
|
| 62 | - * |
|
| 63 | - * @return int|null |
|
| 64 | - */ |
|
| 65 | - abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0); |
|
| 56 | + /** |
|
| 57 | + * Visit a query node. See class docstring. |
|
| 58 | + * |
|
| 59 | + * @param array $args |
|
| 60 | + * @param FieldConfig $fieldConfig |
|
| 61 | + * @param int $childScore |
|
| 62 | + * |
|
| 63 | + * @return int|null |
|
| 64 | + */ |
|
| 65 | + abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0); |
|
| 66 | 66 | } |
| 67 | 67 | \ No newline at end of file |