spiral / framework

src/Scaffolder/src/Command/FilterCommand.php

Spacing +2 added lines, -2 removed lines

@@ -33,11 +33,11 @@
     {
         $declaration = $this->createDeclaration(FilterDeclaration::class);
 
-        foreach ($this->properties as $property) {
+        foreach ($this->properties as $property){
             $declaration->addProperty($property);
         }
 
-        if ($this->useValidator) {
+        if ($this->useValidator){
             $declaration->addFilterDefinition();
         }
 

Braces +4 added lines, -2 removed lines

@@ -33,11 +33,13 @@
     {
         $declaration = $this->createDeclaration(FilterDeclaration::class);
 
-        foreach ($this->properties as $property) {
+        foreach ($this->properties as $property)
+        {
             $declaration->addProperty($property);
         }
 
-        if ($this->useValidator) {
+        if ($this->useValidator)
+        {
             $declaration->addFilterDefinition();
         }
 

src/Scaffolder/src/Declaration/FilterDeclaration.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 return new $validationClass(validationRules: [
     // Put your validation rules here
 ]);
-PHP,
+php,
             );
     }
 

Spacing +9 added lines, -9 removed lines

@@ -28,12 +28,12 @@ 
         string $name,
         ?string $comment = null,
         ?string $namespace = null,
-    ) {
+    ){
         parent::__construct($config, $name, $comment, $namespace);
 
-        try {
+        try{
             $this->validationConfig = $container->get(ValidationConfig::class);
-        } catch (\Throwable) {
+        }catch (\Throwable){
             // Validation is not configured
             $this->validationConfig = null;
         }
@@ -50,13 +50,13 @@ 
     public function addFilterDefinition(): void
     {
         $validation = $this->validationConfig?->getDefaultValidator();
-        if ($validation === null) {
+        if ($validation === null){
             throw new ValidationException(
                 'Default Validator is not configured. Read more at https://spiral.dev/docs/validation-factory'
             );
         }
 
-        if (!\class_exists($validation)) {
+        if (!\class_exists($validation)){
             throw new ValidationException(
                 \sprintf(
                     'Default Validator "%s" is not class or does not exist.',
@@ -90,7 +90,7 @@ 
         $p = $this->class->addProperty($property->getName())
             ->setType($property->getType());
 
-        foreach ($property->getAttributes() as $attribute) {
+        foreach ($property->getAttributes() as $attribute){
             $p->addAttribute($attribute->getName(), $attribute->getArguments());
         }
     }
@@ -107,12 +107,12 @@ 
     {
         $declaredType = null;
 
-        if (\str_contains($property, ':')) {
+        if (\str_contains($property, ':')){
             $segments = \explode(':', $property, 3);
 
-            if (\count($segments) === 3) {
+            if (\count($segments) === 3){
                 [$property, $source, $declaredType] = $segments;
-            } else {
+            }else{
                 [$property, $source] = $segments;
             }
         }

Braces +18 added lines, -8 removed lines

@@ -31,9 +31,12 @@ 
     ) {
         parent::__construct($config, $name, $comment, $namespace);
 
-        try {
+        try
+        {
             $this->validationConfig = $container->get(ValidationConfig::class);
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
             // Validation is not configured
             $this->validationConfig = null;
         }
@@ -50,13 +53,15 @@ 
     public function addFilterDefinition(): void
     {
         $validation = $this->validationConfig?->getDefaultValidator();
-        if ($validation === null) {
+        if ($validation === null)
+        {
             throw new ValidationException(
                 'Default Validator is not configured. Read more at https://spiral.dev/docs/validation-factory'
             );
         }
 
-        if (!\class_exists($validation)) {
+        if (!\class_exists($validation))
+        {
             throw new ValidationException(
                 \sprintf(
                     'Default Validator "%s" is not class or does not exist.',
@@ -90,7 +95,8 @@ 
         $p = $this->class->addProperty($property->getName())
             ->setType($property->getType());
 
-        foreach ($property->getAttributes() as $attribute) {
+        foreach ($property->getAttributes() as $attribute)
+        {
             $p->addAttribute($attribute->getName(), $attribute->getArguments());
         }
     }
@@ -107,12 +113,16 @@ 
     {
         $declaredType = null;
 
-        if (\str_contains($property, ':')) {
+        if (\str_contains($property, ':'))
+        {
             $segments = \explode(':', $property, 3);
 
-            if (\count($segments) === 3) {
+            if (\count($segments) === 3)
+            {
                 [$property, $source, $declaredType] = $segments;
-            } else {
+            }
+            else
+            {
                 [$property, $source] = $segments;
             }
         }

src/Scaffolder/src/Declaration/AbstractDeclaration.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
         string $name,
         ?string $comment = null,
         ?string $namespace = null,
-    ) {
+    ){
         $this->namespace = new PhpNamespace($namespace ?? $this->config->classNamespace(static::TYPE, $name));
 
         $this->class = $this->namespace

src/Scaffolder/src/Declaration/ConfigDeclaration.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
         $this->files->write(
             $filename,
             $file->render() . PHP_EOL . (new Dumper())->dump(new Literal('return [];')),
-            FilesInterface::READONLY,
+            FilesInterface::readonly,
             true,
         );
     }

Spacing +18 added lines, -18 removed lines

@@ -35,7 +35,7 @@ 
         protected ?string $comment = null,
         private readonly string $directory = '',
         ?string $namespace = null,
-    ) {
+    ){
         parent::__construct($config, $name, $comment, $namespace);
     }
 
@@ -45,8 +45,8 @@ 
         $this->class->addConstant('CONFIG', $configName)->setPublic();
 
         $filename = $this->makeConfigFilename($configName);
-        if ($reverse) {
-            if (!$this->files->exists($filename)) {
+        if ($reverse){
+            if (!$this->files->exists($filename)){
                 throw new ScaffolderException(\sprintf("Config filename %s doesn't exist", $filename));
             }
 
@@ -54,7 +54,7 @@ 
             $this->declareGetters($defaultsFromFile);
 
             $this->class->getProperty('config')->setValue($this->defaultValues->get($defaultsFromFile));
-        } elseif (!$this->files->exists($filename)) {
+        } elseif (!$this->files->exists($filename)){
             $this->touchConfigFile($filename);
         }
     }
@@ -110,7 +110,7 @@ 
 
         $this->files->write(
             $filename,
-            $file->render() . PHP_EOL . (new Dumper())->dump(new Literal('return [];')),
+            $file->render().PHP_EOL.(new Dumper())->dump(new Literal('return [];')),
             FilesInterface::READONLY,
             true,
         );
@@ -128,7 +128,7 @@ 
         $getters = [];
         $gettersByKey = [];
 
-        foreach ($defaults as $key => $value) {
+        foreach ($defaults as $key => $value){
             $key = (string)$key;
             $getter = $this->makeGetterName($key);
             $getters[] = $getter;
@@ -136,19 +136,19 @@ 
             $method = $this->class->addMethod($getter)->setPublic();
             $method->setBody(\sprintf('return $this->config[\'%s\'];', $key));
 
-            if (\is_array($value)) {
+            if (\is_array($value)){
                 $gettersByKey[] = ['key' => $key, 'value' => $value];
             }
 
             $returnTypeHint = $this->typeHints->getHint(\gettype($value));
-            if ($returnTypeHint !== null) {
+            if ($returnTypeHint !== null){
                 $method->setReturnType($returnTypeHint);
             }
         }
 
-        foreach ($gettersByKey as $item) {
+        foreach ($gettersByKey as $item){
             $method = $this->declareGettersByKey($getters, $item['key'], $item['value']);
-            if ($method !== null) {
+            if ($method !== null){
                 $getters[] = $method->getName();
             }
         }
@@ -157,30 +157,30 @@ 
     private function declareGettersByKey(array $methodNames, string $key, array $value): ?Method
     {
         //Won't create if there's less than 2 sub-items
-        if (\count($value) < 2) {
+        if (\count($value) < 2){
             return null;
         }
 
         $singularKey = $this->singularize($key);
         $name = $this->makeGetterName($singularKey);
-        if (\in_array($name, $methodNames, true)) {
+        if (\in_array($name, $methodNames, true)){
             $name = $this->makeGetterName($singularKey, 'get', 'by');
         }
 
         //Name conflict, won't merge
-        if (\in_array($name, $methodNames, true)) {
+        if (\in_array($name, $methodNames, true)){
             return null;
         }
 
         $keyType = defineArrayType(\array_keys($value), '-mixed-');
         $valueType = defineArrayType(\array_values($value), '-mixed-');
         //We need a fixed structure here
-        if ($keyType === '-mixed-' || $valueType === '-mixed-') {
+        if ($keyType === '-mixed-' || $valueType === '-mixed-'){
             return null;
         }
 
         //Won't create for associated arrays
-        if ($this->typeAnnotations->mapType($keyType) === 'int' && \array_is_list($value)) {
+        if ($this->typeAnnotations->mapType($keyType) === 'int' && \array_is_list($value)){
             return null;
         }
 
@@ -190,7 +190,7 @@ 
 
         $param = $method->addParameter($singularKey);
         $paramTypeHint = $this->typeHints->getHint($keyType);
-        if ($paramTypeHint !== null) {
+        if ($paramTypeHint !== null){
             $param->setType($paramTypeHint);
         }
 
@@ -200,13 +200,13 @@ 
     private function makeGetterName(string $name, string $prefix = 'get', string $postfix = ''): string
     {
         $chunks = [];
-        if (!empty($prefix)) {
+        if (!empty($prefix)){
             $chunks[] = $prefix;
         }
 
         $name = $this->slugify->slugify($name, ['lowercase' => false]);
         $chunks[] = \count($chunks) !== 0 ? $this->classify($name) : $name;
-        if (!empty($postfix)) {
+        if (!empty($postfix)){
             $chunks[] = \ucfirst($postfix);
         }
 

Braces +33 added lines, -16 removed lines

@@ -45,8 +45,10 @@ 
         $this->class->addConstant('CONFIG', $configName)->setPublic();
 
         $filename = $this->makeConfigFilename($configName);
-        if ($reverse) {
-            if (!$this->files->exists($filename)) {
+        if ($reverse)
+        {
+            if (!$this->files->exists($filename))
+            {
                 throw new ScaffolderException(\sprintf("Config filename %s doesn't exist", $filename));
             }
 
@@ -54,7 +56,9 @@ 
             $this->declareGetters($defaultsFromFile);
 
             $this->class->getProperty('config')->setValue($this->defaultValues->get($defaultsFromFile));
-        } elseif (!$this->files->exists($filename)) {
+        }
+        elseif (!$this->files->exists($filename))
+        {
             $this->touchConfigFile($filename);
         }
     }
@@ -128,7 +132,8 @@ 
         $getters = [];
         $gettersByKey = [];
 
-        foreach ($defaults as $key => $value) {
+        foreach ($defaults as $key => $value)
+        {
             $key = (string)$key;
             $getter = $this->makeGetterName($key);
             $getters[] = $getter;
@@ -136,19 +141,23 @@ 
             $method = $this->class->addMethod($getter)->setPublic();
             $method->setBody(\sprintf('return $this->config[\'%s\'];', $key));
 
-            if (\is_array($value)) {
+            if (\is_array($value))
+            {
                 $gettersByKey[] = ['key' => $key, 'value' => $value];
             }
 
             $returnTypeHint = $this->typeHints->getHint(\gettype($value));
-            if ($returnTypeHint !== null) {
+            if ($returnTypeHint !== null)
+            {
                 $method->setReturnType($returnTypeHint);
             }
         }
 
-        foreach ($gettersByKey as $item) {
+        foreach ($gettersByKey as $item)
+        {
             $method = $this->declareGettersByKey($getters, $item['key'], $item['value']);
-            if ($method !== null) {
+            if ($method !== null)
+            {
                 $getters[] = $method->getName();
             }
         }
@@ -157,30 +166,35 @@ 
     private function declareGettersByKey(array $methodNames, string $key, array $value): ?Method
     {
         //Won't create if there's less than 2 sub-items
-        if (\count($value) < 2) {
+        if (\count($value) < 2)
+        {
             return null;
         }
 
         $singularKey = $this->singularize($key);
         $name = $this->makeGetterName($singularKey);
-        if (\in_array($name, $methodNames, true)) {
+        if (\in_array($name, $methodNames, true))
+        {
             $name = $this->makeGetterName($singularKey, 'get', 'by');
         }
 
         //Name conflict, won't merge
-        if (\in_array($name, $methodNames, true)) {
+        if (\in_array($name, $methodNames, true))
+        {
             return null;
         }
 
         $keyType = defineArrayType(\array_keys($value), '-mixed-');
         $valueType = defineArrayType(\array_values($value), '-mixed-');
         //We need a fixed structure here
-        if ($keyType === '-mixed-' || $valueType === '-mixed-') {
+        if ($keyType === '-mixed-' || $valueType === '-mixed-')
+        {
             return null;
         }
 
         //Won't create for associated arrays
-        if ($this->typeAnnotations->mapType($keyType) === 'int' && \array_is_list($value)) {
+        if ($this->typeAnnotations->mapType($keyType) === 'int' && \array_is_list($value))
+        {
             return null;
         }
 
@@ -190,7 +204,8 @@ 
 
         $param = $method->addParameter($singularKey);
         $paramTypeHint = $this->typeHints->getHint($keyType);
-        if ($paramTypeHint !== null) {
+        if ($paramTypeHint !== null)
+        {
             $param->setType($paramTypeHint);
         }
 
@@ -200,13 +215,15 @@ 
     private function makeGetterName(string $name, string $prefix = 'get', string $postfix = ''): string
     {
         $chunks = [];
-        if (!empty($prefix)) {
+        if (!empty($prefix))
+        {
             $chunks[] = $prefix;
         }
 
         $name = $this->slugify->slugify($name, ['lowercase' => false]);
         $chunks[] = \count($chunks) !== 0 ? $this->classify($name) : $name;
-        if (!empty($postfix)) {
+        if (!empty($postfix))
+        {
             $chunks[] = \ucfirst($postfix);
         }
 

src/Queue/src/Failed/LogFailedJobHandler.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 {
     public function __construct(
         private readonly ExceptionReporterInterface $reporter
-    ) {
+    ){
     }
 
     public function handle(string $driver, string $queue, string $job, mixed $payload, \Throwable $e): void

src/Queue/src/Event/JobProcessed.php

Spacing +1 added lines, -1 removed lines

@@ -13,6 +13,6 @@
         public readonly string $id,
         public readonly mixed $payload,
         public readonly array $headers = []
-    ) {
+    ){
     }
 }

src/Queue/tests/ContainerRegistryT.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
 final class ContainerRegistryT extends TestCase
 {
-    private ContainerInterface|m\MockInterface $container;
+    private ContainerInterface | m\MockInterface $container;
     private ContainerRegistry $registry;
 
     protected function setUp(): void

src/Queue/tests/Interceptor/Consume/ErrorHandlerInterceptorTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
             $handler = m::mock(FailedJobHandlerInterface::class)
         );
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload)){
             $this->markTestIncomplete('FailedJobHandlerInterface does not support non-array payloads');
             return;
         }

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
             $handler = m::mock(FailedJobHandlerInterface::class)
         );
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload))
+        {
             $this->markTestIncomplete('FailedJobHandlerInterface does not support non-array payloads');
             return;
         }

src/Stempler/src/Node/Hidden.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     /** @param TNode[] $nodes */
     public function __construct(
         public array $nodes
-    ) {
+    ){
     }
 
     public function getIterator(): \Generator