@@ -89,9 +89,9 @@ discard block |
||
| 89 | 89 | new ExceptionInterceptor(), |
| 90 | 90 | ], self::createMock(CoreInterface::class)); |
| 91 | 91 | |
| 92 | - try { |
|
| 92 | + try{ |
|
| 93 | 93 | $pipeline->callAction('controller', 'action'); |
| 94 | - } catch (\RuntimeException) { |
|
| 94 | + }catch (\RuntimeException){ |
|
| 95 | 95 | // Attributes won't be lost after legacy interceptor |
| 96 | 96 | self::assertSame(['key' => 'value'], $state->context->getAttributes()); |
| 97 | 97 | } |
@@ -102,8 +102,8 @@ discard block |
||
| 102 | 102 | */ |
| 103 | 103 | private function createPipeline( |
| 104 | 104 | array $interceptors = [], |
| 105 | - CoreInterface|HandlerInterface|null $lastHandler = null, |
|
| 106 | - EventDispatcherInterface|null $dispatcher = null, |
|
| 105 | + CoreInterface | HandlerInterface | null $lastHandler = null, |
|
| 106 | + EventDispatcherInterface | null $dispatcher = null, |
|
| 107 | 107 | ): InterceptorPipeline |
| 108 | 108 | { |
| 109 | 109 | $pipeline = new InterceptorPipeline($dispatcher); |
@@ -111,7 +111,7 @@ discard block |
||
| 111 | 111 | $lastHandler instanceof CoreInterface and $pipeline = $pipeline->withCore($lastHandler); |
| 112 | 112 | $lastHandler instanceof HandlerInterface and $pipeline = $pipeline->withHandler($lastHandler); |
| 113 | 113 | |
| 114 | - foreach ($interceptors as $interceptor) { |
|
| 114 | + foreach ($interceptors as $interceptor){ |
|
| 115 | 115 | $pipeline->addInterceptor($interceptor); |
| 116 | 116 | } |
| 117 | 117 | |
@@ -89,9 +89,12 @@ discard block |
||
| 89 | 89 | new ExceptionInterceptor(), |
| 90 | 90 | ], self::createMock(CoreInterface::class)); |
| 91 | 91 | |
| 92 | - try { |
|
| 92 | + try |
|
| 93 | + { |
|
| 93 | 94 | $pipeline->callAction('controller', 'action'); |
| 94 | - } catch (\RuntimeException) { |
|
| 95 | + } |
|
| 96 | + catch (\RuntimeException) |
|
| 97 | + { |
|
| 95 | 98 | // Attributes won't be lost after legacy interceptor |
| 96 | 99 | self::assertSame(['key' => 'value'], $state->context->getAttributes()); |
| 97 | 100 | } |
@@ -111,7 +114,8 @@ discard block |
||
| 111 | 114 | $lastHandler instanceof CoreInterface and $pipeline = $pipeline->withCore($lastHandler); |
| 112 | 115 | $lastHandler instanceof HandlerInterface and $pipeline = $pipeline->withHandler($lastHandler); |
| 113 | 116 | |
| 114 | - foreach ($interceptors as $interceptor) { |
|
| 117 | + foreach ($interceptors as $interceptor) |
|
| 118 | + { |
|
| 115 | 119 | $pipeline->addInterceptor($interceptor); |
| 116 | 120 | } |
| 117 | 121 | |
@@ -12,8 +12,8 @@ |
||
| 12 | 12 | public function __construct( |
| 13 | 13 | private readonly ?string $controller = null, |
| 14 | 14 | private readonly ?string $action = null, |
| 15 | - private readonly ?array $parameters = null, |
|
| 16 | - ) { |
|
| 15 | + private readonly ? array $parameters = null, |
|
| 16 | + ){ |
|
| 17 | 17 | } |
| 18 | 18 | |
| 19 | 19 | public function process(string $controller, string $action, array $parameters, CoreInterface $core): mixed |
@@ -13,7 +13,7 @@ |
||
| 13 | 13 | public function __construct( |
| 14 | 14 | private string $attribute, |
| 15 | 15 | private mixed $value, |
| 16 | - ) { |
|
| 16 | + ){ |
|
| 17 | 17 | } |
| 18 | 18 | |
| 19 | 19 | public function intercept(CallContextInterface $context, HandlerInterface $handler): mixed |
@@ -22,7 +22,7 @@ discard block |
||
| 22 | 22 | /** @internal */ |
| 23 | 23 | protected ContainerInterface $container, |
| 24 | 24 | protected bool $resolveFromPath = true, |
| 25 | - ) { |
|
| 25 | + ){ |
|
| 26 | 26 | } |
| 27 | 27 | |
| 28 | 28 | /** |
@@ -34,14 +34,14 @@ discard block |
||
| 34 | 34 | { |
| 35 | 35 | // Resolve controller method |
| 36 | 36 | $method = $context->getTarget()->getReflection(); |
| 37 | - if ($method === null) { |
|
| 37 | + if ($method === null){ |
|
| 38 | 38 | $this->resolveFromPath or throw new TargetCallException( |
| 39 | 39 | "Reflection not provided for target `{$context->getTarget()}`.", |
| 40 | 40 | TargetCallException::NOT_FOUND, |
| 41 | 41 | ); |
| 42 | 42 | |
| 43 | 43 | $path = $context->getTarget()->getPath(); |
| 44 | - if (\count($path) !== 2) { |
|
| 44 | + if (\count($path) !== 2){ |
|
| 45 | 45 | throw new TargetCallException( |
| 46 | 46 | "Invalid target path to resolve reflection for `{$context->getTarget()}`." |
| 47 | 47 | . ' Expected two parts: class and method.', |
@@ -52,13 +52,13 @@ discard block |
||
| 52 | 52 | $method = ActionResolver::pathToReflection(\reset($path), \end($path)); |
| 53 | 53 | } |
| 54 | 54 | |
| 55 | - if ($method instanceof \ReflectionFunction) { |
|
| 55 | + if ($method instanceof \ReflectionFunction){ |
|
| 56 | 56 | $method->invokeArgs( |
| 57 | 57 | $this->resolveArguments($method, $context) |
| 58 | 58 | ); |
| 59 | 59 | } |
| 60 | 60 | |
| 61 | - if (!$method instanceof \ReflectionMethod) { |
|
| 61 | + if (!$method instanceof \ReflectionMethod){ |
|
| 62 | 62 | throw new TargetCallException("Action not found for target `{$context->getTarget()}`."); |
| 63 | 63 | } |
| 64 | 64 | |
@@ -34,14 +34,16 @@ discard block |
||
| 34 | 34 | { |
| 35 | 35 | // Resolve controller method |
| 36 | 36 | $method = $context->getTarget()->getReflection(); |
| 37 | - if ($method === null) { |
|
| 37 | + if ($method === null) |
|
| 38 | + { |
|
| 38 | 39 | $this->resolveFromPath or throw new TargetCallException( |
| 39 | 40 | "Reflection not provided for target `{$context->getTarget()}`.", |
| 40 | 41 | TargetCallException::NOT_FOUND, |
| 41 | 42 | ); |
| 42 | 43 | |
| 43 | 44 | $path = $context->getTarget()->getPath(); |
| 44 | - if (\count($path) !== 2) { |
|
| 45 | + if (\count($path) !== 2) |
|
| 46 | + { |
|
| 45 | 47 | throw new TargetCallException( |
| 46 | 48 | "Invalid target path to resolve reflection for `{$context->getTarget()}`." |
| 47 | 49 | . ' Expected two parts: class and method.', |
@@ -52,13 +54,15 @@ discard block |
||
| 52 | 54 | $method = ActionResolver::pathToReflection(\reset($path), \end($path)); |
| 53 | 55 | } |
| 54 | 56 | |
| 55 | - if ($method instanceof \ReflectionFunction) { |
|
| 57 | + if ($method instanceof \ReflectionFunction) |
|
| 58 | + { |
|
| 56 | 59 | $method->invokeArgs( |
| 57 | 60 | $this->resolveArguments($method, $context) |
| 58 | 61 | ); |
| 59 | 62 | } |
| 60 | 63 | |
| 61 | - if (!$method instanceof \ReflectionMethod) { |
|
| 64 | + if (!$method instanceof \ReflectionMethod) |
|
| 65 | + { |
|
| 62 | 66 | throw new TargetCallException("Action not found for target `{$context->getTarget()}`."); |
| 63 | 67 | } |
| 64 | 68 | |
@@ -12,6 +12,6 @@ |
||
| 12 | 12 | public function __construct( |
| 13 | 13 | public readonly CallContextInterface $context, |
| 14 | 14 | public readonly InterceptorInterface $interceptor, |
| 15 | - ) { |
|
| 15 | + ){ |
|
| 16 | 16 | } |
| 17 | 17 | } |
@@ -7,7 +7,7 @@ |
||
| 7 | 7 | // Load the original class to make an alias |
| 8 | 8 | use Spiral\Interceptors\Exception\TargetCallException; |
| 9 | 9 | |
| 10 | -if (!\class_exists(TargetCallException::class)) { |
|
| 10 | +if (!\class_exists(TargetCallException::class)){ |
|
| 11 | 11 | /** |
| 12 | 12 | * Unable to perform user action or find controller. |
| 13 | 13 | * |
@@ -7,7 +7,8 @@ |
||
| 7 | 7 | // Load the original class to make an alias |
| 8 | 8 | use Spiral\Interceptors\Exception\TargetCallException; |
| 9 | 9 | |
| 10 | -if (!\class_exists(TargetCallException::class)) { |
|
| 10 | +if (!\class_exists(TargetCallException::class)) |
|
| 11 | +{ |
|
| 11 | 12 | /** |
| 12 | 13 | * Unable to perform user action or find controller. |
| 13 | 14 | * |
@@ -5,7 +5,7 @@ |
||
| 5 | 5 | namespace Spiral\Core\Exception; |
| 6 | 6 | |
| 7 | 7 | // Load the original class to make an alias |
| 8 | -if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)) { |
|
| 8 | +if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)){ |
|
| 9 | 9 | /** |
| 10 | 10 | * @deprecated will be removed in Spiral v4.0 |
| 11 | 11 | * Use {@see \Spiral\Interceptors\Exception\InterceptorException} instead. |
@@ -5,7 +5,8 @@ |
||
| 5 | 5 | namespace Spiral\Core\Exception; |
| 6 | 6 | |
| 7 | 7 | // Load the original class to make an alias |
| 8 | -if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)) { |
|
| 8 | +if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)) |
|
| 9 | +{ |
|
| 9 | 10 | /** |
| 10 | 11 | * @deprecated will be removed in Spiral v4.0 |
| 11 | 12 | * Use {@see \Spiral\Interceptors\Exception\InterceptorException} instead. |
@@ -30,10 +30,10 @@ discard block |
||
| 30 | 30 | |
| 31 | 31 | public function __construct( |
| 32 | 32 | private readonly ?EventDispatcherInterface $dispatcher = null |
| 33 | - ) { |
|
| 33 | + ){ |
|
| 34 | 34 | } |
| 35 | 35 | |
| 36 | - public function addInterceptor(CoreInterceptorInterface|InterceptorInterface $interceptor): void |
|
| 36 | + public function addInterceptor(CoreInterceptorInterface | InterceptorInterface $interceptor): void |
|
| 37 | 37 | { |
| 38 | 38 | $this->interceptors[] = $interceptor; |
| 39 | 39 | } |
@@ -59,13 +59,13 @@ discard block |
||
| 59 | 59 | */ |
| 60 | 60 | public function callAction(string $controller, string $action, array $parameters = []): mixed |
| 61 | 61 | { |
| 62 | - if ($this->context === null) { |
|
| 62 | + if ($this->context === null){ |
|
| 63 | 63 | return $this->handle( |
| 64 | 64 | new CallContext(Target::fromPathArray([$controller, $action]), $parameters), |
| 65 | 65 | ); |
| 66 | 66 | } |
| 67 | 67 | |
| 68 | - if ($this->context->getTarget()->getPath() === [$controller, $action]) { |
|
| 68 | + if ($this->context->getTarget()->getPath() === [$controller, $action]){ |
|
| 69 | 69 | return $this->handle($this->context->withArguments($parameters)); |
| 70 | 70 | } |
| 71 | 71 | |
@@ -81,13 +81,13 @@ discard block |
||
| 81 | 81 | */ |
| 82 | 82 | public function handle(CallContext $context): mixed |
| 83 | 83 | { |
| 84 | - if ($this->core === null && $this->handler === null) { |
|
| 84 | + if ($this->core === null && $this->handler === null){ |
|
| 85 | 85 | throw new InterceptorException('Unable to invoke pipeline without last handler.'); |
| 86 | 86 | } |
| 87 | 87 | |
| 88 | 88 | $path = $context->getTarget()->getPath(); |
| 89 | 89 | |
| 90 | - if (isset($this->interceptors[$this->position])) { |
|
| 90 | + if (isset($this->interceptors[$this->position])){ |
|
| 91 | 91 | $interceptor = $this->interceptors[$this->position]; |
| 92 | 92 | $handler = $this->nextWithContext($context); |
| 93 | 93 | |
@@ -59,13 +59,15 @@ discard block |
||
| 59 | 59 | */ |
| 60 | 60 | public function callAction(string $controller, string $action, array $parameters = []): mixed |
| 61 | 61 | { |
| 62 | - if ($this->context === null) { |
|
| 62 | + if ($this->context === null) |
|
| 63 | + { |
|
| 63 | 64 | return $this->handle( |
| 64 | 65 | new CallContext(Target::fromPathArray([$controller, $action]), $parameters), |
| 65 | 66 | ); |
| 66 | 67 | } |
| 67 | 68 | |
| 68 | - if ($this->context->getTarget()->getPath() === [$controller, $action]) { |
|
| 69 | + if ($this->context->getTarget()->getPath() === [$controller, $action]) |
|
| 70 | + { |
|
| 69 | 71 | return $this->handle($this->context->withArguments($parameters)); |
| 70 | 72 | } |
| 71 | 73 | |
@@ -81,13 +83,15 @@ discard block |
||
| 81 | 83 | */ |
| 82 | 84 | public function handle(CallContext $context): mixed |
| 83 | 85 | { |
| 84 | - if ($this->core === null && $this->handler === null) { |
|
| 86 | + if ($this->core === null && $this->handler === null) |
|
| 87 | + { |
|
| 85 | 88 | throw new InterceptorException('Unable to invoke pipeline without last handler.'); |
| 86 | 89 | } |
| 87 | 90 | |
| 88 | 91 | $path = $context->getTarget()->getPath(); |
| 89 | 92 | |
| 90 | - if (isset($this->interceptors[$this->position])) { |
|
| 93 | + if (isset($this->interceptors[$this->position])) |
|
| 94 | + { |
|
| 91 | 95 | $interceptor = $this->interceptors[$this->position]; |
| 92 | 96 | $handler = $this->nextWithContext($context); |
| 93 | 97 | |
@@ -21,11 +21,11 @@ |
||
| 21 | 21 | public function __construct( |
| 22 | 22 | private readonly CoreInterface $core, |
| 23 | 23 | ?EventDispatcherInterface $dispatcher = null |
| 24 | - ) { |
|
| 24 | + ){ |
|
| 25 | 25 | $this->pipeline = new InterceptorPipeline($dispatcher); |
| 26 | 26 | } |
| 27 | 27 | |
| 28 | - public function addInterceptor(CoreInterceptorInterface|InterceptorInterface $interceptor): void |
|
| 28 | + public function addInterceptor(CoreInterceptorInterface | InterceptorInterface $interceptor): void |
|
| 29 | 29 | { |
| 30 | 30 | $this->pipeline->addInterceptor($interceptor); |
| 31 | 31 | } |