@@ -110,7 +110,7 @@ discard block |
||
| 110 | 110 | } |
| 111 | 111 | |
| 112 | 112 | /** |
| 113 | - * @param Query|Field $query |
|
| 113 | + * @param Query $query |
|
| 114 | 114 | * @param AbstractObjectType $currentLevelSchema |
| 115 | 115 | * @return array|bool|mixed |
| 116 | 116 | */ |
@@ -317,7 +317,7 @@ discard block |
||
| 317 | 317 | } |
| 318 | 318 | |
| 319 | 319 | /** |
| 320 | - * @param $query Query|FragmentInterface |
|
| 320 | + * @param Query $query Query|FragmentInterface |
|
| 321 | 321 | * @param $queryType AbstractObjectType|TypeInterface|Field|AbstractType |
| 322 | 322 | * @param $resolvedValue mixed |
| 323 | 323 | * @param $value array |
@@ -497,7 +497,7 @@ discard block |
||
| 497 | 497 | * Fragments (anonymous and named), and Fields. The core of the function is simple: recurse until we hit the base |
| 498 | 498 | * case of a Field and yield that back up to the visitor up in `doVisit`. |
| 499 | 499 | * |
| 500 | - * @param Query|Field|FragmentInterface $queryNode |
|
| 500 | + * @param Query $queryNode |
|
| 501 | 501 | * @param AbstractField $currentLevelAST |
| 502 | 502 | * |
| 503 | 503 | * @return \Generator |
@@ -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 |
@@ -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 |