applied changes from code review

This commit is contained in:
Fabio Ivona
2021-11-26 15:28:43 +01:00
parent cdd67a6900
commit 5f0752e874

View File

@ -19,7 +19,7 @@ trait Extendable
*/
private static array $extends = [];
/** @var array<string, array<Closure>> */
/** @var array<string, array<Closure(Closure $next, mixed ...$arguments): void>> */
private static array $pipes = [];
/**
@ -49,9 +49,8 @@ trait Extendable
};
}
//@phpstan-ignore-next-line
self::pipe($name, function ($next, ...$arguments) use ($handler, $filter) {
//@phpstan-ignore-next-line
/** @phpstan-ignore-next-line */
if ($filter($this->value)) {
//@phpstan-ignore-next-line
$handler->bindTo($this, get_class($this))(...$arguments);
@ -71,30 +70,12 @@ trait Extendable
return array_key_exists($name, static::$extends);
}
/**
* Checks if pipes are registered for a given expectation.
*/
public static function hasPipes(string $name): bool
{
return array_key_exists($name, static::$pipes);
}
/**
* @return array<int, Closure>
*/
public function pipes(string $name, object $context, string $scope): array
private function pipes(string $name, object $context, string $scope): array
{
if (!self::hasPipes($name)) {
return [];
}
$decorators = [];
foreach (self::$pipes[$name] as $decorator) {
$decorators[] = $decorator->bindTo($context, $scope);
}
//@phpstan-ignore-next-line
return $decorators;
return array_map(fn(Closure $pipe) => $pipe->bindTo($context, $scope), self::$pipes[$name] ?? []);
}
/**