@@ -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 |