Skip to content

Commit 5f01d45

Browse files
authored
Merge pull request #1 from pamil/argument-organiser-cleanup
Clean up VariadicArgumentOrganiser
2 parents 6b6cc43 + 0efc8d5 commit 5f01d45

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

src/Argument/VariadicArgumentOrganiser.php

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,13 +42,10 @@ public function organiseArguments(ReflectionFunctionAbstract $function, array $a
4242
{
4343
$organisedArguments = $this->decoratedArgumentOrganiser->organiseArguments($function, $arguments);
4444

45-
$parameters = $function->getParameters();
46-
$numberOfParameters = $function->getNumberOfParameters();
47-
48-
if($numberOfParameters > 0 && $parameters[$numberOfParameters - 1]->isVariadic()) {
49-
$organisedArguments = array_merge($organisedArguments, array_diff($arguments, $organisedArguments));
45+
if (!$function->isVariadic()) {
46+
return $organisedArguments;
5047
}
5148

52-
return $organisedArguments;
49+
return array_merge($organisedArguments, array_diff($arguments, $organisedArguments));
5350
}
54-
}
51+
}

0 commit comments

Comments
 (0)