chore: CS fixes

Author: keradus

AbstractTypeExtension.php

@@ -18,9 +18,6 @@
  */
 abstract class AbstractTypeExtension implements FormTypeExtensionInterface
 {
-    /**
-     * @return void
-     */
     public function configureOptions(OptionsResolver $resolver): void
     {
     }

FormTypeExtensionInterface.php

@@ -25,9 +25,6 @@ interface FormTypeExtensionInterface
      */
     public static function getExtendedTypes(): iterable;
 
-    /**
-     * @return void
-     */
     public function configureOptions(OptionsResolver $resolver): void;
 
     /**

Tests/ChoiceList/Factory/DefaultChoiceListFactoryTest.php

@@ -159,9 +159,9 @@ public function testCreateFromChoicesGroupedTraversable()
     {
         $list = $this->factory->createListFromChoices(
             new \ArrayIterator([
-                    'Group 1' => ['A' => $this->obj1, 'B' => $this->obj2],
-                    'Group 2' => ['C' => $this->obj3, 'D' => $this->obj4],
-                ])
+                'Group 1' => ['A' => $this->obj1, 'B' => $this->obj2],
+                'Group 2' => ['C' => $this->obj3, 'D' => $this->obj4],
+            ])
         );
 
         $this->assertObjectListWithGeneratedValues($list);
@@ -941,7 +941,7 @@ private function assertFlatViewWithAttr($view)
                     'C',
                     ['attr2' => 'value2']
                 ),
-                ]
+            ]
         ), $view);
     }
 
@@ -987,7 +987,7 @@ private function assertGroupedView($view)
                     'Group 2',
                     [2 => new ChoiceView($this->obj3, '2', 'C')]
                 ),
-                ]
+            ]
         ), $view);
     }
 

Tests/ChoiceList/Loader/CallbackChoiceLoaderTest.php

@@ -69,8 +69,8 @@ public function testLoadChoicesForValuesLoadsChoiceListOnFirstCall()
     public function testLoadValuesForChoicesCastsCallbackItemsToString()
     {
         $choices = [
-           (object) ['id' => 2],
-           (object) ['id' => 3],
+            (object) ['id' => 2],
+            (object) ['id' => 3],
         ];
 
         $value = fn ($item) => $item->id;

Tests/Extension/Core/Type/ChoiceTypeTest.php

@@ -1895,8 +1895,8 @@ public function testInitializeWithEmptyChoices()
     {
         $this->assertInstanceOf(
             FormInterface::class, $this->factory->createNamed('name', static::TESTED_TYPE, null, [
-            'choices' => [],
-        ]));
+                'choices' => [],
+            ]));
     }
 
     public function testInitializeWithDefaultObjectChoice()

Tests/Extension/Core/Type/FormTypeTest.php

@@ -156,24 +156,24 @@ public function testDataClassMayBeNull()
     {
         $this->assertInstanceOf(
             FormBuilderInterface::class, $this->factory->createBuilder(static::TESTED_TYPE, null, [
-            'data_class' => null,
-        ]));
+                'data_class' => null,
+            ]));
     }
 
     public function testDataClassMayBeAbstractClass()
     {
         $this->assertInstanceOf(
             FormBuilderInterface::class, $this->factory->createBuilder(static::TESTED_TYPE, null, [
-            'data_class' => 'Symfony\Component\Form\Tests\Fixtures\AbstractAuthor',
-        ]));
+                'data_class' => 'Symfony\Component\Form\Tests\Fixtures\AbstractAuthor',
+            ]));
     }
 
     public function testDataClassMayBeInterface()
     {
         $this->assertInstanceOf(
             FormBuilderInterface::class, $this->factory->createBuilder(static::TESTED_TYPE, null, [
-            'data_class' => 'Symfony\Component\Form\Tests\Fixtures\AuthorInterface',
-        ]));
+                'data_class' => 'Symfony\Component\Form\Tests\Fixtures\AuthorInterface',
+            ]));
     }
 
     public function testDataClassMustBeValidClassOrInterface()
@@ -402,7 +402,7 @@ public function testSubformCallsSettersIfTheObjectChanged()
             // referenceCopy has a getter that returns a copy
             'referenceCopy' => [
                 'firstName' => 'Foo',
-        ],
+            ],
         ]);
 
         $this->assertEquals('Foo', $author->getReferenceCopy()->firstName);
@@ -680,8 +680,8 @@ public function testDataMapperTransformationFailedExceptionInvalidMessageIsUsed(
     public function testPassZeroLabelToView()
     {
         $view = $this->factory->create(static::TESTED_TYPE, null, [
-                'label' => '0',
-            ])
+            'label' => '0',
+        ])
             ->createView();
 
         $this->assertSame('0', $view->vars['label']);

Tests/Extension/DataCollector/FormDataCollectorTest.php

@@ -71,7 +71,7 @@ public function testBuildPreliminaryFormTree()
             ],
             'errors' => [],
             'children' => [],
-         ];
+        ];
 
         $formData = [
             'id' => 'name',
@@ -87,11 +87,11 @@ public function testBuildPreliminaryFormTree()
                 'norm' => null,
             ],
             'errors' => [],
-             'has_children_error' => false,
-             'children' => [
-                 'child' => $childFormData,
-             ],
-         ];
+            'has_children_error' => false,
+            'children' => [
+                'child' => $childFormData,
+            ],
+        ];
 
         $this->assertEquals([
             'forms' => [
@@ -102,7 +102,7 @@ public function testBuildPreliminaryFormTree()
                 spl_object_hash($this->childForm) => $childFormData,
             ],
             'nb_errors' => 0,
-         ], $this->dataCollector->getData());
+        ], $this->dataCollector->getData());
     }
 
     public function testBuildMultiplePreliminaryFormTrees()

Tests/Extension/Validator/Constraints/FormValidatorFunctionalTest.php

@@ -88,9 +88,9 @@ public function testFieldConstraintsInvalidateFormIfFieldIsSubmitted()
     public function testNonCompositeConstraintValidatedOnce()
     {
         $form = $this->formFactory->create(TextType::class, null, [
-                'constraints' => [new NotBlank(['groups' => ['foo', 'bar']])],
-                'validation_groups' => ['foo', 'bar'],
-            ]);
+            'constraints' => [new NotBlank(['groups' => ['foo', 'bar']])],
+            'validation_groups' => ['foo', 'bar'],
+        ]);
         $form->submit('');
 
         $violations = $this->validator->validate($form);

Tests/Extension/Validator/Constraints/FormValidatorTest.php

@@ -184,8 +184,8 @@ public function testDontValidateIfNoValidationGroups()
         $object = new \stdClass();
 
         $form = $this->getBuilder('name', '\stdClass', [
-                'validation_groups' => [],
-            ])
+            'validation_groups' => [],
+        ])
             ->setData($object)
             ->setCompound(true)
             ->setDataMapper(new DataMapper())
@@ -256,12 +256,12 @@ public function testDontValidateIfNotSynchronized()
         $object = new \stdClass();
 
         $form = $this->getBuilder('name', '\stdClass', [
-                'invalid_message' => 'invalid_message_key',
-                // Invalid message parameters must be supported, because the
-                // invalid message can be a translation key
-                // see https://github.com/symfony/symfony/issues/5144
-                'invalid_message_parameters' => ['{{ foo }}' => 'bar'],
-            ])
+            'invalid_message' => 'invalid_message_key',
+            // Invalid message parameters must be supported, because the
+            // invalid message can be a translation key
+            // see https://github.com/symfony/symfony/issues/5144
+            'invalid_message_parameters' => ['{{ foo }}' => 'bar'],
+        ])
             ->setData($object)
             ->addViewTransformer(new CallbackTransformer(
                 static fn ($data) => $data,
@@ -292,13 +292,13 @@ public function testAddInvalidErrorEvenIfNoValidationGroups()
         $object = new \stdClass();
 
         $form = $this->getBuilder('name', '\stdClass', [
-                'invalid_message' => 'invalid_message_key',
-                // Invalid message parameters must be supported, because the
-                // invalid message can be a translation key
-                // see https://github.com/symfony/symfony/issues/5144
-                'invalid_message_parameters' => ['{{ foo }}' => 'bar'],
-                'validation_groups' => [],
-            ])
+            'invalid_message' => 'invalid_message_key',
+            // Invalid message parameters must be supported, because the
+            // invalid message can be a translation key
+            // see https://github.com/symfony/symfony/issues/5144
+            'invalid_message_parameters' => ['{{ foo }}' => 'bar'],
+            'validation_groups' => [],
+        ])
             ->setData($object)
             ->addViewTransformer(new CallbackTransformer(
                 static fn ($data) => $data,

Tests/NativeRequestHandlerTest.php

@@ -172,8 +172,8 @@ public function testMethodOverrideHeaderIgnoredIfNotPost()
         $form = $this->createForm('param1', 'POST');
 
         $this->setRequestData('GET', [
-                'param1' => 'DATA',
-            ]);
+            'param1' => 'DATA',
+        ]);
 
         $_SERVER['HTTP_X_HTTP_METHOD_OVERRIDE'] = 'PUT';
 

Tests/SimpleFormTest.php

@@ -503,9 +503,9 @@ public function testSetDataConvertsScalarToStringIfOnlyModelTransformer()
     {
         $form = $this->getBuilder()
             ->addModelTransformer(new FixedDataTransformer([
-            '' => '',
-            1 => 23,
-        ]))
+                '' => '',
+                1 => 23,
+            ]))
             ->getForm();
 
         $form->setData(1);

Tests/Util/StringUtilTest.php

@@ -56,7 +56,7 @@ public static function spaceProvider(): array
             ['0020'],
             ['00A0'],
             ['1680'],
-//            ['180E'],
+            // ['180E'],
             ['2000'],
             ['2001'],
             ['2002'],