spiral / framework

src/Views/src/Exception/RenderException.php

Spacing +3 added lines, -3 removed lines

@@ -10,9 +10,9 @@
 
     public function __construct(?\Throwable $previous = null)
     {
-        parent::__construct((string) $previous?->getMessage(), (int) ($previous?->getCode() ?? 0), $previous);
-        $this->file = (string) $previous?->getFile();
-        $this->line = (int) $previous?->getLine();
+        parent::__construct((string)$previous?->getMessage(), (int)($previous?->getCode() ?? 0), $previous);
+        $this->file = (string)$previous?->getFile();
+        $this->line = (int)$previous?->getLine();
     }
 
     /**

src/Views/src/ViewCache.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 
     public function reset(?ContextInterface $context = null): void
     {
-        if (empty($context)) {
+        if (empty($context)){
             $this->cache = [];
             return;
         }
@@ -25,7 +25,7 @@ 
      */
     public function resetPath(string $path): void
     {
-        foreach ($this->cache as &$cache) {
+        foreach ($this->cache as &$cache){
             unset($cache[$path], $cache);
         }
     }
@@ -45,7 +45,7 @@ 
      */
     public function get(ContextInterface $context, string $path): ViewInterface
     {
-        if (!$this->has($context, $path)) {
+        if (!$this->has($context, $path)){
             throw new CacheException(\sprintf('No cache is available for %s.', $path));
         }
 

Braces +6 added lines, -3 removed lines

@@ -12,7 +12,8 @@ 
 
     public function reset(?ContextInterface $context = null): void
     {
-        if (empty($context)) {
+        if (empty($context))
+        {
             $this->cache = [];
             return;
         }
@@ -25,7 +26,8 @@ 
      */
     public function resetPath(string $path): void
     {
-        foreach ($this->cache as &$cache) {
+        foreach ($this->cache as &$cache)
+        {
             unset($cache[$path], $cache);
         }
     }
@@ -45,7 +47,8 @@ 
      */
     public function get(ContextInterface $context, string $path): ViewInterface
     {
-        if (!$this->has($context, $path)) {
+        if (!$this->has($context, $path))
+        {
             throw new CacheException(\sprintf('No cache is available for %s.', $path));
         }
 

src/Logger/src/Traits/LoggerTrait.php

Spacing +3 added lines, -3 removed lines

@@ -29,11 +29,11 @@ 
      */
     protected function getLogger(?string $channel = null): LoggerInterface
     {
-        if ($channel !== null) {
+        if ($channel !== null){
             return $this->allocateLogger($channel);
         }
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null){
             return $this->logger;
         }
 
@@ -47,7 +47,7 @@ 
     private function allocateLogger(string $channel): LoggerInterface
     {
         $container = ContainerScope::getContainer();
-        if ($container === null || !$container->has(LogsInterface::class)) {
+        if ($container === null || !$container->has(LogsInterface::class)){
             return $this->logger ?? new NullLogger();
         }
 

Braces +6 added lines, -3 removed lines

@@ -29,11 +29,13 @@ 
      */
     protected function getLogger(?string $channel = null): LoggerInterface
     {
-        if ($channel !== null) {
+        if ($channel !== null)
+        {
             return $this->allocateLogger($channel);
         }
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null)
+        {
             return $this->logger;
         }
 
@@ -47,7 +49,8 @@ 
     private function allocateLogger(string $channel): LoggerInterface
     {
         $container = ContainerScope::getContainer();
-        if ($container === null || !$container->has(LogsInterface::class)) {
+        if ($container === null || !$container->has(LogsInterface::class))
+        {
             return $this->logger ?? new NullLogger();
         }
 

src/Tokenizer/src/InterfaceLocator.php

Spacing +9 added lines, -9 removed lines

@@ -13,18 +13,18 @@ 
 {
     public const INJECTOR = InterfaceLocatorInjector::class;
 
-    public function getInterfaces(string|null $target = null): array
+    public function getInterfaces(string | null $target = null): array
     {
-        if (!empty($target)) {
+        if (!empty($target)){
             $target = new \ReflectionClass($target);
         }
 
         $result = [];
-        foreach ($this->availableInterfaces() as $interface) {
-            try {
+        foreach ($this->availableInterfaces() as $interface){
+            try{
                 $reflection = $this->classReflection($interface);
-            } catch (LocatorException $e) {
-                if ($this->debug) {
+            }catch (LocatorException $e){
+                if ($this->debug){
                     throw $e;
                 }
 
@@ -32,7 +32,7 @@ 
                 continue;
             }
 
-            if (!$this->isTargeted($reflection, $target)) {
+            if (!$this->isTargeted($reflection, $target)){
                 continue;
             }
 
@@ -51,7 +51,7 @@ 
     {
         $interfaces = [];
 
-        foreach ($this->availableReflections() as $reflection) {
+        foreach ($this->availableReflections() as $reflection){
             $interfaces = \array_merge($interfaces, $reflection->getInterfaces());
         }
 
@@ -63,7 +63,7 @@ 
      */
     protected function isTargeted(\ReflectionClass $class, ?\ReflectionClass $target = null): bool
     {
-        if (empty($target)) {
+        if (empty($target)){
             return true;
         }
 

Braces +17 added lines, -8 removed lines

@@ -15,16 +15,22 @@ 
 
     public function getInterfaces(string|null $target = null): array
     {
-        if (!empty($target)) {
+        if (!empty($target))
+        {
             $target = new \ReflectionClass($target);
         }
 
         $result = [];
-        foreach ($this->availableInterfaces() as $interface) {
-            try {
+        foreach ($this->availableInterfaces() as $interface)
+        {
+            try
+            {
                 $reflection = $this->classReflection($interface);
-            } catch (LocatorException $e) {
-                if ($this->debug) {
+            }
+            catch (LocatorException $e)
+            {
+                if ($this->debug)
+                {
                     throw $e;
                 }
 
@@ -32,7 +38,8 @@ 
                 continue;
             }
 
-            if (!$this->isTargeted($reflection, $target)) {
+            if (!$this->isTargeted($reflection, $target))
+            {
                 continue;
             }
 
@@ -51,7 +58,8 @@ 
     {
         $interfaces = [];
 
-        foreach ($this->availableReflections() as $reflection) {
+        foreach ($this->availableReflections() as $reflection)
+        {
             $interfaces = \array_merge($interfaces, $reflection->getInterfaces());
         }
 
@@ -63,7 +71,8 @@ 
      */
     protected function isTargeted(\ReflectionClass $class, ?\ReflectionClass $target = null): bool
     {
-        if (empty($target)) {
+        if (empty($target))
+        {
             return true;
         }
 

src/Tokenizer/src/EnumLocator.php

Spacing +10 added lines, -10 removed lines

@@ -13,18 +13,18 @@ 
 {
     public const INJECTOR = EnumLocatorInjector::class;
 
-    public function getEnums(object|string|null $target = null): array
+    public function getEnums(object | string | null $target = null): array
     {
-        if (!empty($target)) {
+        if (!empty($target)){
             $target = new \ReflectionClass($target);
         }
 
         $result = [];
-        foreach ($this->availableEnums() as $enum) {
-            try {
+        foreach ($this->availableEnums() as $enum){
+            try{
                 $reflection = $this->enumReflection($enum);
-            } catch (LocatorException $e) {
-                if ($this->debug) {
+            }catch (LocatorException $e){
+                if ($this->debug){
                     throw $e;
                 }
 
@@ -32,7 +32,7 @@ 
                 continue;
             }
 
-            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface()) {
+            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface()){
                 continue;
             }
 
@@ -51,7 +51,7 @@ 
     {
         $enums = [];
 
-        foreach ($this->availableReflections() as $reflection) {
+        foreach ($this->availableReflections() as $reflection){
             $enums = \array_merge($enums, $reflection->getEnums());
         }
 
@@ -63,11 +63,11 @@ 
      */
     protected function isTargeted(\ReflectionEnum $enum, ?\ReflectionClass $target = null): bool
     {
-        if ($target === null) {
+        if ($target === null){
             return true;
         }
 
-        if (!$target->isTrait()) {
+        if (!$target->isTrait()){
             //Target is interface or class
             /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
             return $enum->isSubclassOf($target) || $enum->getName() === $target->getName();

Braces +19 added lines, -9 removed lines

@@ -15,16 +15,22 @@ 
 
     public function getEnums(object|string|null $target = null): array
     {
-        if (!empty($target)) {
+        if (!empty($target))
+        {
             $target = new \ReflectionClass($target);
         }
 
         $result = [];
-        foreach ($this->availableEnums() as $enum) {
-            try {
+        foreach ($this->availableEnums() as $enum)
+        {
+            try
+            {
                 $reflection = $this->enumReflection($enum);
-            } catch (LocatorException $e) {
-                if ($this->debug) {
+            }
+            catch (LocatorException $e)
+            {
+                if ($this->debug)
+                {
                     throw $e;
                 }
 
@@ -32,7 +38,8 @@ 
                 continue;
             }
 
-            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface()) {
+            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface())
+            {
                 continue;
             }
 
@@ -51,7 +58,8 @@ 
     {
         $enums = [];
 
-        foreach ($this->availableReflections() as $reflection) {
+        foreach ($this->availableReflections() as $reflection)
+        {
             $enums = \array_merge($enums, $reflection->getEnums());
         }
 
@@ -63,11 +71,13 @@ 
      */
     protected function isTargeted(\ReflectionEnum $enum, ?\ReflectionClass $target = null): bool
     {
-        if ($target === null) {
+        if ($target === null)
+        {
             return true;
         }
 
-        if (!$target->isTrait()) {
+        if (!$target->isTrait())
+        {
             //Target is interface or class
             /** @psalm-suppress RedundantCondition https://github.com/vimeo/psalm/issues/9489 */
             return $enum->isSubclassOf($target) || $enum->getName() === $target->getName();

src/Tokenizer/src/ClassLocator.php

Spacing +10 added lines, -10 removed lines

@@ -13,18 +13,18 @@ 
 {
     public const INJECTOR = ClassLocatorInjector::class;
 
-    public function getClasses(object|string|null $target = null): array
+    public function getClasses(object | string | null $target = null): array
     {
-        if (!empty($target)) {
+        if (!empty($target)){
             $target = new \ReflectionClass($target);
         }
 
         $result = [];
-        foreach ($this->availableClasses() as $class) {
-            try {
+        foreach ($this->availableClasses() as $class){
+            try{
                 $reflection = $this->classReflection($class);
-            } catch (LocatorException $e) {
-                if ($this->debug) {
+            }catch (LocatorException $e){
+                if ($this->debug){
                     throw $e;
                 }
 
@@ -32,7 +32,7 @@ 
                 continue;
             }
 
-            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface()) {
+            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface()){
                 continue;
             }
 
@@ -51,7 +51,7 @@ 
     {
         $classes = [];
 
-        foreach ($this->availableReflections() as $reflection) {
+        foreach ($this->availableReflections() as $reflection){
             $classes = \array_merge($classes, $reflection->getClasses());
         }
 
@@ -63,11 +63,11 @@ 
      */
     protected function isTargeted(\ReflectionClass $class, ?\ReflectionClass $target = null): bool
     {
-        if (empty($target)) {
+        if (empty($target)){
             return true;
         }
 
-        if (!$target->isTrait()) {
+        if (!$target->isTrait()){
             //Target is interface or class
             return $class->isSubclassOf($target) || $class->getName() === $target->getName();
         }

Braces +19 added lines, -9 removed lines

@@ -15,16 +15,22 @@ 
 
     public function getClasses(object|string|null $target = null): array
     {
-        if (!empty($target)) {
+        if (!empty($target))
+        {
             $target = new \ReflectionClass($target);
         }
 
         $result = [];
-        foreach ($this->availableClasses() as $class) {
-            try {
+        foreach ($this->availableClasses() as $class)
+        {
+            try
+            {
                 $reflection = $this->classReflection($class);
-            } catch (LocatorException $e) {
-                if ($this->debug) {
+            }
+            catch (LocatorException $e)
+            {
+                if ($this->debug)
+                {
                     throw $e;
                 }
 
@@ -32,7 +38,8 @@ 
                 continue;
             }
 
-            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface()) {
+            if (!$this->isTargeted($reflection, $target) || $reflection->isInterface())
+            {
                 continue;
             }
 
@@ -51,7 +58,8 @@ 
     {
         $classes = [];
 
-        foreach ($this->availableReflections() as $reflection) {
+        foreach ($this->availableReflections() as $reflection)
+        {
             $classes = \array_merge($classes, $reflection->getClasses());
         }
 
@@ -63,11 +71,13 @@ 
      */
     protected function isTargeted(\ReflectionClass $class, ?\ReflectionClass $target = null): bool
     {
-        if (empty($target)) {
+        if (empty($target))
+        {
             return true;
         }
 
-        if (!$target->isTrait()) {
+        if (!$target->isTrait())
+        {
             //Target is interface or class
             return $class->isSubclassOf($target) || $class->getName() === $target->getName();
         }

src/Filters/src/Exception/SetterException.php

Spacing +2 added lines, -2 removed lines

@@ -9,8 +9,8 @@
     public function __construct(?\Throwable $previous = null, ?string $message = null)
     {
         parent::__construct(
-            message: $message ?? 'Unable to set value. The given data was invalid.',
-            previous: $previous,
+            message : $message ?? 'Unable to set value. The given data was invalid.',
+            previous : $previous,
         );
     }
 }

src/Stempler/tests/Transform/BaseTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
 {
     protected function compile(string $source, array $visitors = [], ?LoaderInterface $loader = null)
     {
-        if ($loader === null) {
+        if ($loader === null){
             $loader = new StringLoader();
             $loader->set('root', $source);
         }
@@ -62,11 +62,11 @@ 
         $builder->getCompiler()->addRenderer(new DynamicRenderer(new DirectiveGroup()));
         $builder->getCompiler()->addRenderer(new HTMLRenderer());
 
-        foreach ($this->getVisitors() as $visitor) {
+        foreach ($this->getVisitors() as $visitor){
             $builder->addVisitor($visitor);
         }
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $builder->addVisitor($visitor);
         }
 
@@ -80,6 +80,6 @@ 
 
     protected function getFixtureLoader(): LoaderInterface
     {
-        return new DirectoryLoader(__DIR__ . '/../fixtures');
+        return new DirectoryLoader(__DIR__.'/../fixtures');
     }
 }

Braces +6 added lines, -3 removed lines

@@ -27,7 +27,8 @@ 
 {
     protected function compile(string $source, array $visitors = [], ?LoaderInterface $loader = null)
     {
-        if ($loader === null) {
+        if ($loader === null)
+        {
             $loader = new StringLoader();
             $loader->set('root', $source);
         }
@@ -62,11 +63,13 @@ 
         $builder->getCompiler()->addRenderer(new DynamicRenderer(new DirectiveGroup()));
         $builder->getCompiler()->addRenderer(new HTMLRenderer());
 
-        foreach ($this->getVisitors() as $visitor) {
+        foreach ($this->getVisitors() as $visitor)
+        {
             $builder->addVisitor($visitor);
         }
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor)
+        {
             $builder->addVisitor($visitor);
         }
 

src/Stempler/src/Compiler/Result.php

Spacing +7 added lines, -7 removed lines

@@ -24,22 +24,22 @@ 
 
     public function withinContext(?Context $ctx, callable $body): void
     {
-        if ($ctx === null || $ctx->getPath() === null) {
+        if ($ctx === null || $ctx->getPath() === null){
             $body($this);
             return;
         }
 
-        try {
+        try{
             $this->parent = Location::fromContext($ctx, $this->parent);
             $body($this);
-        } finally {
+        }finally{
             $this->parent = $this->parent->parent;
         }
     }
 
     public function push(string $content, ?Context $ctx = null): void
     {
-        if ($ctx !== null && $ctx->getPath() !== null) {
+        if ($ctx !== null && $ctx->getPath() !== null){
             $this->locations[\strlen($this->content)] = Location::fromContext($ctx, $this->parent);
         }
 
@@ -59,8 +59,8 @@ 
         $paths = [];
 
         // We can scan top level only
-        foreach ($this->locations as $loc) {
-            if (!\in_array($loc->path, $paths, true)) {
+        foreach ($this->locations as $loc){
+            if (!\in_array($loc->path, $paths, true)){
                 $paths[] = $loc->path;
             }
         }
@@ -75,7 +75,7 @@ 
     {
         $locations = [];
 
-        foreach ($this->locations as $offset => $location) {
+        foreach ($this->locations as $offset => $location){
             $locations[$offset] = $location;
         }
 

Braces +15 added lines, -7 removed lines

@@ -24,22 +24,27 @@ 
 
     public function withinContext(?Context $ctx, callable $body): void
     {
-        if ($ctx === null || $ctx->getPath() === null) {
+        if ($ctx === null || $ctx->getPath() === null)
+        {
             $body($this);
             return;
         }
 
-        try {
+        try
+        {
             $this->parent = Location::fromContext($ctx, $this->parent);
             $body($this);
-        } finally {
+        }
+        finally
+        {
             $this->parent = $this->parent->parent;
         }
     }
 
     public function push(string $content, ?Context $ctx = null): void
     {
-        if ($ctx !== null && $ctx->getPath() !== null) {
+        if ($ctx !== null && $ctx->getPath() !== null)
+        {
             $this->locations[\strlen($this->content)] = Location::fromContext($ctx, $this->parent);
         }
 
@@ -59,8 +64,10 @@ 
         $paths = [];
 
         // We can scan top level only
-        foreach ($this->locations as $loc) {
-            if (!\in_array($loc->path, $paths, true)) {
+        foreach ($this->locations as $loc)
+        {
+            if (!\in_array($loc->path, $paths, true))
+            {
                 $paths[] = $loc->path;
             }
         }
@@ -75,7 +82,8 @@ 
     {
         $locations = [];
 
-        foreach ($this->locations as $offset => $location) {
+        foreach ($this->locations as $offset => $location)
+        {
             $locations[$offset] = $location;
         }