spiral / framework

src/SendIt/tests/JobTest.php

Spacing +3 added lines, -3 removed lines

@@ -75,13 +75,13 @@ 
             ['emails' => ['email@domain.com']]
         );
 
-        try {
+        try{
             $this->getHandler()->handle(
                 MailQueue::JOB_NAME,
                 'id',
                 json_encode(MessageSerializer::pack($this->getMail()))
             );
-        } catch (TransportException $e) {
+        }catch (TransportException $e){
         }
 
         $this->logger->mockery_verify();
@@ -98,7 +98,7 @@ 
     private function expectRenderer(Email $email): void
     {
         $this->renderer->expects('render')->withArgs(
-            function (Message $message) {
+            function (Message $message){
                 $this->assertSame($message->getSubject(), 'test');
                 return true;
             }

Braces +7 added lines, -3 removed lines

@@ -75,13 +75,16 @@ 
             ['emails' => ['email@domain.com']]
         );
 
-        try {
+        try
+        {
             $this->getHandler()->handle(
                 MailQueue::JOB_NAME,
                 'id',
                 json_encode(MessageSerializer::pack($this->getMail()))
             );
-        } catch (TransportException $e) {
+        }
+        catch (TransportException $e)
+        {
         }
 
         $this->logger->mockery_verify();
@@ -98,7 +101,8 @@ 
     private function expectRenderer(Email $email): void
     {
         $this->renderer->expects('render')->withArgs(
-            function (Message $message) {
+            function (Message $message)
+            {
                 $this->assertSame($message->getSubject(), 'test');
                 return true;
             }

src/Prototype/src/Command/InjectCommand.php

Braces +25 added lines, -12 removed lines

@@ -55,7 +55,8 @@ 
     public function perform(): void
     {
         $prototyped = $this->locator->getTargetClasses();
-        if ($prototyped === []) {
+        if ($prototyped === [])
+        {
             $this->writeln('<comment>No prototyped classes found.</comment>');
 
             return;
@@ -63,18 +64,23 @@ 
 
         $targets = [];
 
-        foreach ($prototyped as $class) {
+        foreach ($prototyped as $class)
+        {
             $proto = $this->getPrototypeProperties($class, $prototyped);
-            if (empty($proto)) {
+            if (empty($proto))
+            {
                 $modified = $this->modify($class, $proto);
-                if ($modified !== null) {
+                if ($modified !== null)
+                {
                     $targets[] = $modified;
                 }
                 continue;
             }
 
-            foreach ($proto as $target) {
-                if ($target instanceof \Throwable) {
+            foreach ($proto as $target)
+            {
+                if ($target instanceof \Throwable)
+                {
                     $targets[] = [
                         $class->getName(),
                         $target->getMessage(),
@@ -83,7 +89,8 @@ 
                     continue 2;
                 }
 
-                if ($target === null) {
+                if ($target === null)
+                {
                     continue 2;
                 }
             }
@@ -91,14 +98,17 @@ 
             $targets[] = [$class->getName(), $this->mergeNames($proto), $this->mergeTargets($proto)];
 
             $modified = $this->modify($class, $proto);
-            if ($modified !== null) {
+            if ($modified !== null)
+            {
                 $targets[] = $modified;
             }
         }
 
-        if (!empty($targets)) {
+        if (!empty($targets))
+        {
             $grid = $this->table(['Class:', 'Property:', 'Target:']);
-            foreach ($targets as $target) {
+            foreach ($targets as $target)
+            {
                 $grid->addRow($target);
             }
 
@@ -109,7 +119,8 @@ 
     private function modify(\ReflectionClass $class, array $proto): ?array
     {
         $classDefinition = $this->extractor->extract($class->getFilename(), $proto);
-        try {
+        try
+        {
             $modified = $this->injector->injectDependencies(
                 file_get_contents($class->getFileName()),
                 $classDefinition,
@@ -120,7 +131,9 @@ 
 
             file_put_contents($class->getFileName(), $modified);
             return null;
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             return [$class->getName(), $e->getMessage(), "{$e->getFile()}:L{$e->getLine()}"];
         }
     }

Spacing +12 added lines, -12 removed lines

@@ -51,7 +51,7 @@ 
     public function perform(): void
     {
         $prototyped = $this->locator->getTargetClasses();
-        if ($prototyped === []) {
+        if ($prototyped === []){
             $this->writeln('<comment>No prototyped classes found.</comment>');
 
             return;
@@ -59,18 +59,18 @@ 
 
         $targets = [];
 
-        foreach ($prototyped as $class) {
+        foreach ($prototyped as $class){
             $proto = $this->getPrototypeProperties($class, $prototyped);
-            if (empty($proto)) {
+            if (empty($proto)){
                 $modified = $this->modify($class, $proto);
-                if ($modified !== null) {
+                if ($modified !== null){
                     $targets[] = $modified;
                 }
                 continue;
             }
 
-            foreach ($proto as $target) {
-                if ($target instanceof \Throwable) {
+            foreach ($proto as $target){
+                if ($target instanceof \Throwable){
                     $targets[] = [
                         $class->getName(),
                         $target->getMessage(),
@@ -79,7 +79,7 @@ 
                     continue 2;
                 }
 
-                if ($target === null) {
+                if ($target === null){
                     continue 2;
                 }
             }
@@ -87,14 +87,14 @@ 
             $targets[] = [$class->getName(), $this->mergeNames($proto), $this->mergeTargets($proto)];
 
             $modified = $this->modify($class, $proto);
-            if ($modified !== null) {
+            if ($modified !== null){
                 $targets[] = $modified;
             }
         }
 
-        if (!empty($targets)) {
+        if (!empty($targets)){
             $grid = $this->table(['Class:', 'Property:', 'Target:']);
-            foreach ($targets as $target) {
+            foreach ($targets as $target){
                 $grid->addRow($target);
             }
 
@@ -105,7 +105,7 @@ 
     private function modify(\ReflectionClass $class, array $proto): ?array
     {
         $classDefinition = $this->extractor->extract($class->getFilename(), $proto);
-        try {
+        try{
             $modified = $this->injector->injectDependencies(
                 file_get_contents($class->getFileName()),
                 $classDefinition,
@@ -116,7 +116,7 @@ 
 
             file_put_contents($class->getFileName(), $modified);
             return null;
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             return [$class->getName(), $e->getMessage(), "{$e->getFile()}:L{$e->getLine()}"];
         }
     }

src/DataGrid/src/GridFactory.php

Spacing +13 added lines, -13 removed lines

@@ -108,7 +108,7 @@ 
         ['view' => $view, 'source' => $source] = $this->applySorters($view, $source, $schema);
         ['view' => $view, 'source' => $source] = $this->applyPaginator($view, $source, $schema);
 
-        if (!is_iterable($source)) {
+        if (!is_iterable($source)){
             throw new GridViewException('GridView expects the source to be iterable after all.');
         }
 
@@ -126,11 +126,11 @@ 
     protected function getFilters($source, GridSchema $schema): array
     {
         $filters = [];
-        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value) {
-            if ($schema->hasFilter($name)) {
+        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value){
+            if ($schema->hasFilter($name)){
                 $filter = $schema->getFilter($name)->withValue($value);
 
-                if ($filter !== null) {
+                if ($filter !== null){
                     $source = $this->compiler->compile($source, $filter);
                     $filters[$name] = $filter->getValue();
                 }
@@ -142,7 +142,7 @@ 
 
     protected function applyCounter(GridInterface $view, $source, GridSchema $schema): array
     {
-        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)) {
+        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)){
             $view = $view->withOption(GridInterface::COUNT, ($this->count)($source));
         }
 
@@ -160,11 +160,11 @@ 
     protected function getSorters($source, GridSchema $schema): array
     {
         $sorters = [];
-        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value) {
-            if ($schema->hasSorter($name)) {
+        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value){
+            if ($schema->hasSorter($name)){
                 $sorter = $schema->getSorter($name)->withDirection($value);
 
-                if ($sorter !== null) {
+                if ($sorter !== null){
                     $source = $this->compiler->compile($source, $sorter);
                     $sorters[$name] = $sorter->getValue();
                 }
@@ -176,7 +176,7 @@ 
 
     protected function applyPaginator(GridInterface $view, $source, GridSchema $schema): array
     {
-        if ($schema->getPaginator() !== null) {
+        if ($schema->getPaginator() !== null){
             ['source' => $source, 'paginator' => $paginator] = $this->getPaginator($source, $schema);
             $view = $view->withOption(GridInterface::PAGINATOR, $paginator);
         }
@@ -187,12 +187,12 @@ 
     protected function getPaginator($source, GridSchema $schema): array
     {
         $paginator = $schema->getPaginator();
-        if (!$paginator instanceof FilterInterface) {
+        if (!$paginator instanceof FilterInterface){
             throw new CompilerException('Paginator can not be null');
         }
 
         $withValue = $paginator->withValue($this->getOption(static::KEY_PAGINATE));
-        if ($withValue === null) {
+        if ($withValue === null){
             throw new CompilerException('Paginator can not be null');
         }
 
@@ -211,7 +211,7 @@ 
     protected function getOptionArray(string $option): array
     {
         $result = $this->getOption($option);
-        if (!is_array($result)) {
+        if (!is_array($result)){
             return [];
         }
 
@@ -226,7 +226,7 @@ 
      */
     protected function getOption(string $option)
     {
-        if ($this->input->hasValue($option)) {
+        if ($this->input->hasValue($option)){
             return $this->input->getValue($option);
         }
 

Braces +26 added lines, -13 removed lines

@@ -108,7 +108,8 @@ 
         ['view' => $view, 'source' => $source] = $this->applySorters($view, $source, $schema);
         ['view' => $view, 'source' => $source] = $this->applyPaginator($view, $source, $schema);
 
-        if (!is_iterable($source)) {
+        if (!is_iterable($source))
+        {
             throw new GridViewException('GridView expects the source to be iterable after all.');
         }
 
@@ -126,11 +127,14 @@ 
     protected function getFilters($source, GridSchema $schema): array
     {
         $filters = [];
-        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value) {
-            if ($schema->hasFilter($name)) {
+        foreach ($this->getOptionArray(static::KEY_FILTER) ?? [] as $name => $value)
+        {
+            if ($schema->hasFilter($name))
+            {
                 $filter = $schema->getFilter($name)->withValue($value);
 
-                if ($filter !== null) {
+                if ($filter !== null)
+                {
                     $source = $this->compiler->compile($source, $filter);
                     $filters[$name] = $filter->getValue();
                 }
@@ -142,7 +146,8 @@ 
 
     protected function applyCounter(GridInterface $view, $source, GridSchema $schema): array
     {
-        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT)) {
+        if (is_countable($source) && $this->getOption(static::KEY_FETCH_COUNT))
+        {
             $view = $view->withOption(GridInterface::COUNT, ($this->count)($source));
         }
 
@@ -160,11 +165,14 @@ 
     protected function getSorters($source, GridSchema $schema): array
     {
         $sorters = [];
-        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value) {
-            if ($schema->hasSorter($name)) {
+        foreach ($this->getOptionArray(static::KEY_SORT) ?? [] as $name => $value)
+        {
+            if ($schema->hasSorter($name))
+            {
                 $sorter = $schema->getSorter($name)->withDirection($value);
 
-                if ($sorter !== null) {
+                if ($sorter !== null)
+                {
                     $source = $this->compiler->compile($source, $sorter);
                     $sorters[$name] = $sorter->getValue();
                 }
@@ -176,7 +184,8 @@ 
 
     protected function applyPaginator(GridInterface $view, $source, GridSchema $schema): array
     {
-        if ($schema->getPaginator() !== null) {
+        if ($schema->getPaginator() !== null)
+        {
             ['source' => $source, 'paginator' => $paginator] = $this->getPaginator($source, $schema);
             $view = $view->withOption(GridInterface::PAGINATOR, $paginator);
         }
@@ -187,12 +196,14 @@ 
     protected function getPaginator($source, GridSchema $schema): array
     {
         $paginator = $schema->getPaginator();
-        if (!$paginator instanceof FilterInterface) {
+        if (!$paginator instanceof FilterInterface)
+        {
             throw new CompilerException('Paginator can not be null');
         }
 
         $withValue = $paginator->withValue($this->getOption(static::KEY_PAGINATE));
-        if ($withValue === null) {
+        if ($withValue === null)
+        {
             throw new CompilerException('Paginator can not be null');
         }
 
@@ -211,7 +222,8 @@ 
     protected function getOptionArray(string $option): array
     {
         $result = $this->getOption($option);
-        if (!is_array($result)) {
+        if (!is_array($result))
+        {
             return [];
         }
 
@@ -226,7 +238,8 @@ 
      */
     protected function getOption(string $option)
     {
-        if ($this->input->hasValue($option)) {
+        if ($this->input->hasValue($option))
+        {
             return $this->input->getValue($option);
         }
 

src/DataGrid/src/Specification/Value/DatetimeFormatValue.php

Spacing +4 added lines, -4 removed lines

@@ -44,19 +44,19 @@
      */
     public function convert($value)
     {
-        try {
+        try{
             $datetime = DateTimeImmutable::createFromFormat($this->readFrom, (string)$value);
-            if (!$datetime instanceof DateTimeImmutable) {
+            if (!$datetime instanceof DateTimeImmutable){
                 return null;
             }
 
-            if ($this->convertInto !== null) {
+            if ($this->convertInto !== null){
                 $formatted = $datetime->format($this->convertInto);
                 return is_string($formatted) ? $formatted : null;
             }
 
             return $datetime;
-        } catch (Throwable $e) {
+        }catch (Throwable $e){
             return null;
         }
     }

Braces +9 added lines, -4 removed lines

@@ -44,19 +44,24 @@
      */
     public function convert($value)
     {
-        try {
+        try
+        {
             $datetime = DateTimeImmutable::createFromFormat($this->readFrom, (string)$value);
-            if (!$datetime instanceof DateTimeImmutable) {
+            if (!$datetime instanceof DateTimeImmutable)
+            {
                 return null;
             }
 
-            if ($this->convertInto !== null) {
+            if ($this->convertInto !== null)
+            {
                 $formatted = $datetime->format($this->convertInto);
                 return is_string($formatted) ? $formatted : null;
             }
 
             return $datetime;
-        } catch (Throwable $e) {
+        }
+        catch (Throwable $e)
+        {
             return null;
         }
     }

src/Validation/src/Checker/DatetimeChecker.php

Spacing +11 added lines, -11 removed lines

@@ -82,7 +82,7 @@ 
      */
     public function format($value, string $format): bool
     {
-        if (!$this->isApplicableValue($value)) {
+        if (!$this->isApplicableValue($value)){
             return false;
         }
 
@@ -110,7 +110,7 @@ 
      */
     public function timezone($value): bool
     {
-        if (!is_scalar($value)) {
+        if (!is_scalar($value)){
             return false;
         }
 
@@ -150,9 +150,9 @@ 
      */
     private function now(): ?\DateTimeInterface
     {
-        try {
+        try{
             return $this->date($this->now ?: 'now');
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             //here's the fail;
         }
 
@@ -165,25 +165,25 @@ 
      */
     private function date($value): ?\DateTimeInterface
     {
-        if (is_callable($value)) {
+        if (is_callable($value)){
             $value = $value();
         }
 
-        if ($value instanceof \DateTimeInterface) {
+        if ($value instanceof \DateTimeInterface){
             return $value;
         }
 
-        if (!$this->isApplicableValue($value)) {
+        if (!$this->isApplicableValue($value)){
             return null;
         }
 
-        try {
-            if (!$value) {
+        try{
+            if (!$value){
                 $value = '0';
             }
 
             return new \DateTimeImmutable(is_numeric($value) ? sprintf('@%d', $value) : trim($value));
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             //here's the fail;
         }
 
@@ -206,7 +206,7 @@ 
     private function fromField(string $field): ?\DateTimeInterface
     {
         $before = $this->getValidator()->getValue($field);
-        if ($before !== null) {
+        if ($before !== null){
             return $this->date($before);
         }
 

Braces +24 added lines, -11 removed lines

@@ -82,7 +82,8 @@ 
      */
     public function format($value, string $format): bool
     {
-        if (!$this->isApplicableValue($value)) {
+        if (!$this->isApplicableValue($value))
+        {
             return false;
         }
 
@@ -110,7 +111,8 @@ 
      */
     public function timezone($value): bool
     {
-        if (!is_scalar($value)) {
+        if (!is_scalar($value))
+        {
             return false;
         }
 
@@ -150,9 +152,12 @@ 
      */
     private function now(): ?\DateTimeInterface
     {
-        try {
+        try
+        {
             return $this->date($this->now ?: 'now');
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             //here's the fail;
         }
 
@@ -165,25 +170,32 @@ 
      */
     private function date($value): ?\DateTimeInterface
     {
-        if (is_callable($value)) {
+        if (is_callable($value))
+        {
             $value = $value();
         }
 
-        if ($value instanceof \DateTimeInterface) {
+        if ($value instanceof \DateTimeInterface)
+        {
             return $value;
         }
 
-        if (!$this->isApplicableValue($value)) {
+        if (!$this->isApplicableValue($value))
+        {
             return null;
         }
 
-        try {
-            if (!$value) {
+        try
+        {
+            if (!$value)
+            {
                 $value = '0';
             }
 
             return new \DateTimeImmutable(is_numeric($value) ? sprintf('@%d', $value) : trim($value));
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             //here's the fail;
         }
 
@@ -206,7 +218,8 @@ 
     private function fromField(string $field): ?\DateTimeInterface
     {
         $before = $this->getValidator()->getValue($field);
-        if ($before !== null) {
+        if ($before !== null)
+        {
             return $this->date($before);
         }
 

src/Validation/tests/Checkers/DatetimeTest.php

Spacing +22 added lines, -22 removed lines

@@ -44,7 +44,7 @@ 
             [true, $callableNow, $now, true, true]
         ];
 
-        $callableFutureTime = static function () {
+        $callableFutureTime = static function (){
             return time() + 1000;
         };
         yield from [
@@ -52,7 +52,7 @@ 
             [false, $callableFutureTime, $now, true, true],
         ];
 
-        $callablePastTime = static function () {
+        $callablePastTime = static function (){
             return time() - 1000;
         };
         yield from [
@@ -231,24 +231,24 @@ 
     public function validProvider(): array
     {
         return [
-            [true, time() - 1000,],
-            [true, time(),],
-            [true, date('u'),],
-            [true, time() + 1000,],
-            [true, '',],
-            [true, 'tomorrow +2hours',],
-            [true, 'yesterday -2hours',],
-            [true, 'now',],
-            [true, 'now + 1000 seconds',],
-            [true, 'now - 1000 seconds',],
-            [true, 0,],
-            [true, 1.1,],
-            [false, [],],
-            [false, false,],
-            [false, true,],
-            [false, null,],
-            [false, [],],
-            [false, new \stdClass(),],
+            [true, time() - 1000, ],
+            [true, time(), ],
+            [true, date('u'), ],
+            [true, time() + 1000, ],
+            [true, '', ],
+            [true, 'tomorrow +2hours', ],
+            [true, 'yesterday -2hours', ],
+            [true, 'now', ],
+            [true, 'now + 1000 seconds', ],
+            [true, 'now - 1000 seconds', ],
+            [true, 0, ],
+            [true, 1.1, ],
+            [false, [], ],
+            [false, false, ],
+            [false, true, ],
+            [false, null, ],
+            [false, [], ],
+            [false, new \stdClass(), ],
         ];
     }
 
@@ -256,7 +256,7 @@ 
     {
         $checker = new DatetimeChecker();
 
-        foreach (\DateTimeZone::listIdentifiers() as $identifier) {
+        foreach (\DateTimeZone::listIdentifiers() as $identifier){
             $this->assertTrue($checker->timezone($identifier));
             $this->assertFalse($checker->timezone(str_rot13($identifier)));
         }
@@ -397,7 +397,7 @@ 
 
     private function now(): \Closure
     {
-        return static function () {
+        return static function (){
             return \time();
         };
     }

Braces +14 added lines, -7 removed lines

@@ -35,7 +35,8 @@ 
     public function nowProvider(): iterable
     {
         $now = new \DateTime();
-        $callableNow = static function () use ($now) {
+        $callableNow = static function () use ($now)
+        {
             return $now;
         };
 
@@ -44,7 +45,8 @@ 
             [true, $callableNow, $now, true, true]
         ];
 
-        $callableFutureTime = static function () {
+        $callableFutureTime = static function ()
+        {
             return time() + 1000;
         };
         yield from [
@@ -52,7 +54,8 @@ 
             [false, $callableFutureTime, $now, true, true],
         ];
 
-        $callablePastTime = static function () {
+        $callablePastTime = static function ()
+        {
             return time() - 1000;
         };
         yield from [
@@ -256,7 +259,8 @@ 
     {
         $checker = new DatetimeChecker();
 
-        foreach (\DateTimeZone::listIdentifiers() as $identifier) {
+        foreach (\DateTimeZone::listIdentifiers() as $identifier)
+        {
             $this->assertTrue($checker->timezone($identifier));
             $this->assertFalse($checker->timezone(str_rot13($identifier)));
         }
@@ -397,21 +401,24 @@ 
 
     private function now(): \Closure
     {
-        return static function () {
+        return static function ()
+        {
             return \time();
         };
     }
 
     private function inFuture(int $seconds): \Closure
     {
-        return static function () use ($seconds) {
+        return static function () use ($seconds)
+        {
             return \time() + $seconds;
         };
     }
 
     private function inPast(int $seconds): \Closure
     {
-        return static function () use ($seconds) {
+        return static function () use ($seconds)
+        {
             return \time() - $seconds;
         };
     }

src/Framework/Domain/PipelineInterceptor.php

Spacing +18 added lines, -18 removed lines

@@ -44,17 +44,17 @@ 
     public function process(string $controller, string $action, array $parameters, CoreInterface $core)
     {
         $annotation = $this->readAnnotation($controller, $action);
-        if ($core instanceof InterceptorPipeline && $annotation->skipNext) {
+        if ($core instanceof InterceptorPipeline && $annotation->skipNext){
             $this->removeNextInterceptorsFromOriginalPipeline($core);
         }
 
         $pipeline = $this->getCachedPipeline($controller, $action, $annotation);
-        if (!empty($pipeline)) {
-            if ($core instanceof InterceptorPipeline) {
+        if (!empty($pipeline)){
+            if ($core instanceof InterceptorPipeline){
                 $this->injectInterceptorsIntoOriginalPipeline($core, $pipeline);
-            } else {
+            }else{
                 $core = new InterceptableCore($core);
-                foreach ($pipeline as $interceptor) {
+                foreach ($pipeline as $interceptor){
                     $core->addInterceptor($interceptor);
                 }
             }
@@ -70,9 +70,9 @@ 
      */
     private function readAnnotation(string $controller, string $action): Pipeline
     {
-        try {
+        try{
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             return new Pipeline();
         }
 
@@ -91,14 +91,14 @@ 
 
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
         $newInterceptors = [];
-        foreach ($oldInterceptors as $interceptor) {
+        foreach ($oldInterceptors as $interceptor){
             $newInterceptors[] = $interceptor;
-            if ($interceptor instanceof self) {
+            if ($interceptor instanceof self){
                 break;
             }
         }
 
-        if (count($newInterceptors) !== count($oldInterceptors)) {
+        if (count($newInterceptors) !== count($oldInterceptors)){
             $pipelineReflection->setValue($pipeline, $newInterceptors);
         }
         $pipelineReflection->setAccessible(false);
@@ -111,16 +111,16 @@ 
 
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
         $newInterceptors = [];
-        foreach ($oldInterceptors as $interceptor) {
+        foreach ($oldInterceptors as $interceptor){
             $newInterceptors[] = $interceptor;
-            if ($interceptor instanceof self) {
-                foreach ($interceptors as $newInterceptor) {
+            if ($interceptor instanceof self){
+                foreach ($interceptors as $newInterceptor){
                     $newInterceptors[] = $newInterceptor;
                 }
             }
         }
 
-        if (count($newInterceptors) !== count($oldInterceptors)) {
+        if (count($newInterceptors) !== count($oldInterceptors)){
             $pipelineReflection->setValue($pipeline, $newInterceptors);
         }
         $pipelineReflection->setAccessible(false);
@@ -135,7 +135,7 @@ 
     private function getCachedPipeline(string $controller, string $action, Pipeline $annotation): array
     {
         $key = "{$controller}:{$action}";
-        if (!array_key_exists($key, $this->cache)) {
+        if (!array_key_exists($key, $this->cache)){
             $this->cache[$key] = $this->extractAnnotationPipeline($annotation);
         }
 
@@ -149,10 +149,10 @@ 
     private function extractAnnotationPipeline(Pipeline $annotation): array
     {
         $interceptors = [];
-        foreach ($annotation->pipeline as $interceptor) {
-            try {
+        foreach ($annotation->pipeline as $interceptor){
+            try{
                 $interceptors[] = $this->container->get($interceptor);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
             }
         }
 

Braces +39 added lines, -18 removed lines

@@ -44,17 +44,23 @@ 
     public function process(string $controller, string $action, array $parameters, CoreInterface $core)
     {
         $annotation = $this->readAnnotation($controller, $action);
-        if ($core instanceof InterceptorPipeline && $annotation->skipNext) {
+        if ($core instanceof InterceptorPipeline && $annotation->skipNext)
+        {
             $this->removeNextInterceptorsFromOriginalPipeline($core);
         }
 
         $pipeline = $this->getCachedPipeline($controller, $action, $annotation);
-        if (!empty($pipeline)) {
-            if ($core instanceof InterceptorPipeline) {
+        if (!empty($pipeline))
+        {
+            if ($core instanceof InterceptorPipeline)
+            {
                 $this->injectInterceptorsIntoOriginalPipeline($core, $pipeline);
-            } else {
+            }
+            else
+            {
                 $core = new InterceptableCore($core);
-                foreach ($pipeline as $interceptor) {
+                foreach ($pipeline as $interceptor)
+                {
                     $core->addInterceptor($interceptor);
                 }
             }
@@ -70,9 +76,12 @@ 
      */
     private function readAnnotation(string $controller, string $action): Pipeline
     {
-        try {
+        try
+        {
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             return new Pipeline();
         }
 
@@ -91,14 +100,17 @@ 
 
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
         $newInterceptors = [];
-        foreach ($oldInterceptors as $interceptor) {
+        foreach ($oldInterceptors as $interceptor)
+        {
             $newInterceptors[] = $interceptor;
-            if ($interceptor instanceof self) {
+            if ($interceptor instanceof self)
+            {
                 break;
             }
         }
 
-        if (count($newInterceptors) !== count($oldInterceptors)) {
+        if (count($newInterceptors) !== count($oldInterceptors))
+        {
             $pipelineReflection->setValue($pipeline, $newInterceptors);
         }
         $pipelineReflection->setAccessible(false);
@@ -111,16 +123,20 @@ 
 
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
         $newInterceptors = [];
-        foreach ($oldInterceptors as $interceptor) {
+        foreach ($oldInterceptors as $interceptor)
+        {
             $newInterceptors[] = $interceptor;
-            if ($interceptor instanceof self) {
-                foreach ($interceptors as $newInterceptor) {
+            if ($interceptor instanceof self)
+            {
+                foreach ($interceptors as $newInterceptor)
+                {
                     $newInterceptors[] = $newInterceptor;
                 }
             }
         }
 
-        if (count($newInterceptors) !== count($oldInterceptors)) {
+        if (count($newInterceptors) !== count($oldInterceptors))
+        {
             $pipelineReflection->setValue($pipeline, $newInterceptors);
         }
         $pipelineReflection->setAccessible(false);
@@ -135,7 +151,8 @@ 
     private function getCachedPipeline(string $controller, string $action, Pipeline $annotation): array
     {
         $key = "{$controller}:{$action}";
-        if (!array_key_exists($key, $this->cache)) {
+        if (!array_key_exists($key, $this->cache))
+        {
             $this->cache[$key] = $this->extractAnnotationPipeline($annotation);
         }
 
@@ -149,10 +166,14 @@ 
     private function extractAnnotationPipeline(Pipeline $annotation): array
     {
         $interceptors = [];
-        foreach ($annotation->pipeline as $interceptor) {
-            try {
+        foreach ($annotation->pipeline as $interceptor)
+        {
+            try
+            {
                 $interceptors[] = $this->container->get($interceptor);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
             }
         }
 

tests/app/src/Interceptor/Append.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     public function process(string $controller, string $action, array $parameters, CoreInterface $core): array
     {
         $result = $core->callAction($controller, $action, $parameters);
-        if (!is_array($result)) {
+        if (!is_array($result)){
             $result = [];
         }
         $result[] = $this->string;

Braces +2 added lines, -1 removed lines

@@ -19,7 +19,8 @@
     public function process(string $controller, string $action, array $parameters, CoreInterface $core): array
     {
         $result = $core->callAction($controller, $action, $parameters);
-        if (!is_array($result)) {
+        if (!is_array($result))
+        {
             $result = [];
         }
         $result[] = $this->string;

tests/app/src/Bootloader/AppBootloader.php

Braces +2 added lines, -1 removed lines

@@ -173,7 +173,8 @@
     {
         $core = new InterceptableCore($this->core);
 
-        foreach ($interceptors as $interceptor) {
+        foreach ($interceptors as $interceptor)
+        {
             $core->addInterceptor(is_object($interceptor) ? $interceptor : $this->container->get($interceptor));
         }
 

Spacing +3 added lines, -3 removed lines

@@ -89,8 +89,8 @@ 
             new Route('/auth/<action>', new Controller(AuthController::class))
         );
 
-        $views->addDirectory('custom', __DIR__ . '/../../views/custom/');
-        $views->addDirectory('stempler', __DIR__ . '/../../views/stempler/');
+        $views->addDirectory('custom', __DIR__.'/../../views/custom/');
+        $views->addDirectory('stempler', __DIR__.'/../../views/stempler/');
         $views->addEngine(TestEngine::class);
 
         $validation->addAlias('aliased', 'notEmpty');
@@ -232,7 +232,7 @@ 
     {
         $core = new InterceptableCore($this->core);
 
-        foreach ($interceptors as $interceptor) {
+        foreach ($interceptors as $interceptor){
             $core->addInterceptor(is_object($interceptor) ? $interceptor : $this->container->get($interceptor));
         }