spiral / framework

src/Bridge/Stempler/src/Visitor/FormatHTML.php

Spacing +24 added lines, -24 removed lines

@@ -30,30 +30,30 @@ 
 
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
     {
-        if (!$node instanceof Template && !$node instanceof Block && !$node instanceof Tag) {
+        if (!$node instanceof Template && !$node instanceof Block && !$node instanceof Tag){
             return null;
         }
 
-        if ($node instanceof Tag && \in_array($node->name, self::EXCLUDE)) {
+        if ($node instanceof Tag && \in_array($node->name, self::EXCLUDE)){
             // raw nodes
             return null;
         }
 
         $level = $this->getLevel($ctx);
-        if ($level === null) {
+        if ($level === null){
             // not available in some contexts
             return null;
         }
 
-        foreach ($node->nodes as $i => $child) {
-            if (!$child instanceof Raw) {
+        foreach ($node->nodes as $i => $child){
+            if (!$child instanceof Raw){
                 continue;
             }
 
             $position = self::BETWEEN_TAGS;
-            if (!isset($node->nodes[$i + 1])) {
+            if (!isset($node->nodes[$i + 1])){
                 $position = self::BEFORE_CLOSE;
-            } elseif ($node->nodes[$i + 1] instanceof PHP) {
+            } elseif ($node->nodes[$i + 1] instanceof PHP){
                 $position = self::BEFORE_PHP;
             }
 
@@ -74,7 +74,7 @@ 
 
     private function indentContent(string $content, int $level, int $position = self::BETWEEN_TAGS): string
     {
-        if (!\str_contains($content, "\n")) {
+        if (!\str_contains($content, "\n")){
             // no need to do anything
             return $content;
         }
@@ -82,40 +82,40 @@ 
         // we have to apply special rules to the first and the last lines
         $lines = \explode("\n", $content);
 
-        foreach ($lines as $i => $line) {
-            if (\trim($line) === '' && $i !== 0) {
+        foreach ($lines as $i => $line){
+            if (\trim($line) === '' && $i !== 0){
                 unset($lines[$i]);
             }
         }
 
         $lines = \array_values($lines);
-        if ($lines === []) {
+        if ($lines === []){
             $lines[] = '';
         }
 
         $result = '';
-        foreach ($lines as $i => $line) {
-            if (\trim($line) !== '') {
+        foreach ($lines as $i => $line){
+            if (\trim($line) !== ''){
                 $line = $i === 0 ? \rtrim($line) : \trim($line);
             }
 
-            if ($i !== (\count($lines) - 1)) {
-                $result .= $line . "\n" . \str_repeat(self::INDENT, $level);
+            if ($i !== (\count($lines) - 1)){
+                $result .= $line."\n".\str_repeat(self::INDENT, $level);
                 continue;
             }
 
             // working with last line
-            if ($position === self::BEFORE_PHP) {
-                $result .= $line . "\n";
+            if ($position === self::BEFORE_PHP){
+                $result .= $line."\n";
                 break;
             }
 
-            if ($position === self::BEFORE_CLOSE) {
-                $result .= $line . "\n" . \str_repeat(self::INDENT, max($level - 1, 0));
+            if ($position === self::BEFORE_CLOSE){
+                $result .= $line."\n".\str_repeat(self::INDENT, max($level - 1, 0));
                 break;
             }
 
-            $result .= $line . "\n" . \str_repeat(self::INDENT, $level);
+            $result .= $line."\n".\str_repeat(self::INDENT, $level);
         }
 
         return $result;
@@ -124,12 +124,12 @@ 
     private function getLevel(VisitorContext $ctx): ?int
     {
         $level = 0;
-        foreach ($ctx->getScope() as $node) {
-            if ($node instanceof Attr) {
+        foreach ($ctx->getScope() as $node){
+            if ($node instanceof Attr){
                 return null;
             }
 
-            if ($node instanceof Block || $node instanceof Template) {
+            if ($node instanceof Block || $node instanceof Template){
                 continue;
             }
 
@@ -148,7 +148,7 @@ 
      */
     private function normalizeEndings(string $string, bool $joinMultiple = true): string
     {
-        if (!$joinMultiple) {
+        if (!$joinMultiple){
             return \str_replace("\r\n", "\n", $string);
         }
 

Braces +41 added lines, -20 removed lines

@@ -30,30 +30,38 @@ 
 
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
     {
-        if (!$node instanceof Template && !$node instanceof Block && !$node instanceof Tag) {
+        if (!$node instanceof Template && !$node instanceof Block && !$node instanceof Tag)
+        {
             return null;
         }
 
-        if ($node instanceof Tag && \in_array($node->name, self::EXCLUDE)) {
+        if ($node instanceof Tag && \in_array($node->name, self::EXCLUDE))
+        {
             // raw nodes
             return null;
         }
 
         $level = $this->getLevel($ctx);
-        if ($level === null) {
+        if ($level === null)
+        {
             // not available in some contexts
             return null;
         }
 
-        foreach ($node->nodes as $i => $child) {
-            if (!$child instanceof Raw) {
+        foreach ($node->nodes as $i => $child)
+        {
+            if (!$child instanceof Raw)
+            {
                 continue;
             }
 
             $position = self::BETWEEN_TAGS;
-            if (!isset($node->nodes[$i + 1])) {
+            if (!isset($node->nodes[$i + 1]))
+            {
                 $position = self::BEFORE_CLOSE;
-            } elseif ($node->nodes[$i + 1] instanceof PHP) {
+            }
+            elseif ($node->nodes[$i + 1] instanceof PHP)
+            {
                 $position = self::BEFORE_PHP;
             }
 
@@ -74,7 +82,8 @@ 
 
     private function indentContent(string $content, int $level, int $position = self::BETWEEN_TAGS): string
     {
-        if (!\str_contains($content, "\n")) {
+        if (!\str_contains($content, "\n"))
+        {
             // no need to do anything
             return $content;
         }
@@ -82,35 +91,43 @@ 
         // we have to apply special rules to the first and the last lines
         $lines = \explode("\n", $content);
 
-        foreach ($lines as $i => $line) {
-            if (\trim($line) === '' && $i !== 0) {
+        foreach ($lines as $i => $line)
+        {
+            if (\trim($line) === '' && $i !== 0)
+            {
                 unset($lines[$i]);
             }
         }
 
         $lines = \array_values($lines);
-        if ($lines === []) {
+        if ($lines === [])
+        {
             $lines[] = '';
         }
 
         $result = '';
-        foreach ($lines as $i => $line) {
-            if (\trim($line) !== '') {
+        foreach ($lines as $i => $line)
+        {
+            if (\trim($line) !== '')
+            {
                 $line = $i === 0 ? \rtrim($line) : \trim($line);
             }
 
-            if ($i !== (\count($lines) - 1)) {
+            if ($i !== (\count($lines) - 1))
+            {
                 $result .= $line . "\n" . \str_repeat(self::INDENT, $level);
                 continue;
             }
 
             // working with last line
-            if ($position === self::BEFORE_PHP) {
+            if ($position === self::BEFORE_PHP)
+            {
                 $result .= $line . "\n";
                 break;
             }
 
-            if ($position === self::BEFORE_CLOSE) {
+            if ($position === self::BEFORE_CLOSE)
+            {
                 $result .= $line . "\n" . \str_repeat(self::INDENT, max($level - 1, 0));
                 break;
             }
@@ -124,12 +141,15 @@ 
     private function getLevel(VisitorContext $ctx): ?int
     {
         $level = 0;
-        foreach ($ctx->getScope() as $node) {
-            if ($node instanceof Attr) {
+        foreach ($ctx->getScope() as $node)
+        {
+            if ($node instanceof Attr)
+            {
                 return null;
             }
 
-            if ($node instanceof Block || $node instanceof Template) {
+            if ($node instanceof Block || $node instanceof Template)
+            {
                 continue;
             }
 
@@ -148,7 +168,8 @@ 
      */
     private function normalizeEndings(string $string, bool $joinMultiple = true): string
     {
-        if (!$joinMultiple) {
+        if (!$joinMultiple)
+        {
             return \str_replace("\r\n", "\n", $string);
         }
 

src/Bridge/Stempler/src/Visitor/FlattenNodes.php

Spacing +6 added lines, -6 removed lines

@@ -22,14 +22,14 @@ 
 {
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
     {
-        if (!$node instanceof Tag) {
+        if (!$node instanceof Tag){
             return null;
         }
 
         $flatten = [];
-        foreach ($node->nodes as $child) {
-            if ($child instanceof Block || $child instanceof Template || $child instanceof Aggregate) {
-                foreach ($child->nodes as $childNode) {
+        foreach ($node->nodes as $child){
+            if ($child instanceof Block || $child instanceof Template || $child instanceof Aggregate){
+                foreach ($child->nodes as $childNode){
                     $flatten[] = $childNode;
                 }
                 continue;
@@ -51,12 +51,12 @@ 
     private function mergeRaw(array $nodes): array
     {
         $result = [];
-        foreach ($nodes as $node) {
+        foreach ($nodes as $node){
             if (
                 $node instanceof Raw
                 && isset($result[\count($result) - 1])
                 && $result[\count($result) - 1] instanceof Raw
-            ) {
+            ){
                 $result[\count($result) - 1]->content .= $node->content;
                 continue;
             }

Braces +10 added lines, -5 removed lines

@@ -22,14 +22,18 @@ 
 {
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
     {
-        if (!$node instanceof Tag) {
+        if (!$node instanceof Tag)
+        {
             return null;
         }
 
         $flatten = [];
-        foreach ($node->nodes as $child) {
-            if ($child instanceof Block || $child instanceof Template || $child instanceof Aggregate) {
-                foreach ($child->nodes as $childNode) {
+        foreach ($node->nodes as $child)
+        {
+            if ($child instanceof Block || $child instanceof Template || $child instanceof Aggregate)
+            {
+                foreach ($child->nodes as $childNode)
+                {
                     $flatten[] = $childNode;
                 }
                 continue;
@@ -51,7 +55,8 @@ 
     private function mergeRaw(array $nodes): array
     {
         $result = [];
-        foreach ($nodes as $node) {
+        foreach ($nodes as $node)
+        {
             if (
                 $node instanceof Raw
                 && isset($result[\count($result) - 1])

src/Bridge/Monolog/tests/RotateHandlerTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $finalizer->shouldReceive('addFinalizer')->once();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;

Braces +2 added lines, -1 removed lines

@@ -40,7 +40,8 @@
         $finalizer->shouldReceive('addFinalizer')->once();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;

src/Bridge/Monolog/tests/LoggerTest.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     public function testLoggerShouldBeReset()
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;

Braces +2 added lines, -1 removed lines

@@ -25,7 +25,8 @@
     public function testLoggerShouldBeReset()
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;

src/Bridge/Monolog/tests/FactoryTest.php

Braces +6 added lines, -3 removed lines

@@ -54,7 +54,8 @@ 
         $logger = $factory->getLogger();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;
@@ -80,7 +81,8 @@ 
     public function testFinalizerShouldResetDefaultLogger()
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;
@@ -120,7 +122,8 @@ 
     public function testFinalizerShouldNotResetLoggerWhenApplicationTerminating()
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
         $logger = $factory->getLogger();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;
@@ -80,7 +80,7 @@ 
     public function testFinalizerShouldResetDefaultLogger()
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;
@@ -120,7 +120,7 @@ 
     public function testFinalizerShouldNotResetLoggerWhenApplicationTerminating()
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;

src/Bridge/Monolog/tests/TraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $finalizer->shouldReceive('addFinalizer')->once();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;

Braces +2 added lines, -1 removed lines

@@ -40,7 +40,8 @@
         $finalizer->shouldReceive('addFinalizer')->once();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;

src/Bridge/Monolog/src/Config/MonologConfig.php

Spacing +12 added lines, -12 removed lines

@@ -35,18 +35,18 @@ 
      */
     public function getHandlers(string $channel): \Generator
     {
-        if (empty($this->config['handlers'][$channel])) {
+        if (empty($this->config['handlers'][$channel])){
             return;
         }
 
-        foreach ($this->config['handlers'][$channel] as $handler) {
-            if (\is_object($handler) && !$handler instanceof Autowire) {
+        foreach ($this->config['handlers'][$channel] as $handler){
+            if (\is_object($handler) && !$handler instanceof Autowire){
                 yield $handler;
                 continue;
             }
 
             $wire = $this->wire($handler);
-            if (\is_null($wire)) {
+            if (\is_null($wire)){
                 throw new ConfigException(\sprintf('Invalid handler definition for channel `%s`.', $channel));
             }
 
@@ -56,18 +56,18 @@ 
 
     public function getProcessors(string $channel): \Generator
     {
-        if (empty($this->config['processors'][$channel])) {
+        if (empty($this->config['processors'][$channel])){
             return;
         }
 
-        foreach ($this->config['processors'][$channel] as $processor) {
-            if (\is_object($processor) && !$processor instanceof Autowire) {
+        foreach ($this->config['processors'][$channel] as $processor){
+            if (\is_object($processor) && !$processor instanceof Autowire){
                 yield $processor;
                 continue;
             }
 
             $wire = $this->wire($processor);
-            if (\is_null($wire)) {
+            if (\is_null($wire)){
                 throw new ConfigException(\sprintf('Invalid processor definition for channel `%s`.', $channel));
             }
 
@@ -75,17 +75,17 @@ 
         }
     }
 
-    private function wire(Autowire|string|array $definition): ?Autowire
+    private function wire(Autowire | string | array $definition): ?Autowire
     {
-        if ($definition instanceof Autowire) {
+        if ($definition instanceof Autowire){
             return $definition;
         }
 
-        if (\is_string($definition)) {
+        if (\is_string($definition)){
             return new Autowire($definition);
         }
 
-        if (isset($definition['class'])) {
+        if (isset($definition['class'])){
             return new Autowire($definition['class'], $definition['options'] ?? []);
         }
 

Braces +22 added lines, -11 removed lines

@@ -35,18 +35,22 @@ 
      */
     public function getHandlers(string $channel): \Generator
     {
-        if (empty($this->config['handlers'][$channel])) {
+        if (empty($this->config['handlers'][$channel]))
+        {
             return;
         }
 
-        foreach ($this->config['handlers'][$channel] as $handler) {
-            if (\is_object($handler) && !$handler instanceof Autowire) {
+        foreach ($this->config['handlers'][$channel] as $handler)
+        {
+            if (\is_object($handler) && !$handler instanceof Autowire)
+            {
                 yield $handler;
                 continue;
             }
 
             $wire = $this->wire($handler);
-            if (\is_null($wire)) {
+            if (\is_null($wire))
+            {
                 throw new ConfigException(\sprintf('Invalid handler definition for channel `%s`.', $channel));
             }
 
@@ -56,18 +60,22 @@ 
 
     public function getProcessors(string $channel): \Generator
     {
-        if (empty($this->config['processors'][$channel])) {
+        if (empty($this->config['processors'][$channel]))
+        {
             return;
         }
 
-        foreach ($this->config['processors'][$channel] as $processor) {
-            if (\is_object($processor) && !$processor instanceof Autowire) {
+        foreach ($this->config['processors'][$channel] as $processor)
+        {
+            if (\is_object($processor) && !$processor instanceof Autowire)
+            {
                 yield $processor;
                 continue;
             }
 
             $wire = $this->wire($processor);
-            if (\is_null($wire)) {
+            if (\is_null($wire))
+            {
                 throw new ConfigException(\sprintf('Invalid processor definition for channel `%s`.', $channel));
             }
 
@@ -77,15 +85,18 @@ 
 
     private function wire(Autowire|string|array $definition): ?Autowire
     {
-        if ($definition instanceof Autowire) {
+        if ($definition instanceof Autowire)
+        {
             return $definition;
         }
 
-        if (\is_string($definition)) {
+        if (\is_string($definition))
+        {
             return new Autowire($definition);
         }
 
-        if (isset($definition['class'])) {
+        if (isset($definition['class']))
+        {
             return new Autowire($definition['class'], $definition['options'] ?? []);
         }
 

src/Bridge/Monolog/src/LogFactory.php

Spacing +13 added lines, -13 removed lines

@@ -27,7 +27,7 @@ 
         private readonly MonologConfig $config,
         ListenerRegistryInterface $listenerRegistry,
         private readonly FactoryInterface $factory
-    ) {
+    ){
         $this->eventHandler = new EventHandler($listenerRegistry, $config->getEventLevel());
     }
 
@@ -35,8 +35,8 @@ 
     {
         $default = $this->config->getDefault();
 
-        if ($channel === null || $channel === $default) {
-            if ($this->default !== null) {
+        if ($channel === null || $channel === $default){
+            if ($this->default !== null){
                 // we should use only one default logger per system
                 return $this->default;
             }
@@ -63,7 +63,7 @@ 
 
     public function reset(): void
     {
-        if ($this->default instanceof ResettableInterface) {
+        if ($this->default instanceof ResettableInterface){
             $this->default->reset();
         }
     }
@@ -79,15 +79,15 @@ 
         // always include default handler
         $handlers = [];
 
-        foreach ($this->config->getHandlers($channel) as $handler) {
-            if (!$handler instanceof Autowire) {
+        foreach ($this->config->getHandlers($channel) as $handler){
+            if (!$handler instanceof Autowire){
                 $handlers[] = $handler;
                 continue;
             }
 
-            try {
+            try{
                 $handlers[] = $handler->resolve($this->factory);
-            } catch (ContainerExceptionInterface $e) {
+            }catch (ContainerExceptionInterface $e){
                 throw new ConfigException($e->getMessage(), $e->getCode(), $e);
             }
         }
@@ -105,20 +105,20 @@ 
     protected function getProcessors(string $channel): array
     {
         $processors = [];
-        foreach ($this->config->getProcessors($channel) as $processor) {
-            if (!$processor instanceof Autowire) {
+        foreach ($this->config->getProcessors($channel) as $processor){
+            if (!$processor instanceof Autowire){
                 $processors[] = $processor;
                 continue;
             }
 
-            try {
+            try{
                 $processors[] = $processor->resolve($this->factory);
-            } catch (ContainerExceptionInterface $e) {
+            }catch (ContainerExceptionInterface $e){
                 throw new ConfigException($e->getMessage(), $e->getCode(), $e);
             }
         }
 
-        if ($processors === []) {
+        if ($processors === []){
             $processors[] = new PsrLogMessageProcessor();
         }
 

Braces +26 added lines, -12 removed lines

@@ -35,8 +35,10 @@ 
     {
         $default = $this->config->getDefault();
 
-        if ($channel === null || $channel === $default) {
-            if ($this->default !== null) {
+        if ($channel === null || $channel === $default)
+        {
+            if ($this->default !== null)
+            {
                 // we should use only one default logger per system
                 return $this->default;
             }
@@ -63,7 +65,8 @@ 
 
     public function reset(): void
     {
-        if ($this->default instanceof ResettableInterface) {
+        if ($this->default instanceof ResettableInterface)
+        {
             $this->default->reset();
         }
     }
@@ -79,15 +82,20 @@ 
         // always include default handler
         $handlers = [];
 
-        foreach ($this->config->getHandlers($channel) as $handler) {
-            if (!$handler instanceof Autowire) {
+        foreach ($this->config->getHandlers($channel) as $handler)
+        {
+            if (!$handler instanceof Autowire)
+            {
                 $handlers[] = $handler;
                 continue;
             }
 
-            try {
+            try
+            {
                 $handlers[] = $handler->resolve($this->factory);
-            } catch (ContainerExceptionInterface $e) {
+            }
+            catch (ContainerExceptionInterface $e)
+            {
                 throw new ConfigException($e->getMessage(), $e->getCode(), $e);
             }
         }
@@ -105,20 +113,26 @@ 
     protected function getProcessors(string $channel): array
     {
         $processors = [];
-        foreach ($this->config->getProcessors($channel) as $processor) {
-            if (!$processor instanceof Autowire) {
+        foreach ($this->config->getProcessors($channel) as $processor)
+        {
+            if (!$processor instanceof Autowire)
+            {
                 $processors[] = $processor;
                 continue;
             }
 
-            try {
+            try
+            {
                 $processors[] = $processor->resolve($this->factory);
-            } catch (ContainerExceptionInterface $e) {
+            }
+            catch (ContainerExceptionInterface $e)
+            {
                 throw new ConfigException($e->getMessage(), $e->getCode(), $e);
             }
         }
 
-        if ($processors === []) {
+        if ($processors === [])
+        {
             $processors[] = new PsrLogMessageProcessor();
         }
 

src/Bridge/Monolog/src/EventHandler.php

Braces +2 added lines, -1 removed lines

@@ -29,7 +29,8 @@
             $record['context']
         );
 
-        foreach ($this->listenerRegistry->getListeners() as $listener) {
+        foreach ($this->listenerRegistry->getListeners() as $listener)
+        {
             $listener($e);
         }
 

Spacing +3 added lines, -3 removed lines

@@ -16,11 +16,11 @@ 
         private readonly ListenerRegistryInterface $listenerRegistry,
         int $level = Logger::DEBUG,
         bool $bubble = true
-    ) {
+    ){
         parent::__construct($level, $bubble);
     }
 
-    public function handle(array|LogRecord $record): bool
+    public function handle(array | LogRecord $record): bool
     {
         $e = new LogEvent(
             $record['datetime'],
@@ -30,7 +30,7 @@ 
             $record['context']
         );
 
-        foreach ($this->listenerRegistry->getListeners() as $listener) {
+        foreach ($this->listenerRegistry->getListeners() as $listener){
             $listener($e);
         }