spiral / framework

src/Console/tests/Configurator/Attribute/FillPropertiesTest.php

Braces +8 added lines, -4 removed lines

@@ -36,7 +36,8 @@ 
             ->method('getArgument')
             ->willReturnOnConsecutiveCalls(5, 'foo', ['foo', 'bar'], 0.5, true);
 
-        $command = new #[AsCommand('foo')] class extends Command {
+        $command = new #[AsCommand('foo')] class extends Command
+        {
             #[Argument]
             public int $intVal;
 
@@ -75,7 +76,8 @@ 
             ->expects($this->never())
             ->method('getArgument');
 
-        $command = new #[AsCommand('foo')] class extends Command {
+        $command = new #[AsCommand('foo')] class extends Command
+        {
             #[Argument]
             public int $arg;
         };
@@ -98,7 +100,8 @@ 
             ->method('getOption')
             ->willReturnOnConsecutiveCalls(5, 'foo', ['foo', 'bar'], 0.5, true, true);
 
-        $command = new #[AsCommand('foo')] class extends Command {
+        $command = new #[AsCommand('foo')] class extends Command
+        {
             #[Option(mode: InputOption::VALUE_REQUIRED)]
             public int $intVal;
 
@@ -141,7 +144,8 @@ 
             ->expects($this->never())
             ->method('getOption');
 
-        $command = new #[AsCommand('foo')] class extends Command {
+        $command = new #[AsCommand('foo')] class extends Command
+        {
             #[Option(mode: InputOption::VALUE_REQUIRED)]
             public int $option;
         };

src/Console/tests/Configurator/Attribute/OptionsTest.php

Braces +28 added lines, -14 removed lines

@@ -26,7 +26,8 @@ 
     public function testRequired(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_REQUIRED)]
                 private int $option;
             }
@@ -44,7 +45,8 @@ 
     public function testDefaultValue(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_OPTIONAL)]
                 private string $option = 'some';
             }
@@ -62,7 +64,8 @@ 
     public function testNullable(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_OPTIONAL)]
                 private ?string $option;
             }
@@ -80,7 +83,8 @@ 
     public function testWithName(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(name: 'customName', mode: InputOption::VALUE_OPTIONAL)]
                 private ?string $option;
             }
@@ -98,7 +102,8 @@ 
     public function testWithShortcut(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(shortcut: 't', mode: InputOption::VALUE_OPTIONAL)]
                 private ?string $option;
             }
@@ -116,7 +121,8 @@ 
     public function testWithDescription(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(description: 'Some description', mode: InputOption::VALUE_OPTIONAL)]
                 private ?string $option;
             }
@@ -134,7 +140,8 @@ 
     public function testModeOptionalArray(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY)]
                 private array $option = [];
             }
@@ -152,7 +159,8 @@ 
     public function testModeRequiredArray(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY)]
                 private array $option = [];
             }
@@ -170,7 +178,8 @@ 
     public function testModeValueNegatable(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_NEGATABLE)]
                 private bool $option;
             }
@@ -188,7 +197,8 @@ 
     public function testModeValueNone(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_NONE)]
                 private bool $option;
             }
@@ -206,7 +216,8 @@ 
     public function testWithSuggestedValue(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_REQUIRED, suggestedValues: [1, 0])]
                 private int $option;
             }
@@ -232,7 +243,8 @@ 
     public function testUnionTypeWithBuiltInAndNot(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option(mode: InputOption::VALUE_OPTIONAL)]
                 private int|\stdClass $option;
             }
@@ -251,7 +263,8 @@ 
     {
         $this->expectException(ConfiguratorException::class);
         $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option]
                 private \stdClass $option;
             }
@@ -262,7 +275,8 @@ 
     {
         $this->expectException(ConfiguratorException::class);
         $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Option]
                 private \stdClass&\Traversable $option;
             }

src/Console/tests/Configurator/Attribute/ArgumentsTest.php

Braces +26 added lines, -13 removed lines

@@ -25,7 +25,8 @@ 
     public function testRequired(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private int $arg;
             }
@@ -41,7 +42,8 @@ 
     public function testWithDefaultValue(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private int $arg = 1;
             }
@@ -57,7 +59,8 @@ 
     public function testNullable(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private ?int $arg;
             }
@@ -73,7 +76,8 @@ 
     public function testWithName(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument(name: 'customName')]
                 private int $arg;
             }
@@ -89,7 +93,8 @@ 
     public function testWithDescription(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument(description: 'Some description')]
                 private int $arg;
             }
@@ -105,7 +110,8 @@ 
     public function testWithSuggestedValue(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument(suggestedValues: [1, 0])]
                 private int $arg;
             }
@@ -129,7 +135,8 @@ 
     public function testArrayRequired(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private array $arg;
             }
@@ -145,7 +152,8 @@ 
     public function testArrayNotRequired(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private ?array $arg;
             }
@@ -161,7 +169,8 @@ 
     public function testArrayShouldBeLast(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private ?array $arg;
 
@@ -180,7 +189,8 @@ 
     public function testUnionTypeWithBuiltInAndNot(): void
     {
         $result = $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private int|\stdClass $arg;
             }
@@ -197,7 +207,8 @@ 
     {
         $this->expectException(ConfiguratorException::class);
         $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private \stdClass $arg;
             }
@@ -208,7 +219,8 @@ 
     {
         $this->expectException(ConfiguratorException::class);
         $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private \stdClass&\Traversable $arg;
             }
@@ -219,7 +231,8 @@ 
     {
         $this->expectException(ConfiguratorException::class);
         $this->parser->parse(new \ReflectionClass(
-            new #[AsCommand(name: 'foo')] class {
+            new #[AsCommand(name: 'foo')] class
+            {
                 #[Argument]
                 private array $arg;
 

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

Braces +44 added lines, -22 removed lines

@@ -49,13 +49,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)
@@ -63,13 +66,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)
@@ -82,9 +88,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;
             }
 
@@ -107,19 +115,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;
         }
 
@@ -129,16 +140,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!'
                     );
@@ -164,7 +179,8 @@ 
     {
         $type = $property->hasType() ? $property->getType() : null;
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType)
+        {
             return $value;
         }
 
@@ -180,7 +196,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())
             );
@@ -188,19 +205,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/Configurator/DefaultConfigurator.php

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/AttributeConfigurator.php

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/Console/src/Configurator/Configurator.php

Braces +4 added lines, -2 removed lines

@@ -21,8 +21,10 @@
 
     public function configure(Command $command, \ReflectionClass $reflection): void
     {
-        foreach ($this->configurators as $configurator) {
-            if ($configurator->canConfigure($command, $reflection)) {
+        foreach ($this->configurators as $configurator)
+        {
+            if ($configurator->canConfigure($command, $reflection))
+            {
                 $configurator->configure($command, $reflection);
                 return;
             }

src/Console/src/Configurator/SignatureConfigurator.php

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/PromptArguments.php

Braces +16 added lines, -8 removed lines

@@ -23,8 +23,10 @@ 
     {
         $io = new SymfonyStyle($input, $output);
 
-        foreach ($command->getDefinition()->getArguments() as $argument) {
-            if ($argument->isRequired() && $input->getArgument($argument->getName()) === null) {
+        foreach ($command->getDefinition()->getArguments() as $argument)
+        {
+            if ($argument->isRequired() && $input->getArgument($argument->getName()) === null)
+            {
                 $input->setArgument(
                     $argument->getName(),
                     $io->ask($this->getQuestion($command, $argument))
@@ -37,25 +39,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;
             }
         }