spiral / framework

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

Spacing +44 added lines, -44 removed lines

@@ -25,7 +25,7 @@ 
 {
     public function __construct(
         private readonly ReaderInterface $reader = new AttributeReader()
-    ) {
+    ){
     }
 
     public function hasCommandAttribute(\ReflectionClass $reflection): bool
@@ -38,7 +38,7 @@ 
     {
         $attribute = $this->reader->firstClassMetadata($reflection, AsCommand::class);
 
-        if ($attribute === null) {
+        if ($attribute === null){
             $attribute = $reflection->getAttributes(SymfonyAsCommand::class)[0]->newInstance();
         }
 
@@ -57,13 +57,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)
@@ -71,16 +71,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())){
                 $value = $this->typecast($input->getOption($attribute->name ?? $property->getName()), $property);
 
-                if ($value !== null || $this->getPropertyType($property)->allowsNull()) {
+                if ($value !== null || $this->getPropertyType($property)->allowsNull()){
                     $property->setValue($command, $value);
                 }
             }
@@ -100,20 +100,20 @@ 
                 ? $reflection->getMethod('__invoke')
                 : null);
 
-        if ($method === null) {
+        if ($method === null){
             return $reflection;
         }
 
         $parameter = null;
-        foreach ($method->getParameters() as $param) {
+        foreach ($method->getParameters() as $param){
             $attribute = $this->reader->firstParameterMetadata($param, AsInput::class);
-            if ($attribute !== null) {
+            if ($attribute !== null){
                 $parameter = $param;
                 break;
             }
         }
 
-        if ($parameter === null) {
+        if ($parameter === null){
             return $reflection;
         }
 
@@ -124,9 +124,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;
             }
 
@@ -144,24 +144,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;
         }
 
@@ -171,21 +171,21 @@ 
     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);
             $mode = $attribute->mode;
 
-            if ($mode === null) {
+            if ($mode === null){
                 $mode = $this->guessOptionMode($type, $property);
             }
 
-            if ($mode === InputOption::VALUE_NONE || $mode === InputOption::VALUE_NEGATABLE) {
-                if ($type->getName() !== 'bool') {
+            if ($mode === InputOption::VALUE_NONE || $mode === InputOption::VALUE_NEGATABLE){
+                if ($type->getName() !== 'bool'){
                     throw new ConfiguratorException(
                         'Options properties with mode `VALUE_NONE` or `VALUE_NEGATABLE` must be bool!'
                     );
@@ -198,9 +198,9 @@ 
                 name: $attribute->name ?? $property->getName(),
                 shortcut: $attribute->shortcut,
                 mode: $mode,
-                description: (string) $attribute->description,
+                description: (string)$attribute->description,
                 default: $hasDefaultValue ? $property->getDefaultValue() : null,
-                suggestedValues: $attribute->suggestedValues
+                suggestedValues : $attribute->suggestedValues
             );
         }
 
@@ -211,37 +211,37 @@ 
     {
         $type = $property->hasType() ? $property->getType() : null;
 
-        if (!$type instanceof \ReflectionNamedType || $value === null) {
+        if (!$type instanceof \ReflectionNamedType || $value === null){
             return $value;
         }
 
-        if (!$type->isBuiltin() && \enum_exists($type->getName())) {
-            try {
+        if (!$type->isBuiltin() && \enum_exists($type->getName())){
+            try{
                 /** @var class-string<\BackedEnum> $enum */
                 $enum = $type->getName();
 
                 return $enum::from($value);
-            } catch (\Throwable) {
+            }catch (\Throwable){
                 throw new ConfiguratorException(\sprintf('Wrong option value. Allowed options: `%s`.', \implode(
                     '`, `',
-                    \array_map(static fn (\BackedEnum $item): string => (string) $item->value, $enum::cases())
+                    \array_map(static fn (\BackedEnum $item): string => (string)$item->value, $enum::cases())
                 )));
             }
         }
 
         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())
             );
@@ -249,23 +249,23 @@ 
 
         $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 instanceof \ReflectionNamedType && $type->isBuiltin()) {
+        if ($type instanceof \ReflectionUnionType){
+            foreach ($type->getTypes() as $type){
+                if ($type instanceof \ReflectionNamedType && $type->isBuiltin()){
                     return $type;
                 }
             }
         }
 
-        if ($type instanceof \ReflectionNamedType && !$type->isBuiltin() && \enum_exists($type->getName())) {
+        if ($type instanceof \ReflectionNamedType && !$type->isBuiltin() && \enum_exists($type->getName())){
             return $type;
         }
 
-        if ($type instanceof \ReflectionNamedType && $type->isBuiltin() && $type->getName() !== 'object') {
+        if ($type instanceof \ReflectionNamedType && $type->isBuiltin() && $type->getName() !== 'object'){
             return $type;
         }
 

Braces +67 added lines, -33 removed lines

@@ -38,7 +38,8 @@ 
     {
         $attribute = $this->reader->firstClassMetadata($reflection, AsCommand::class);
 
-        if ($attribute === null) {
+        if ($attribute === null)
+        {
             $attribute = $reflection->getAttributes(SymfonyAsCommand::class)[0]->newInstance();
         }
 
@@ -57,13 +58,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)
@@ -71,16 +75,20 @@ 
             }
         }
 
-        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()))
+            {
                 $value = $this->typecast($input->getOption($attribute->name ?? $property->getName()), $property);
 
-                if ($value !== null || $this->getPropertyType($property)->allowsNull()) {
+                if ($value !== null || $this->getPropertyType($property)->allowsNull())
+                {
                     $property->setValue($command, $value);
                 }
             }
@@ -100,20 +108,24 @@ 
                 ? $reflection->getMethod('__invoke')
                 : null);
 
-        if ($method === null) {
+        if ($method === null)
+        {
             return $reflection;
         }
 
         $parameter = null;
-        foreach ($method->getParameters() as $param) {
+        foreach ($method->getParameters() as $param)
+        {
             $attribute = $this->reader->firstParameterMetadata($param, AsInput::class);
-            if ($attribute !== null) {
+            if ($attribute !== null)
+            {
                 $parameter = $param;
                 break;
             }
         }
 
-        if ($parameter === null) {
+        if ($parameter === null)
+        {
             return $reflection;
         }
 
@@ -124,9 +136,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;
             }
 
@@ -149,19 +163,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;
         }
 
@@ -171,21 +188,26 @@ 
     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);
             $mode = $attribute->mode;
 
-            if ($mode === null) {
+            if ($mode === null)
+            {
                 $mode = $this->guessOptionMode($type, $property);
             }
 
-            if ($mode === InputOption::VALUE_NONE || $mode === InputOption::VALUE_NEGATABLE) {
-                if ($type->getName() !== 'bool') {
+            if ($mode === InputOption::VALUE_NONE || $mode === InputOption::VALUE_NEGATABLE)
+            {
+                if ($type->getName() !== 'bool')
+                {
                     throw new ConfiguratorException(
                         'Options properties with mode `VALUE_NONE` or `VALUE_NEGATABLE` must be bool!'
                     );
@@ -211,17 +233,22 @@ 
     {
         $type = $property->hasType() ? $property->getType() : null;
 
-        if (!$type instanceof \ReflectionNamedType || $value === null) {
+        if (!$type instanceof \ReflectionNamedType || $value === null)
+        {
             return $value;
         }
 
-        if (!$type->isBuiltin() && \enum_exists($type->getName())) {
-            try {
+        if (!$type->isBuiltin() && \enum_exists($type->getName()))
+        {
+            try
+            {
                 /** @var class-string<\BackedEnum> $enum */
                 $enum = $type->getName();
 
                 return $enum::from($value);
-            } catch (\Throwable) {
+            }
+            catch (\Throwable)
+            {
                 throw new ConfiguratorException(\sprintf('Wrong option value. Allowed options: `%s`.', \implode(
                     '`, `',
                     \array_map(static fn (\BackedEnum $item): string => (string) $item->value, $enum::cases())
@@ -241,7 +268,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())
             );
@@ -249,23 +277,29 @@ 
 
         $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 instanceof \ReflectionNamedType && $type->isBuiltin()) {
+        if ($type instanceof \ReflectionUnionType)
+        {
+            foreach ($type->getTypes() as $type)
+            {
+                if ($type instanceof \ReflectionNamedType && $type->isBuiltin())
+                {
                     return $type;
                 }
             }
         }
 
-        if ($type instanceof \ReflectionNamedType && !$type->isBuiltin() && \enum_exists($type->getName())) {
+        if ($type instanceof \ReflectionNamedType && !$type->isBuiltin() && \enum_exists($type->getName()))
+        {
             return $type;
         }
 
-        if ($type instanceof \ReflectionNamedType && $type->isBuiltin() && $type->getName() !== 'object') {
+        if ($type instanceof \ReflectionNamedType && $type->isBuiltin() && $type->getName() !== 'object')
+        {
             return $type;
         }