Youshido /
GraphQL
Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
Complex classes like Processor often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Processor, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 42 | class Processor |
||
| 43 | { |
||
| 44 | |||
| 45 | const TYPE_NAME_QUERY = '__typename'; |
||
| 46 | |||
| 47 | /** @var ExecutionContext */ |
||
| 48 | protected $executionContext; |
||
| 49 | |||
| 50 | /** @var ResolveValidatorInterface */ |
||
| 51 | protected $resolveValidator; |
||
| 52 | |||
| 53 | /** @var array */ |
||
| 54 | protected $data; |
||
| 55 | |||
| 56 | /** @var int */ |
||
| 57 | protected $maxComplexity; |
||
| 58 | |||
| 59 | /** @var array DeferredResult[] */ |
||
| 60 | protected $deferredResults = []; |
||
| 61 | |||
| 62 | 67 | public function __construct(AbstractSchema $schema) |
|
| 63 | { |
||
| 64 | 67 | if (empty($this->executionContext)) { |
|
| 65 | 67 | $this->executionContext = new ExecutionContext($schema); |
|
| 66 | 67 | $this->executionContext->setContainer(new Container()); |
|
| 67 | 67 | } |
|
| 68 | |||
| 69 | 67 | $this->resolveValidator = new ResolveValidator($this->executionContext); |
|
|
|
|||
| 70 | 67 | } |
|
| 71 | |||
| 72 | 65 | public function processPayload($payload, $variables = [], $reducers = []) |
|
| 73 | { |
||
| 74 | 65 | $this->data = []; |
|
| 75 | |||
| 76 | try { |
||
| 77 | 65 | $this->parseAndCreateRequest($payload, $variables); |
|
| 78 | |||
| 79 | 64 | if ($this->maxComplexity) { |
|
| 80 | 1 | $reducers[] = new MaxComplexityQueryVisitor($this->maxComplexity); |
|
| 81 | 1 | } |
|
| 82 | |||
| 83 | 64 | if ($reducers) { |
|
| 84 | 2 | $reducer = new Reducer(); |
|
| 85 | 2 | $reducer->reduceQuery($this->executionContext, $reducers); |
|
| 86 | 2 | } |
|
| 87 | |||
| 88 | 64 | foreach ($this->executionContext->getRequest()->getAllOperations() as $query) { |
|
| 89 | 64 | if ($operationResult = $this->resolveQuery($query)) { |
|
| 90 | 64 | $this->data = array_merge($this->data, $operationResult); |
|
| 91 | 64 | }; |
|
| 92 | 64 | } |
|
| 93 | |||
| 94 | // If the processor found any deferred results, resolve them now. |
||
| 95 | 64 | if (!empty($this->data) && $this->deferredResults) { |
|
| 96 | try { |
||
| 97 | 4 | while ($deferredResolver = array_shift($this->deferredResults)) { |
|
| 98 | 4 | $deferredResolver->resolve(); |
|
| 99 | 4 | } |
|
| 100 | 4 | } catch (\Exception $e) { |
|
| 101 | $this->executionContext->addError($e); |
||
| 102 | 4 | } finally { |
|
| 103 | 4 | $this->data = static::unpackDeferredResults($this->data); |
|
| 104 | } |
||
| 105 | 4 | } |
|
| 106 | |||
| 107 | 65 | } catch (\Exception $e) { |
|
| 108 | 5 | $this->executionContext->addError($e); |
|
| 109 | } |
||
| 110 | |||
| 111 | 65 | return $this; |
|
| 112 | } |
||
| 113 | |||
| 114 | /** |
||
| 115 | * Unpack results stored inside deferred resolvers. |
||
| 116 | * |
||
| 117 | * @param mixed $result |
||
| 118 | * The result ree. |
||
| 119 | * |
||
| 120 | * @return mixed |
||
| 121 | * The unpacked result. |
||
| 122 | */ |
||
| 123 | 4 | public static function unpackDeferredResults($result) |
|
| 124 | { |
||
| 125 | 4 | while ($result instanceof DeferredResult) { |
|
| 126 | 4 | $result = $result->result; |
|
| 127 | 4 | } |
|
| 128 | |||
| 129 | 4 | if (is_array($result)) { |
|
| 130 | 4 | foreach ($result as $key => $value) { |
|
| 131 | 4 | $result[$key] = static::unpackDeferredResults($value); |
|
| 132 | 4 | } |
|
| 133 | 4 | } |
|
| 134 | |||
| 135 | 4 | return $result; |
|
| 136 | } |
||
| 137 | |||
| 138 | 64 | protected function resolveQuery(AstQuery $query) |
|
| 156 | |||
| 157 | 64 | protected function resolveField(FieldInterface $field, AstFieldInterface $ast, $parentValue = null, $fromObject = false) |
|
| 216 | |||
| 217 | 64 | private function prepareAstArguments(FieldInterface $field, AstFieldInterface $query, Request $request) |
|
| 227 | |||
| 228 | 38 | private function prepareArgumentValue($argumentValue, AbstractType $argumentType, Request $request) |
|
| 229 | { |
||
| 230 | 38 | switch ($argumentType->getKind()) { |
|
| 231 | 38 | case TypeMap::KIND_LIST: |
|
| 232 | /** @var $argumentType AbstractListType */ |
||
| 233 | 12 | $result = []; |
|
| 234 | 12 | if ($argumentValue instanceof AstInputList || is_array($argumentValue)) { |
|
| 235 | 9 | $list = is_array($argumentValue) ? $argumentValue : $argumentValue->getValue(); |
|
| 236 | 9 | foreach ($list as $item) { |
|
| 237 | 9 | $result[] = $this->prepareArgumentValue($item, $argumentType->getItemType()->getNullableType(), $request); |
|
| 238 | 9 | } |
|
| 239 | 9 | } else { |
|
| 240 | 3 | if ($argumentValue instanceof VariableReference) { |
|
| 241 | 3 | return $this->getVariableReferenceArgumentValue($argumentValue, $argumentType, $request); |
|
| 242 | } |
||
| 243 | } |
||
| 244 | |||
| 245 | 9 | return $result; |
|
| 246 | |||
| 247 | 37 | case TypeMap::KIND_INPUT_OBJECT: |
|
| 248 | /** @var $argumentType AbstractInputObjectType */ |
||
| 249 | 6 | $result = []; |
|
| 250 | 6 | if ($argumentValue instanceof AstInputObject) { |
|
| 251 | 5 | foreach ($argumentType->getFields() as $field) { |
|
| 252 | /** @var $field Field */ |
||
| 253 | 5 | if ($field->getConfig()->has('defaultValue')) { |
|
| 254 | 1 | $result[$field->getName()] = $field->getType()->getNullableType()->parseInputValue($field->getConfig()->get('defaultValue')); |
|
| 255 | 1 | } |
|
| 256 | 5 | } |
|
| 257 | 5 | foreach ($argumentValue->getValue() as $key => $item) { |
|
| 258 | 5 | if ($argumentType->hasField($key)) { |
|
| 259 | 5 | $result[$key] = $this->prepareArgumentValue($item, $argumentType->getField($key)->getType()->getNullableType(), $request); |
|
| 260 | 5 | } else { |
|
| 261 | $result[$key] = $item; |
||
| 262 | } |
||
| 263 | 5 | } |
|
| 264 | 5 | } else { |
|
| 265 | 2 | if ($argumentValue instanceof VariableReference) { |
|
| 266 | return $this->getVariableReferenceArgumentValue($argumentValue, $argumentType, $request); |
||
| 267 | } else { |
||
| 268 | 2 | if (is_array($argumentValue)) { |
|
| 269 | 1 | return $argumentValue; |
|
| 270 | } |
||
| 271 | } |
||
| 272 | } |
||
| 273 | |||
| 274 | 6 | return $result; |
|
| 275 | |||
| 276 | 36 | case TypeMap::KIND_SCALAR: |
|
| 277 | 36 | case TypeMap::KIND_ENUM: |
|
| 278 | /** @var $argumentValue AstLiteral|VariableReference */ |
||
| 279 | 36 | if ($argumentValue instanceof VariableReference) { |
|
| 280 | 7 | return $this->getVariableReferenceArgumentValue($argumentValue, $argumentType, $request); |
|
| 281 | } else { |
||
| 282 | 31 | if ($argumentValue instanceof AstLiteral) { |
|
| 283 | 19 | return $argumentValue->getValue(); |
|
| 284 | } else { |
||
| 285 | 13 | return $argumentValue; |
|
| 286 | } |
||
| 287 | } |
||
| 288 | } |
||
| 289 | |||
| 290 | throw new ResolveException('Argument type not supported'); |
||
| 291 | } |
||
| 292 | |||
| 293 | 9 | private function getVariableReferenceArgumentValue(VariableReference $variableReference, AbstractType $argumentType, Request $request) |
|
| 294 | { |
||
| 295 | 9 | $variable = $variableReference->getVariable(); |
|
| 296 | 9 | if ($argumentType->getKind() === TypeMap::KIND_LIST) { |
|
| 297 | if ( |
||
| 298 | 3 | (!$variable->isArray() && !is_array($variable->getValue())) || |
|
| 299 | 3 | ($variable->getTypeName() !== $argumentType->getNamedType()->getNullableType()->getName()) || |
|
| 300 | 3 | ($argumentType->getNamedType()->getKind() === TypeMap::KIND_NON_NULL && $variable->isArrayElementNullable()) |
|
| 301 | 3 | ) { |
|
| 302 | 1 | throw new ResolveException(sprintf('Invalid variable "%s" type, allowed type is "%s"', $variable->getName(), $argumentType->getNamedType()->getNullableType()->getName()), $variable->getLocation()); |
|
| 303 | } |
||
| 304 | 3 | } else { |
|
| 305 | 7 | if ($variable->getTypeName() !== $argumentType->getName()) { |
|
| 306 | 1 | throw new ResolveException(sprintf('Invalid variable "%s" type, allowed type is "%s"', $variable->getName(), $argumentType->getName()), $variable->getLocation()); |
|
| 307 | } |
||
| 308 | } |
||
| 309 | |||
| 310 | 8 | $requestValue = $request->getVariable($variable->getName()); |
|
| 311 | 8 | if ((null === $requestValue && $variable->isNullable()) && !$request->hasVariable($variable->getName())) { |
|
| 312 | throw new ResolveException(sprintf('Variable "%s" does not exist in request', $variable->getName()), $variable->getLocation()); |
||
| 313 | } |
||
| 314 | |||
| 315 | 8 | return $requestValue; |
|
| 316 | } |
||
| 317 | |||
| 318 | |||
| 319 | /** |
||
| 320 | * @param FieldInterface $field |
||
| 321 | * @param AbstractObjectType $type |
||
| 322 | * @param AstFieldInterface $ast |
||
| 323 | * @param $resolvedValue |
||
| 324 | * @return array |
||
| 325 | */ |
||
| 326 | 36 | private function collectResult(FieldInterface $field, AbstractObjectType $type, $ast, $resolvedValue) |
|
| 327 | { |
||
| 328 | 36 | $result = []; |
|
| 329 | |||
| 330 | 36 | foreach ($ast->getFields() as $astField) { |
|
| 331 | 36 | switch (true) { |
|
| 332 | 36 | case $astField instanceof TypedFragmentReference: |
|
| 333 | 2 | $astName = $astField->getTypeName(); |
|
| 334 | 2 | $typeName = $type->getName(); |
|
| 335 | |||
| 336 | 2 | View Code Duplication | if ($typeName !== $astName) { |
| 337 | 2 | foreach ($type->getInterfaces() as $interface) { |
|
| 338 | 1 | if ($interface->getName() === $astName) { |
|
| 339 | $result = array_merge($result, $this->collectResult($field, $type, $astField, $resolvedValue)); |
||
| 340 | |||
| 341 | break; |
||
| 342 | } |
||
| 343 | 2 | } |
|
| 344 | |||
| 345 | 2 | continue; |
|
| 346 | } |
||
| 347 | |||
| 348 | 2 | $result = array_merge($result, $this->collectResult($field, $type, $astField, $resolvedValue)); |
|
| 349 | |||
| 350 | 2 | break; |
|
| 351 | |||
| 352 | 36 | case $astField instanceof FragmentReference: |
|
| 353 | 5 | $astFragment = $this->executionContext->getRequest()->getFragment($astField->getName()); |
|
| 354 | 5 | $astFragmentModel = $astFragment->getModel(); |
|
| 355 | 5 | $typeName = $type->getName(); |
|
| 356 | |||
| 357 | 5 | View Code Duplication | if ($typeName !== $astFragmentModel) { |
| 358 | 1 | foreach ($type->getInterfaces() as $interface) { |
|
| 359 | 1 | if ($interface->getName() === $astFragmentModel) { |
|
| 360 | 1 | $result = array_merge($result, $this->collectResult($field, $type, $astFragment, $resolvedValue)); |
|
| 361 | |||
| 362 | 1 | break; |
|
| 363 | } |
||
| 364 | |||
| 365 | 1 | } |
|
| 366 | |||
| 367 | 1 | continue; |
|
| 368 | } |
||
| 369 | |||
| 370 | 5 | $result = array_merge($result, $this->collectResult($field, $type, $astFragment, $resolvedValue)); |
|
| 371 | |||
| 372 | 5 | break; |
|
| 373 | |||
| 374 | 36 | default: |
|
| 375 | 36 | $result[$this->getAlias($astField)] = $this->resolveField($field, $astField, $resolvedValue, true); |
|
| 376 | 36 | } |
|
| 377 | 36 | } |
|
| 378 | |||
| 379 | 36 | return $result; |
|
| 380 | } |
||
| 381 | |||
| 382 | /** |
||
| 383 | * Apply post-process callbacks to all deferred resolvers. |
||
| 384 | */ |
||
| 385 | 58 | protected function deferredResolve($resolvedValue, callable $callback) { |
|
| 396 | |||
| 397 | 52 | protected function resolveScalar(FieldInterface $field, AstFieldInterface $ast, $parentValue) |
|
| 409 | |||
| 410 | 24 | protected function resolveList(FieldInterface $field, AstFieldInterface $ast, $parentValue) |
|
| 411 | { |
||
| 412 | /** @var AstQuery $ast */ |
||
| 413 | 24 | $resolvedValue = $this->doResolve($field, $ast, $parentValue); |
|
| 414 | |||
| 415 | return $this->deferredResolve($resolvedValue, function ($resolvedValue) use ($field, $ast, $parentValue) { |
||
| 416 | 24 | $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue); |
|
| 417 | |||
| 418 | 22 | if (null === $resolvedValue) { |
|
| 419 | 6 | return null; |
|
| 420 | } |
||
| 421 | |||
| 422 | /** @var AbstractListType $type */ |
||
| 423 | 21 | $type = $field->getType()->getNullableType(); |
|
| 424 | 21 | $itemType = $type->getNamedType(); |
|
| 425 | |||
| 426 | 21 | $fakeAst = clone $ast; |
|
| 427 | 21 | if ($fakeAst instanceof AstQuery) { |
|
| 428 | 20 | $fakeAst->setArguments([]); |
|
| 429 | 20 | } |
|
| 430 | |||
| 431 | 21 | $fakeField = new Field([ |
|
| 432 | 21 | 'name' => $field->getName(), |
|
| 433 | 21 | 'type' => $itemType, |
|
| 434 | 21 | 'args' => $field->getArguments(), |
|
| 435 | 21 | ]); |
|
| 436 | |||
| 437 | 21 | $result = []; |
|
| 438 | 21 | foreach ($resolvedValue as $resolvedValueItem) { |
|
| 439 | try { |
||
| 440 | $fakeField->getConfig()->set('resolve', function () use ($resolvedValueItem) { |
||
| 441 | 20 | return $resolvedValueItem; |
|
| 442 | 20 | }); |
|
| 443 | |||
| 444 | 20 | switch ($itemType->getNullableType()->getKind()) { |
|
| 445 | 20 | case TypeMap::KIND_ENUM: |
|
| 446 | 20 | case TypeMap::KIND_SCALAR: |
|
| 447 | 5 | $value = $this->resolveScalar($fakeField, $fakeAst, $resolvedValueItem); |
|
| 448 | |||
| 449 | 4 | break; |
|
| 450 | |||
| 451 | |||
| 452 | 16 | case TypeMap::KIND_OBJECT: |
|
| 453 | 14 | $value = $this->resolveObject($fakeField, $fakeAst, $resolvedValueItem); |
|
| 454 | |||
| 455 | 14 | break; |
|
| 456 | |||
| 457 | 3 | case TypeMap::KIND_UNION: |
|
| 458 | 3 | case TypeMap::KIND_INTERFACE: |
|
| 459 | 3 | $value = $this->resolveComposite($fakeField, $fakeAst, $resolvedValueItem); |
|
| 460 | |||
| 461 | 3 | break; |
|
| 462 | |||
| 463 | default: |
||
| 464 | $value = null; |
||
| 465 | 19 | } |
|
| 466 | 20 | } catch (\Exception $e) { |
|
| 467 | 1 | $this->executionContext->addError($e); |
|
| 468 | |||
| 469 | 1 | $value = null; |
|
| 470 | } |
||
| 471 | |||
| 472 | 20 | $result[] = $value; |
|
| 473 | 21 | } |
|
| 474 | |||
| 475 | 21 | return $result; |
|
| 476 | 24 | }); |
|
| 477 | } |
||
| 478 | |||
| 479 | 37 | protected function resolveObject(FieldInterface $field, AstFieldInterface $ast, $parentValue, $fromUnion = false) |
|
| 480 | { |
||
| 481 | 37 | $resolvedValue = $parentValue; |
|
| 482 | 37 | if (!$fromUnion) { |
|
| 483 | 32 | $resolvedValue = $this->doResolve($field, $ast, $parentValue); |
|
| 484 | 32 | } |
|
| 485 | |||
| 486 | return $this->deferredResolve($resolvedValue, function ($resolvedValue) use ($field, $ast, $parentValue) { |
||
| 487 | 37 | $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue); |
|
| 488 | |||
| 489 | 37 | if (null === $resolvedValue) { |
|
| 490 | 6 | return null; |
|
| 491 | } |
||
| 492 | /** @var AbstractObjectType $type */ |
||
| 493 | 36 | $type = $field->getType()->getNullableType(); |
|
| 494 | |||
| 495 | try { |
||
| 496 | 36 | return $this->collectResult($field, $type, $ast, $resolvedValue); |
|
| 497 | 4 | } catch (\Exception $e) { |
|
| 498 | 4 | return null; |
|
| 499 | } |
||
| 500 | 37 | }); |
|
| 501 | } |
||
| 502 | |||
| 503 | 7 | protected function resolveComposite(FieldInterface $field, AstFieldInterface $ast, $parentValue) |
|
| 504 | { |
||
| 505 | /** @var AstQuery $ast */ |
||
| 506 | 7 | $resolvedValue = $this->doResolve($field, $ast, $parentValue); |
|
| 507 | 7 | return $this->deferredResolve($resolvedValue, function ($resolvedValue) use ($field, $ast, $parentValue) { |
|
| 508 | 7 | $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue); |
|
| 509 | |||
| 510 | 7 | if (null === $resolvedValue) { |
|
| 511 | return null; |
||
| 512 | } |
||
| 513 | |||
| 514 | /** @var AbstractUnionType $type */ |
||
| 515 | 7 | $type = $field->getType()->getNullableType(); |
|
| 516 | 7 | $resolveInfo = new ResolveInfo( |
|
| 517 | 7 | $field, |
|
| 518 | 7 | $ast instanceof AstQuery ? $ast->getFields() : [], |
|
| 519 | 7 | $this->executionContext |
|
| 520 | 7 | ); |
|
| 521 | 7 | $resolvedType = $type->resolveType($resolvedValue, $resolveInfo); |
|
| 522 | |||
| 523 | 7 | if (!$resolvedType) { |
|
| 524 | throw new ResolveException('Resolving function must return type'); |
||
| 525 | } |
||
| 526 | |||
| 527 | 7 | if ($type instanceof AbstractInterfaceType) { |
|
| 528 | 6 | $this->resolveValidator->assertTypeImplementsInterface($resolvedType, $type); |
|
| 529 | 6 | } else { |
|
| 530 | 1 | $this->resolveValidator->assertTypeInUnionTypes($resolvedType, $type); |
|
| 531 | } |
||
| 532 | |||
| 533 | 7 | $fakeField = new Field([ |
|
| 534 | 7 | 'name' => $field->getName(), |
|
| 535 | 7 | 'type' => $resolvedType, |
|
| 536 | 7 | 'args' => $field->getArguments(), |
|
| 537 | 7 | ]); |
|
| 538 | |||
| 539 | 7 | return $this->resolveObject($fakeField, $ast, $resolvedValue, true); |
|
| 540 | 7 | }); |
|
| 541 | } |
||
| 542 | |||
| 543 | 65 | protected function parseAndCreateRequest($payload, $variables = []) |
|
| 556 | |||
| 557 | 58 | protected function doResolve(FieldInterface $field, AstFieldInterface $ast, $parentValue = null) |
|
| 565 | |||
| 566 | 58 | protected function parseArgumentsValues(FieldInterface $field, AstFieldInterface $ast) |
|
| 567 | { |
||
| 568 | 58 | $values = []; |
|
| 569 | 58 | $defaults = []; |
|
| 570 | |||
| 571 | 58 | foreach ($field->getArguments() as $argument) { |
|
| 572 | /** @var $argument InputField */ |
||
| 573 | 43 | if ($argument->getConfig()->has('defaultValue')) { |
|
| 574 | 7 | $defaults[$argument->getName()] = $argument->getConfig()->getDefaultValue(); |
|
| 575 | 7 | } |
|
| 576 | 58 | } |
|
| 577 | |||
| 578 | 58 | foreach ($ast->getArguments() as $astArgument) { |
|
| 579 | 34 | $argument = $field->getArgument($astArgument->getName()); |
|
| 580 | 34 | $argumentType = $argument->getType()->getNullableType(); |
|
| 581 | |||
| 582 | 34 | $values[$argument->getName()] = $argumentType->parseValue($astArgument->getValue()); |
|
| 583 | |||
| 584 | 34 | if (isset($defaults[$argument->getName()])) { |
|
| 585 | 3 | unset($defaults[$argument->getName()]); |
|
| 586 | 3 | } |
|
| 587 | 58 | } |
|
| 588 | |||
| 589 | 58 | return array_merge($values, $defaults); |
|
| 590 | } |
||
| 591 | |||
| 592 | 64 | private function getAlias(AstFieldInterface $ast) |
|
| 596 | |||
| 597 | 58 | protected function createResolveInfo(FieldInterface $field, array $astFields) |
|
| 601 | |||
| 602 | |||
| 603 | /** |
||
| 604 | * You can access ExecutionContext to check errors and inject dependencies |
||
| 605 | * |
||
| 606 | * @return ExecutionContext |
||
| 607 | */ |
||
| 608 | 11 | public function getExecutionContext() |
|
| 612 | |||
| 613 | 64 | public function getResponseData() |
|
| 614 | { |
||
| 615 | 64 | $result = []; |
|
| 616 | |||
| 617 | 64 | if (!empty($this->data)) { |
|
| 618 | 63 | $result['data'] = $this->data; |
|
| 619 | 63 | } |
|
| 620 | |||
| 621 | 64 | if ($this->executionContext->hasErrors()) { |
|
| 622 | 19 | $result['errors'] = $this->executionContext->getErrorsArray(); |
|
| 623 | 19 | } |
|
| 624 | |||
| 625 | 64 | return $result; |
|
| 626 | } |
||
| 627 | |||
| 628 | /** |
||
| 629 | * @return int |
||
| 630 | */ |
||
| 631 | public function getMaxComplexity() |
||
| 635 | |||
| 636 | /** |
||
| 637 | * @param int $maxComplexity |
||
| 638 | */ |
||
| 639 | 1 | public function setMaxComplexity($maxComplexity) |
|
| 643 | |||
| 644 | } |
||
| 645 |
Loading history...
This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.
If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.
In this case you can add the
@ignorePhpDoc annotation to the duplicate definition and it will be ignored.