minor symfony#57901 Code style change in `@PER-CS2.0 affecting @Symfony` (parentheses for anonymous classes) (bonroyage)

This PR was squashed before being merged into the 7.2 branch.

Discussion
----------

Code style change in ``@PER`-CS2.0` affecting ``@Symfony`` (parentheses for anonymous classes)

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

I have created a PR (PHP-CS-Fixer/PHP-CS-Fixer#8140) in the PHP-CS-Fixer repo to bring the ``@PER`-CS2.0` ruleset in line with the specifications on the `new_with_parentheses` rule for anonymous classes. Since the ``@Symfony`` ruleset builds upon the ``@PER`-CS2.0` ruleset, they would like confirmation that the Symfony community is OK with this change affecting the Symfony ruleset as well. Should it not be, I'll push another commit there ensuring that the change does not affect ``@Symfony``.

Therefore, this PR is not meant to be merged, but function as an RFC to get your opinion and show the effect it would have when applied to the Symfony source.

Commits
-------

506e0dd Code style change in ``@PER`-CS2.0` affecting ``@Symfony`` (parentheses for anonymous classes)

Author: derrabus

.php-cs-fixer.dist.php

@@ -37,6 +37,7 @@
         'nullable_type_declaration' => true,
         'nullable_type_declaration_for_default_null_value' => true,
         'trailing_comma_in_multiline' => ['elements' => ['arrays', 'match', 'parameters']],
+        'new_with_parentheses' => ['anonymous_class' => false],
     ])
     ->setRiskyAllowed(true)
     ->setFinder(

src/Symfony/Bridge/Doctrine/Tests/Messenger/DoctrineOpenTransactionLoggerMiddlewareTest.php

@@ -29,7 +29,7 @@ class DoctrineOpenTransactionLoggerMiddlewareTest extends MiddlewareTestCase
 
     protected function setUp(): void
     {
-        $this->logger = new class() extends AbstractLogger {
+        $this->logger = new class extends AbstractLogger {
             public array $logs = [];
 
             public function log($level, $message, $context = []): void

src/Symfony/Bridge/Doctrine/Tests/Validator/Constraints/UniqueEntityValidatorTest.php

@@ -926,7 +926,7 @@ public static function resultWithEmptyIterator(): array
         $entity = new SingleIntIdEntity(1, 'foo');
 
         return [
-            [$entity, new class() implements \Iterator {
+            [$entity, new class implements \Iterator {
                 public function current(): mixed
                 {
                     return null;
@@ -950,7 +950,7 @@ public function rewind(): void
                 {
                 }
             }],
-            [$entity, new class() implements \Iterator {
+            [$entity, new class implements \Iterator {
                 public function current(): mixed
                 {
                     return false;

src/Symfony/Bridge/Twig/Extension/TranslationExtension.php

@@ -47,7 +47,7 @@ public function getTranslator(): TranslatorInterface
                 throw new \LogicException(\sprintf('You cannot use the "%s" if the Translation Contracts are not available. Try running "composer require symfony/translation".', __CLASS__));
             }
 
-            $this->translator = new class() implements TranslatorInterface {
+            $this->translator = new class implements TranslatorInterface {
                 use TranslatorTrait;
             };
         }

src/Symfony/Bridge/Twig/Tests/Extension/AbstractLayoutTestCase.php

@@ -2652,7 +2652,7 @@ public function testHelpWithTranslatableMessage()
 
     public function testHelpWithTranslatableInterface()
     {
-        $message = new class() implements TranslatableInterface {
+        $message = new class implements TranslatableInterface {
             public function trans(TranslatorInterface $translator, ?string $locale = null): string
             {
                 return $translator->trans('foo');

src/Symfony/Bundle/FrameworkBundle/Tests/CacheWarmer/ConfigBuilderCacheWarmerTest.php

@@ -189,7 +189,7 @@ public function testExtensionAddedInKernel()
         $kernel = new class($this->varDir) extends TestKernel {
             protected function build(ContainerBuilder $container): void
             {
-                $container->registerExtension(new class() extends Extension implements ConfigurationInterface {
+                $container->registerExtension(new class extends Extension implements ConfigurationInterface {
                     public function load(array $configs, ContainerBuilder $container): void
                     {
                     }
@@ -276,7 +276,7 @@ protected function build(ContainerBuilder $container): void
             {
                 /** @var TestSecurityExtension $extension */
                 $extension = $container->getExtension('test_security');
-                $extension->addAuthenticatorFactory(new class() implements TestAuthenticatorFactoryInterface {
+                $extension->addAuthenticatorFactory(new class implements TestAuthenticatorFactoryInterface {
                     public function getKey(): string
                     {
                         return 'token';
@@ -292,19 +292,19 @@ public function registerBundles(): iterable
             {
                 yield from parent::registerBundles();
 
-                yield new class() extends Bundle {
+                yield new class extends Bundle {
                     public function getContainerExtension(): ExtensionInterface
                     {
                         return new TestSecurityExtension();
                     }
                 };
 
-                yield new class() extends Bundle {
+                yield new class extends Bundle {
                     public function build(ContainerBuilder $container): void
                     {
                         /** @var TestSecurityExtension $extension */
                         $extension = $container->getExtension('test_security');
-                        $extension->addAuthenticatorFactory(new class() implements TestAuthenticatorFactoryInterface {
+                        $extension->addAuthenticatorFactory(new class implements TestAuthenticatorFactoryInterface {
                             public function getKey(): string
                             {
                                 return 'form-login';

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/ProfilerPassTest.php

@@ -66,7 +66,7 @@ public function testValidCollector()
 
     public static function provideValidCollectorWithTemplateUsingAutoconfigure(): \Generator
     {
-        yield [new class() implements TemplateAwareDataCollectorInterface {
+        yield [new class implements TemplateAwareDataCollectorInterface {
             public function collect(Request $request, Response $response, ?\Throwable $exception = null): void
             {
             }
@@ -86,7 +86,7 @@ public static function getTemplate(): string
             }
         }];
 
-        yield [new class() extends AbstractDataCollector {
+        yield [new class extends AbstractDataCollector {
             public function collect(Request $request, Response $response, ?\Throwable $exception = null): void
             {
             }

src/Symfony/Bundle/FrameworkBundle/Tests/Test/WebTestCaseTest.php

@@ -388,7 +388,7 @@ private function getRequestTester(): WebTestCase
 
     private function getTester(KernelBrowser $client): WebTestCase
     {
-        $tester = new class() extends WebTestCase {
+        $tester = new class extends WebTestCase {
             use WebTestAssertionsTrait {
                 getClient as public;
             }

src/Symfony/Bundle/SecurityBundle/Tests/DataCollector/SecurityDataCollectorTest.php

@@ -226,7 +226,7 @@ public function testCollectCollectsDecisionLogWhenStrategyIsAffirmative()
         $voter1 = new DummyVoter();
         $voter2 = new DummyVoter();
 
-        $decoratedVoter1 = new TraceableVoter($voter1, new class() implements EventDispatcherInterface {
+        $decoratedVoter1 = new TraceableVoter($voter1, new class implements EventDispatcherInterface {
             public function dispatch(object $event, ?string $eventName = null): object
             {
                 return new \stdClass();
@@ -301,7 +301,7 @@ public function testCollectCollectsDecisionLogWhenStrategyIsUnanimous()
         $voter1 = new DummyVoter();
         $voter2 = new DummyVoter();
 
-        $decoratedVoter1 = new TraceableVoter($voter1, new class() implements EventDispatcherInterface {
+        $decoratedVoter1 = new TraceableVoter($voter1, new class implements EventDispatcherInterface {
             public function dispatch(object $event, ?string $eventName = null): object
             {
                 return new \stdClass();

src/Symfony/Component/AssetMapper/Tests/AssetMapperCompilerTest.php

@@ -21,7 +21,7 @@ class AssetMapperCompilerTest extends TestCase
 {
     public function testCompile()
     {
-        $compiler1 = new class() implements AssetCompilerInterface {
+        $compiler1 = new class implements AssetCompilerInterface {
             public function supports(MappedAsset $asset): bool
             {
                 return 'css' === $asset->publicExtension;
@@ -33,7 +33,7 @@ public function compile(string $content, MappedAsset $asset, AssetMapperInterfac
             }
         };
 
-        $compiler2 = new class() implements AssetCompilerInterface {
+        $compiler2 = new class implements AssetCompilerInterface {
             public function supports(MappedAsset $asset): bool
             {
                 return 'js' === $asset->publicExtension;
@@ -45,7 +45,7 @@ public function compile(string $content, MappedAsset $asset, AssetMapperInterfac
             }
         };
 
-        $compiler3 = new class() implements AssetCompilerInterface {
+        $compiler3 = new class implements AssetCompilerInterface {
             public function supports(MappedAsset $asset): bool
             {
                 return 'js' === $asset->publicExtension;