spiral / framework

src/Console/src/Configurator/Attribute/Parser.php

Spacing +33 added lines, -33 removed lines

@@ -24,7 +24,7 @@ 
 {
     public function __construct(
         private readonly ReaderInterface $reader = new AttributeReader()
-    ) {
+    ){
     }
 
     public function hasCommandAttribute(\ReflectionClass $reflection): bool
@@ -37,7 +37,7 @@ 
     {
         $attribute = $this->reader->firstClassMetadata($reflection, AsCommand::class);
 
-        if ($attribute === null) {
+        if ($attribute === null){
             $attribute = $reflection->getAttributes(SymfonyAsCommand::class)[0]->newInstance();
         }
 
@@ -54,13 +54,13 @@ 
     {
         $reflection = new \ReflectionClass($command);
 
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property){
             $attribute = $this->reader->firstPropertyMetadata($property, Argument::class);
-            if ($attribute === null) {
+            if ($attribute === null){
                 continue;
             }
 
-            if ($input->hasArgument($attribute->name ?? $property->getName())) {
+            if ($input->hasArgument($attribute->name ?? $property->getName())){
                 $property->setValue(
                     $command,
                     $this->typecast($input->getArgument($attribute->name ?? $property->getName()), $property)
@@ -68,13 +68,13 @@ 
             }
         }
 
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property){
             $attribute = $this->reader->firstPropertyMetadata($property, Option::class);
-            if ($attribute === null) {
+            if ($attribute === null){
                 continue;
             }
 
-            if ($input->hasOption($attribute->name ?? $property->getName())) {
+            if ($input->hasOption($attribute->name ?? $property->getName())){
                 $property->setValue(
                     $command,
                     $this->typecast($input->getOption($attribute->name ?? $property->getName()), $property)
@@ -87,9 +87,9 @@ 
     {
         $result = [];
         $arrayArgument = null;
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property){
             $attribute = $this->reader->firstPropertyMetadata($property, Argument::class);
-            if ($attribute === null) {
+            if ($attribute === null){
                 continue;
             }
 
@@ -107,24 +107,24 @@ 
             $argument = new InputArgument(
                 name: $attribute->name ?? $property->getName(),
                 mode: $mode,
-                description: (string) $attribute->description,
+                description: (string)$attribute->description,
                 default: $property->hasDefaultValue() ? $property->getDefaultValue() : null,
-                suggestedValues: $attribute->suggestedValues
+                suggestedValues : $attribute->suggestedValues
             );
 
-            if ($arrayArgument !== null && $isArray) {
+            if ($arrayArgument !== null && $isArray){
                 throw new ConfiguratorException('There must be only one array argument!');
             }
 
             // It must be used at the end of the argument list.
-            if ($isArray) {
+            if ($isArray){
                 $arrayArgument = $argument;
                 continue;
             }
             $result[] = $argument;
         }
 
-        if ($arrayArgument !== null) {
+        if ($arrayArgument !== null){
             $result[] = $arrayArgument;
         }
 
@@ -134,16 +134,16 @@ 
     private function parseOptions(\ReflectionClass $reflection): array
     {
         $result = [];
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property){
             $attribute = $this->reader->firstPropertyMetadata($property, Option::class);
-            if ($attribute === null) {
+            if ($attribute === null){
                 continue;
             }
 
             $type = $this->getPropertyType($property);
 
-            if ($attribute->mode === InputOption::VALUE_NONE || $attribute->mode === InputOption::VALUE_NEGATABLE) {
-                if ($type->getName() !== 'bool') {
+            if ($attribute->mode === InputOption::VALUE_NONE || $attribute->mode === InputOption::VALUE_NEGATABLE){
+                if ($type->getName() !== 'bool'){
                     throw new ConfiguratorException(
                         'Options properties with mode `VALUE_NONE` or `VALUE_NEGATABLE` must be bool!'
                     );
@@ -156,9 +156,9 @@ 
                 name: $attribute->name ?? $property->getName(),
                 shortcut: $attribute->shortcut,
                 mode: $attribute->mode,
-                description: (string) $attribute->description,
+                description: (string)$attribute->description,
                 default: $hasDefaultValue ? $property->getDefaultValue() : null,
-                suggestedValues: $attribute->suggestedValues
+                suggestedValues : $attribute->suggestedValues
             );
         }
 
@@ -169,23 +169,23 @@ 
     {
         $type = $property->hasType() ? $property->getType() : null;
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType){
             return $value;
         }
 
         return match ($type->getName()) {
-            'int' => (int) $value,
-            'string' => (string) $value,
-            'bool' => (bool) $value,
-            'float' => (float) $value,
-            'array' => (array) $value,
+            'int' => (int)$value,
+            'string' => (string)$value,
+            'bool' => (bool)$value,
+            'float' => (float)$value,
+            'array' => (array)$value,
             default => $value
         };
     }
 
     private function getPropertyType(\ReflectionProperty $property): \ReflectionNamedType
     {
-        if (!$property->hasType()) {
+        if (!$property->hasType()){
             throw new ConfiguratorException(
                 \sprintf('Please, specify the type for the `%s` property!', $property->getName())
             );
@@ -193,19 +193,19 @@ 
 
         $type = $property->getType();
 
-        if ($type instanceof \ReflectionIntersectionType) {
+        if ($type instanceof \ReflectionIntersectionType){
             throw new ConfiguratorException(\sprintf('Invalid type for the `%s` property.', $property->getName()));
         }
 
-        if ($type instanceof \ReflectionUnionType) {
-            foreach ($type->getTypes() as $type) {
-                if ($type->isBuiltin()) {
+        if ($type instanceof \ReflectionUnionType){
+            foreach ($type->getTypes() as $type){
+                if ($type->isBuiltin()){
                     return $type;
                 }
             }
         }
 
-        if ($type instanceof \ReflectionNamedType && $type->isBuiltin()) {
+        if ($type instanceof \ReflectionNamedType && $type->isBuiltin()){
             return $type;
         }
 

Braces +46 added lines, -23 removed lines

@@ -37,7 +37,8 @@ 
     {
         $attribute = $this->reader->firstClassMetadata($reflection, AsCommand::class);
 
-        if ($attribute === null) {
+        if ($attribute === null)
+        {
             $attribute = $reflection->getAttributes(SymfonyAsCommand::class)[0]->newInstance();
         }
 
@@ -54,13 +55,16 @@ 
     {
         $reflection = new \ReflectionClass($command);
 
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property)
+        {
             $attribute = $this->reader->firstPropertyMetadata($property, Argument::class);
-            if ($attribute === null) {
+            if ($attribute === null)
+            {
                 continue;
             }
 
-            if ($input->hasArgument($attribute->name ?? $property->getName())) {
+            if ($input->hasArgument($attribute->name ?? $property->getName()))
+            {
                 $property->setValue(
                     $command,
                     $this->typecast($input->getArgument($attribute->name ?? $property->getName()), $property)
@@ -68,13 +72,16 @@ 
             }
         }
 
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property)
+        {
             $attribute = $this->reader->firstPropertyMetadata($property, Option::class);
-            if ($attribute === null) {
+            if ($attribute === null)
+            {
                 continue;
             }
 
-            if ($input->hasOption($attribute->name ?? $property->getName())) {
+            if ($input->hasOption($attribute->name ?? $property->getName()))
+            {
                 $property->setValue(
                     $command,
                     $this->typecast($input->getOption($attribute->name ?? $property->getName()), $property)
@@ -87,9 +94,11 @@ 
     {
         $result = [];
         $arrayArgument = null;
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property)
+        {
             $attribute = $this->reader->firstPropertyMetadata($property, Argument::class);
-            if ($attribute === null) {
+            if ($attribute === null)
+            {
                 continue;
             }
 
@@ -112,19 +121,22 @@ 
                 suggestedValues: $attribute->suggestedValues
             );
 
-            if ($arrayArgument !== null && $isArray) {
+            if ($arrayArgument !== null && $isArray)
+            {
                 throw new ConfiguratorException('There must be only one array argument!');
             }
 
             // It must be used at the end of the argument list.
-            if ($isArray) {
+            if ($isArray)
+            {
                 $arrayArgument = $argument;
                 continue;
             }
             $result[] = $argument;
         }
 
-        if ($arrayArgument !== null) {
+        if ($arrayArgument !== null)
+        {
             $result[] = $arrayArgument;
         }
 
@@ -134,16 +146,20 @@ 
     private function parseOptions(\ReflectionClass $reflection): array
     {
         $result = [];
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property)
+        {
             $attribute = $this->reader->firstPropertyMetadata($property, Option::class);
-            if ($attribute === null) {
+            if ($attribute === null)
+            {
                 continue;
             }
 
             $type = $this->getPropertyType($property);
 
-            if ($attribute->mode === InputOption::VALUE_NONE || $attribute->mode === InputOption::VALUE_NEGATABLE) {
-                if ($type->getName() !== 'bool') {
+            if ($attribute->mode === InputOption::VALUE_NONE || $attribute->mode === InputOption::VALUE_NEGATABLE)
+            {
+                if ($type->getName() !== 'bool')
+                {
                     throw new ConfiguratorException(
                         'Options properties with mode `VALUE_NONE` or `VALUE_NEGATABLE` must be bool!'
                     );
@@ -169,7 +185,8 @@ 
     {
         $type = $property->hasType() ? $property->getType() : null;
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType)
+        {
             return $value;
         }
 
@@ -185,7 +202,8 @@ 
 
     private function getPropertyType(\ReflectionProperty $property): \ReflectionNamedType
     {
-        if (!$property->hasType()) {
+        if (!$property->hasType())
+        {
             throw new ConfiguratorException(
                 \sprintf('Please, specify the type for the `%s` property!', $property->getName())
             );
@@ -193,19 +211,24 @@ 
 
         $type = $property->getType();
 
-        if ($type instanceof \ReflectionIntersectionType) {
+        if ($type instanceof \ReflectionIntersectionType)
+        {
             throw new ConfiguratorException(\sprintf('Invalid type for the `%s` property.', $property->getName()));
         }
 
-        if ($type instanceof \ReflectionUnionType) {
-            foreach ($type->getTypes() as $type) {
-                if ($type->isBuiltin()) {
+        if ($type instanceof \ReflectionUnionType)
+        {
+            foreach ($type->getTypes() as $type)
+            {
+                if ($type->isBuiltin())
+                {
                     return $type;
                 }
             }
         }
 
-        if ($type instanceof \ReflectionNamedType && $type->isBuiltin()) {
+        if ($type instanceof \ReflectionNamedType && $type->isBuiltin())
+        {
             return $type;
         }
 

src/Console/src/PromptArguments.php

Braces +18 added lines, -9 removed lines

@@ -24,13 +24,16 @@ 
     {
         $io = new SymfonyStyle($input, $output);
 
-        foreach ($command->getDefinition()->getArguments() as $argument) {
+        foreach ($command->getDefinition()->getArguments() as $argument)
+        {
             // Skip default argument "the command to execute"
-            if ($argument->getName() === 'command') {
+            if ($argument->getName() === 'command')
+            {
                 continue;
             }
 
-            if ($argument->isRequired() && $input->getArgument($argument->getName()) === null) {
+            if ($argument->isRequired() && $input->getArgument($argument->getName()) === null)
+            {
                 $input->setArgument(
                     $argument->getName(),
                     $io->ask($this->getQuestion($command, $argument))
@@ -43,25 +46,31 @@ 
     {
         $reflection = new \ReflectionClass($command);
 
-        foreach ($this->reader->getClassMetadata($reflection, Question::class) as $question) {
-            if ($question->argument === null) {
+        foreach ($this->reader->getClassMetadata($reflection, Question::class) as $question)
+        {
+            if ($question->argument === null)
+            {
                 throw new ConsoleException(
                     'When using a `Question` attribute on a console command class, the argument parameter is required.'
                 );
             }
 
-            if ($argument->getName() === $question->argument) {
+            if ($argument->getName() === $question->argument)
+            {
                 return $question->question;
             }
         }
 
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property)
+        {
             $question = $this->reader->firstPropertyMetadata($property, Question::class);
-            if ($question === null) {
+            if ($question === null)
+            {
                 continue;
             }
 
-            if ($argument->getName() === ($question->argument ?? $property->getName())) {
+            if ($argument->getName() === ($question->argument ?? $property->getName()))
+            {
                 return $question->question;
             }
         }

Spacing +10 added lines, -10 removed lines

@@ -20,20 +20,20 @@ 
 {
     public function __construct(
         private readonly ReaderInterface $reader = new AttributeReader()
-    ) {
+    ){
     }
 
     public function promptMissedArguments(Command $command, InputInterface $input, OutputInterface $output): void
     {
         $io = new SymfonyStyle($input, $output);
 
-        foreach ($command->getDefinition()->getArguments() as $argument) {
+        foreach ($command->getDefinition()->getArguments() as $argument){
             // Skip default argument "the command to execute"
-            if ($argument->getName() === 'command') {
+            if ($argument->getName() === 'command'){
                 continue;
             }
 
-            if ($argument->isRequired() && $input->getArgument($argument->getName()) === null) {
+            if ($argument->isRequired() && $input->getArgument($argument->getName()) === null){
                 $input->setArgument(
                     $argument->getName(),
                     $io->ask($this->getQuestion($command, $argument))
@@ -46,25 +46,25 @@ 
     {
         $reflection = new \ReflectionClass($command);
 
-        foreach ($this->reader->getClassMetadata($reflection, Question::class) as $question) {
-            if ($question->argument === null) {
+        foreach ($this->reader->getClassMetadata($reflection, Question::class) as $question){
+            if ($question->argument === null){
                 throw new ConsoleException(
                     'When using a `Question` attribute on a console command class, the argument parameter is required.'
                 );
             }
 
-            if ($argument->getName() === $question->argument) {
+            if ($argument->getName() === $question->argument){
                 return $question->question;
             }
         }
 
-        foreach ($reflection->getProperties() as $property) {
+        foreach ($reflection->getProperties() as $property){
             $question = $this->reader->firstPropertyMetadata($property, Question::class);
-            if ($question === null) {
+            if ($question === null){
                 continue;
             }
 
-            if ($argument->getName() === ($question->argument ?? $property->getName())) {
+            if ($argument->getName() === ($question->argument ?? $property->getName())){
                 return $question->question;
             }
         }

src/Console/src/Configurator/ConstantBasedConfigurator.php

Spacing +3 added lines, -3 removed lines

@@ -14,13 +14,13 @@
     public function configure(Command $command, \ReflectionClass $reflection): void
     {
         $command->setName($reflection->getConstant('NAME'));
-        $command->setDescription((string) $reflection->getConstant('DESCRIPTION'));
+        $command->setDescription((string)$reflection->getConstant('DESCRIPTION'));
 
-        foreach ($reflection->getMethod('defineOptions')->invoke($command) as $option) {
+        foreach ($reflection->getMethod('defineOptions')->invoke($command) as $option){
             \call_user_func_array([$command, 'addOption'], $option);
         }
 
-        foreach ($reflection->getMethod('defineArguments')->invoke($command) as $argument) {
+        foreach ($reflection->getMethod('defineArguments')->invoke($command) as $argument){
             \call_user_func_array([$command, 'addArgument'], $argument);
         }
     }

Braces +4 added lines, -2 removed lines

@@ -16,11 +16,13 @@
         $command->setName($reflection->getConstant('NAME'));
         $command->setDescription((string) $reflection->getConstant('DESCRIPTION'));
 
-        foreach ($reflection->getMethod('defineOptions')->invoke($command) as $option) {
+        foreach ($reflection->getMethod('defineOptions')->invoke($command) as $option)
+        {
             \call_user_func_array([$command, 'addOption'], $option);
         }
 
-        foreach ($reflection->getMethod('defineArguments')->invoke($command) as $argument) {
+        foreach ($reflection->getMethod('defineArguments')->invoke($command) as $argument)
+        {
             \call_user_func_array([$command, 'addArgument'], $argument);
         }
     }

src/Console/src/Configurator/SignatureBasedConfigurator.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
 {
     public function __construct(
         private readonly Parser $parser
-    ) {
+    ){
     }
 
     public function canConfigure(Command $command, \ReflectionClass $reflection): bool
@@ -24,15 +24,15 @@ 
 
     public function configure(Command $command, \ReflectionClass $reflection): void
     {
-        $result = $this->parser->parse((string) $reflection->getConstant('SIGNATURE'));
+        $result = $this->parser->parse((string)$reflection->getConstant('SIGNATURE'));
 
         $command->setName($result->name);
 
-        foreach ($result->options as $option) {
+        foreach ($result->options as $option){
             $command->getDefinition()->addOption($option);
         }
 
-        foreach ($result->arguments as $argument) {
+        foreach ($result->arguments as $argument){
             $command->getDefinition()->addArgument($argument);
         }
     }

Braces +4 added lines, -2 removed lines

@@ -28,11 +28,13 @@
 
         $command->setName($result->name);
 
-        foreach ($result->options as $option) {
+        foreach ($result->options as $option)
+        {
             $command->getDefinition()->addOption($option);
         }
 
-        foreach ($result->arguments as $argument) {
+        foreach ($result->arguments as $argument)
+        {
             $command->getDefinition()->addArgument($argument);
         }
     }

src/Console/src/Configurator/AttributeBasedConfigurator.php

Spacing +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
 {
     public function __construct(
         private readonly Parser $parser
-    ) {
+    ){
     }
 
     public function canConfigure(Command $command, \ReflectionClass $reflection): bool
@@ -27,14 +27,14 @@ 
         $result = $this->parser->parse($reflection);
 
         $command->setName($result->name);
-        $command->setDescription((string) $result->description);
-        $command->setHelp((string) $result->help);
+        $command->setDescription((string)$result->description);
+        $command->setHelp((string)$result->help);
 
-        foreach ($result->options as $option) {
+        foreach ($result->options as $option){
             $command->getDefinition()->addOption($option);
         }
 
-        foreach ($result->arguments as $argument) {
+        foreach ($result->arguments as $argument){
             $command->getDefinition()->addArgument($argument);
         }
     }

Braces +4 added lines, -2 removed lines

@@ -30,11 +30,13 @@
         $command->setDescription((string) $result->description);
         $command->setHelp((string) $result->help);
 
-        foreach ($result->options as $option) {
+        foreach ($result->options as $option)
+        {
             $command->getDefinition()->addOption($option);
         }
 
-        foreach ($result->arguments as $argument) {
+        foreach ($result->arguments as $argument)
+        {
             $command->getDefinition()->addArgument($argument);
         }
     }

src/Tokenizer/tests/Listener/ListenerInvokerTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
         $listener = \Mockery::mock(TokenizationListenerInterface::class);
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class){
             $listener->shouldReceive('listen')
                 ->once()
                 ->with($class);

Braces +2 added lines, -1 removed lines

@@ -30,7 +30,8 @@
 
         $listener = \Mockery::mock(TokenizationListenerInterface::class);
 
-        foreach ($classes as $class) {
+        foreach ($classes as $class)
+        {
             $listener->shouldReceive('listen')
                 ->once()
                 ->with($class);

src/Tokenizer/tests/Listener/ClassLocatorByDefinitionTest.php

Spacing +4 added lines, -4 removed lines

@@ -21,8 +21,8 @@ 
     use MockeryPHPUnitIntegration;
 
     private ClassLocatorByDefinition $locator;
-    private ClassesInterface|m\LegacyMockInterface|m\MockInterface $classes;
-    private ScopedClassesInterface|m\LegacyMockInterface|m\MockInterface $scopedClasses;
+    private ClassesInterface | m\LegacyMockInterface | m\MockInterface $classes;
+    private ScopedClassesInterface | m\LegacyMockInterface | m\MockInterface $scopedClasses;
 
     protected function setUp(): void
     {
@@ -106,11 +106,11 @@ 
         $definition = \iterator_to_array((new AttributesParser(new AttributeReader()))
             ->parse(new $listener))[0];
 
-        if ($definition->getScope() === null) {
+        if ($definition->getScope() === null){
             $this->classes
                 ->shouldReceive('getClasses')
                 ->andReturn($classes);
-        } else {
+        }else{
             $this->scopedClasses
                 ->shouldReceive('getScopedClasses')
                 ->with($definition->getScope())

Braces +5 added lines, -2 removed lines

@@ -106,11 +106,14 @@
         $definition = \iterator_to_array((new AttributesParser(new AttributeReader()))
             ->parse(new $listener))[0];
 
-        if ($definition->getScope() === null) {
+        if ($definition->getScope() === null)
+        {
             $this->classes
                 ->shouldReceive('getClasses')
                 ->andReturn($classes);
-        } else {
+        }
+        else
+        {
             $this->scopedClasses
                 ->shouldReceive('getScopedClasses')
                 ->with($definition->getScope())

src/Tokenizer/src/Listener/AttributesParser.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 {
     public function __construct(
         private readonly ReaderInterface $reader,
-    ) {
+    ){
     }
 
     /**
@@ -25,8 +25,8 @@ 
     {
         $listener = new \ReflectionClass($listener);
 
-        foreach ($this->reader->getClassMetadata($listener) as $attribute) {
-            if ($attribute instanceof ListenerDefinitionInterface) {
+        foreach ($this->reader->getClassMetadata($listener) as $attribute){
+            if ($attribute instanceof ListenerDefinitionInterface){
                 yield $attribute;
             }
         }

Braces +4 added lines, -2 removed lines

@@ -25,8 +25,10 @@
     {
         $listener = new \ReflectionClass($listener);
 
-        foreach ($this->reader->getClassMetadata($listener) as $attribute) {
-            if ($attribute instanceof ListenerDefinitionInterface) {
+        foreach ($this->reader->getClassMetadata($listener) as $attribute)
+        {
+            if ($attribute instanceof ListenerDefinitionInterface)
+            {
                 yield $attribute;
             }
         }

src/Tokenizer/src/Listener/ClassLocatorByDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     public function __construct(
         private readonly ClassesInterface $classes,
         private readonly ScopedClassesInterface $scopedClasses,
-    ) {
+    ){
     }
 
     /**