@@ -53,7 +53,7 @@ discard block |
||
| 53 | 53 | $validator = ConfigValidator::getInstance(); |
| 54 | 54 | |
| 55 | 55 | if (!$validator->validate($this->data, $this->getContextRules(), $this->extraFieldsAllowed)) { |
| 56 | - throw new ConfigurationException('Config is not valid for ' . ($contextObject ? get_class($contextObject) : null) . "\n" . implode("\n", $validator->getErrorsArray(false))); |
|
| 56 | + throw new ConfigurationException('Config is not valid for '.($contextObject ? get_class($contextObject) : null)."\n".implode("\n", $validator->getErrorsArray(false))); |
|
| 57 | 57 | } |
| 58 | 58 | |
| 59 | 59 | $this->build(); |
@@ -126,7 +126,7 @@ discard block |
||
| 126 | 126 | } elseif (substr($method, 0, 2) == 'is') { |
| 127 | 127 | $propertyName = lcfirst(substr($method, 2)); |
| 128 | 128 | } else { |
| 129 | - throw new \Exception('Call to undefined method ' . $method); |
|
| 129 | + throw new \Exception('Call to undefined method '.$method); |
|
| 130 | 130 | } |
| 131 | 131 | |
| 132 | 132 | return $this->get($propertyName); |
@@ -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,34 +16,34 @@ |
||
| 16 | 16 | |
| 17 | 17 | class MaxComplexityQueryVisitor extends AbstractQueryVisitor { |
| 18 | 18 | |
| 19 | - public $maxScore; |
|
| 19 | + public $maxScore; |
|
| 20 | 20 | |
| 21 | - protected $defaultScore = 1; |
|
| 21 | + protected $defaultScore = 1; |
|
| 22 | 22 | |
| 23 | - public function __construct($max) { |
|
| 23 | + public function __construct($max) { |
|
| 24 | 24 | parent::__construct(); |
| 25 | 25 | |
| 26 | 26 | $this->maxScore = $max; |
| 27 | - } |
|
| 28 | - |
|
| 29 | - /** |
|
| 30 | - * @param array $args |
|
| 31 | - * @param FieldConfig $fieldConfig |
|
| 32 | - * @param int $childScore |
|
| 33 | - * |
|
| 34 | - * @return int|null |
|
| 35 | - * @throws \Exception |
|
| 36 | - */ |
|
| 37 | - public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) { |
|
| 27 | + } |
|
| 28 | + |
|
| 29 | + /** |
|
| 30 | + * @param array $args |
|
| 31 | + * @param FieldConfig $fieldConfig |
|
| 32 | + * @param int $childScore |
|
| 33 | + * |
|
| 34 | + * @return int|null |
|
| 35 | + * @throws \Exception |
|
| 36 | + */ |
|
| 37 | + public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) { |
|
| 38 | 38 | $cost = $fieldConfig->get('cost'); |
| 39 | 39 | if (is_callable($cost)) { |
| 40 | - $cost = $cost($args, $fieldConfig, $childScore); |
|
| 40 | + $cost = $cost($args, $fieldConfig, $childScore); |
|
| 41 | 41 | } |
| 42 | 42 | $cost = $cost ?: $this->defaultScore; |
| 43 | 43 | $this->memo += $cost; |
| 44 | 44 | if ($this->memo > $this->maxScore) { |
| 45 | - throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore); |
|
| 45 | + throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore); |
|
| 46 | 46 | } |
| 47 | 47 | return $cost; |
| 48 | - } |
|
| 48 | + } |
|
| 49 | 49 | } |
| 50 | 50 | \ No newline at end of file |
@@ -109,7 +109,7 @@ discard block |
||
| 109 | 109 | } |
| 110 | 110 | |
| 111 | 111 | /** |
| 112 | - * @param Query|Field $query |
|
| 112 | + * @param Query $query |
|
| 113 | 113 | * @param AbstractObjectType $currentLevelSchema |
| 114 | 114 | * @return array|bool|mixed |
| 115 | 115 | */ |
@@ -312,7 +312,7 @@ discard block |
||
| 312 | 312 | } |
| 313 | 313 | |
| 314 | 314 | /** |
| 315 | - * @param $query Query|FragmentInterface |
|
| 315 | + * @param Query $query Query|FragmentInterface |
|
| 316 | 316 | * @param $queryType AbstractObjectType|TypeInterface|Field|AbstractType |
| 317 | 317 | * @param $resolvedValue mixed |
| 318 | 318 | * @param $value array |
@@ -415,6 +415,9 @@ discard block |
||
| 415 | 415 | return $result; |
| 416 | 416 | } |
| 417 | 417 | |
| 418 | + /** |
|
| 419 | + * @param integer $max |
|
| 420 | + */ |
|
| 418 | 421 | public function setMaxComplexity($max) { |
| 419 | 422 | $this->maxComplexity = $max; |
| 420 | 423 | } |
@@ -470,7 +473,7 @@ discard block |
||
| 470 | 473 | * |
| 471 | 474 | * childScore costs are accumulated via values sent into the coroutine. |
| 472 | 475 | * |
| 473 | - * @param Query|Field|FragmentInterface $queryNode |
|
| 476 | + * @param Query $queryNode |
|
| 474 | 477 | * @param AbstractField $currentLevelAST |
| 475 | 478 | * |
| 476 | 479 | * @return \Generator |
@@ -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; |
@@ -13,24 +13,24 @@ |
||
| 13 | 13 | |
| 14 | 14 | abstract class AbstractQueryVisitor { |
| 15 | 15 | |
| 16 | - protected $initialValue = 0; |
|
| 16 | + protected $initialValue = 0; |
|
| 17 | 17 | |
| 18 | - protected $memo; |
|
| 18 | + protected $memo; |
|
| 19 | 19 | |
| 20 | - public function __construct() { |
|
| 20 | + public function __construct() { |
|
| 21 | 21 | $this->memo = $this->initialValue; |
| 22 | - } |
|
| 22 | + } |
|
| 23 | 23 | |
| 24 | - public function getMemo() { |
|
| 24 | + public function getMemo() { |
|
| 25 | 25 | return $this->memo; |
| 26 | - } |
|
| 27 | - |
|
| 28 | - /** |
|
| 29 | - * @param array $args |
|
| 30 | - * @param FieldConfig $fieldConfig |
|
| 31 | - * @param int $childScore |
|
| 32 | - * |
|
| 33 | - * @return int|null |
|
| 34 | - */ |
|
| 35 | - abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0); |
|
| 26 | + } |
|
| 27 | + |
|
| 28 | + /** |
|
| 29 | + * @param array $args |
|
| 30 | + * @param FieldConfig $fieldConfig |
|
| 31 | + * @param int $childScore |
|
| 32 | + * |
|
| 33 | + * @return int|null |
|
| 34 | + */ |
|
| 35 | + abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0); |
|
| 36 | 36 | } |
| 37 | 37 | \ 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', |