minor #2077 [CI] Update php-cs-fixer config (smnandre)

This PR was squashed before being merged into the 2.x branch. Discussion ---------- [CI] Update php-cs-fixer config * Update phh-cs-fixer version * Follow symfony/symfony configuration (#57941) * Apply new rules Commits ------- 09597b3 [CI] Update php-cs-fixer config
symfony · Aug 20, 2024 · 09cc7d0 · 09cc7d0
2 parents 70c68d8 + 09597b3
commit 09cc7d0
Show file tree

Hide file tree

Showing 14 changed files with 82 additions and 89 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -29,8 +29,6 @@
         '@Symfony' => true,
         '@Symfony:risky' => true,
         'header_comment' => ['header' => $fileHeaderComment],
-        // TODO: Remove once the "compiler_optimized" set includes "sprintf"
-        'native_function_invocation' => ['include' => ['@compiler_optimized', 'sprintf'], 'scope' => 'namespaced', 'strict' => true],
         'trailing_comma_in_multiline' => ['elements' => ['arrays', 'match', 'parameters']],
     ])
     ->setRiskyAllowed(true)

diff --git a/composer.json b/composer.json
@@ -7,6 +7,6 @@
     "require-dev": {
         "symfony/filesystem": "^5.2|^6.0|^7.0",
         "symfony/finder": "^5.4|^6.0|^7.0",
-        "php-cs-fixer/shim": "^3.13"
+        "php-cs-fixer/shim": "^3.62"
     }
 }
diff --git a/src/LiveComponent/tests/Integration/LiveComponentHydratorTest.php b/src/LiveComponent/tests/Integration/LiveComponentHydratorTest.php
diff --git a/src/LiveComponent/tests/Unit/Attribute/AsLiveComponentTest.php b/src/LiveComponent/tests/Unit/Attribute/AsLiveComponentTest.php
@@ -29,7 +29,7 @@ final class AsLiveComponentTest extends TestCase
     public function testPreDehydrateMethodsAreOrderedByPriority(): void
     {
         $hooks = AsLiveComponent::preDehydrateMethods(
-            new class() {
+            new class {
                 #[PreDehydrate(priority: -10)]
                 public function hook1()
                 {
@@ -56,7 +56,7 @@ public function hook3()
     public function testPostHydrateMethodsAreOrderedByPriority(): void
     {
         $hooks = AsLiveComponent::postHydrateMethods(
-            new class() {
+            new class {
                 #[PostHydrate(priority: -10)]
                 public function hook1()
                 {
@@ -83,7 +83,7 @@ public function hook3()
     public function testPreMountHooksAreOrderedByPriority(): void
     {
         $hooks = AsLiveComponent::preReRenderMethods(
-            new class() {
+            new class {
                 #[PreReRender(priority: -10)]
                 public function hook1()
                 {

diff --git a/src/LiveComponent/tests/Unit/Attribute/LivePropTest.php b/src/LiveComponent/tests/Unit/Attribute/LivePropTest.php
@@ -33,14 +33,14 @@ public function testDehydrateWithMethod(): void
 
     public function testCanCallCalculateFieldNameAsString(): void
     {
-        $component = new class() {};
+        $component = new class {};
 
         $this->assertSame('field', (new LiveProp(false, null, null, false, [], 'field'))->calculateFieldName($component, 'fallback'));
     }
 
     public function testCanCallCalculateFieldNameAsMethod(): void
     {
-        $component = new class() {
+        $component = new class {
             public function fieldName(): string
             {
                 return 'foo';
@@ -52,7 +52,7 @@ public function fieldName(): string
 
     public function testCanCallCalculateFieldNameWhenNotSet(): void
     {
-        $component = new class() {};
+        $component = new class {};
 
         $this->assertSame('fallback', (new LiveProp())->calculateFieldName($component, 'fallback'));
     }

diff --git a/src/Map/tests/MapTest.php b/src/Map/tests/MapTest.php
@@ -83,7 +83,7 @@ public function testWithMaximumConfiguration(): void
             ->center(new Point(48.8566, 2.3522))
             ->zoom(6)
             ->fitBoundsToMarkers()
-            ->options(new class() implements MapOptionsInterface {
+            ->options(new class implements MapOptionsInterface {
                 public function toArray(): array
                 {
                     return [

diff --git a/src/Turbo/src/TurboBundle.php b/src/Turbo/src/TurboBundle.php
@@ -34,7 +34,7 @@ public function build(ContainerBuilder $container): void
     {
         parent::build($container);
 
-        $container->addCompilerPass(new class() implements CompilerPassInterface {
+        $container->addCompilerPass(new class implements CompilerPassInterface {
             public function process(ContainerBuilder $container): void
             {
                 if (!$container->hasDefinition('turbo.broadcaster.imux')) {

diff --git a/src/TwigComponent/src/Attribute/AsTwigComponent.php b/src/TwigComponent/src/Attribute/AsTwigComponent.php
@@ -78,8 +78,6 @@ public function serviceConfig(): array
     /**
      * @param object|class-string $component
      *
-     * @return ?\ReflectionMethod
-     *
      * @internal
      */
     public static function mountMethod(object|string $component): ?\ReflectionMethod

diff --git a/src/TwigComponent/tests/Integration/ComponentFactoryTest.php b/src/TwigComponent/tests/Integration/ComponentFactoryTest.php
@@ -94,7 +94,7 @@ public function testExceptionThrownIfRequiredMountParameterIsMissingFromPassedDa
 
     public function testStringableObjectCanBePassedToComponent(): void
     {
-        $attributes = $this->factory()->create('component_a', ['propB' => 'B', 'data-item-id-param' => new class() {
+        $attributes = $this->factory()->create('component_a', ['propB' => 'B', 'data-item-id-param' => new class {
             public function __toString(): string
             {
                 return 'test';

diff --git a/src/TwigComponent/tests/Unit/Attribute/AsTwigComponentTest.php b/src/TwigComponent/tests/Unit/Attribute/AsTwigComponentTest.php
@@ -24,7 +24,7 @@ final class AsTwigComponentTest extends TestCase
     public function testPreMountHooksAreOrderedByPriority(): void
     {
         $hooks = AsTwigComponent::preMountMethods(
-            new class() {
+            new class {
                 #[PreMount(priority: -10)]
                 public function hook1()
                 {
@@ -51,7 +51,7 @@ public function hook3()
     public function testPostMountHooksAreOrderedByPriority(): void
     {
         $hooks = AsTwigComponent::postMountMethods(
-            new class() {
+            new class {
                 #[PostMount(priority: -10)]
                 public function hook1()
                 {

diff --git a/src/TwigComponent/tests/Unit/ComponentAttributesTest.php b/src/TwigComponent/tests/Unit/ComponentAttributesTest.php
@@ -27,7 +27,7 @@ public function testCanConvertToString(): void
     {
         $attributes = new ComponentAttributes([
             'class' => 'foo',
-            'style' => new class() {
+            'style' => new class {
                 public function __toString(): string
                 {
                     return 'color:black;';
@@ -216,7 +216,7 @@ public function testRenderSingleAttribute(): void
     public function testRenderingSingleAttributeExcludesFromString(): void
     {
         $attributes = new ComponentAttributes([
-            'attr1' => new class() {
+            'attr1' => new class {
                 public function __toString(): string
                 {
                     return 'value1';

diff --git a/src/TwigComponent/tests/Unit/ComputedPropertiesProxyTest.php b/src/TwigComponent/tests/Unit/ComputedPropertiesProxyTest.php
@@ -21,7 +21,7 @@ final class ComputedPropertiesProxyTest extends TestCase
 {
     public function testProxyCachesGetMethodReturns(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {
+        $proxy = new ComputedPropertiesProxy(new class {
             private int $count = 0;
 
             public function getCount(): int
@@ -37,7 +37,7 @@ public function getCount(): int
 
     public function testProxyCachesIsMethodReturns(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {
+        $proxy = new ComputedPropertiesProxy(new class {
             private int $count = 0;
 
             public function isCount(): int
@@ -53,7 +53,7 @@ public function isCount(): int
 
     public function testProxyCachesHasMethodReturns(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {
+        $proxy = new ComputedPropertiesProxy(new class {
             private int $count = 0;
 
             public function hasCount(): int
@@ -69,7 +69,7 @@ public function hasCount(): int
 
     public function testCanProxyPublicProperties(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {
+        $proxy = new ComputedPropertiesProxy(new class {
             public $foo = 'bar';
         });
 
@@ -78,7 +78,7 @@ public function testCanProxyPublicProperties(): void
 
     public function testCanProxyArrayAccess(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() implements \ArrayAccess {
+        $proxy = new ComputedPropertiesProxy(new class implements \ArrayAccess {
             private $array = ['foo' => 'bar'];
 
             public function offsetExists(mixed $offset): bool
@@ -105,7 +105,7 @@ public function offsetUnset(mixed $offset): void
 
     public function testCannotProxyMethodsThatDoNotExist(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {});
+        $proxy = new ComputedPropertiesProxy(new class {});
 
         $this->expectException(\InvalidArgumentException::class);
 
@@ -114,7 +114,7 @@ public function testCannotProxyMethodsThatDoNotExist(): void
 
     public function testCannotPassArgumentsToProxiedMethods(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {});
+        $proxy = new ComputedPropertiesProxy(new class {});
 
         $this->expectException(\InvalidArgumentException::class);
 
@@ -123,7 +123,7 @@ public function testCannotPassArgumentsToProxiedMethods(): void
 
     public function testCannotProxyMethodsWithRequiredArguments(): void
     {
-        $proxy = new ComputedPropertiesProxy(new class() {
+        $proxy = new ComputedPropertiesProxy(new class {
             public function getValue(int $value): int
             {
                 return $value;

diff --git a/src/TwigComponent/tests/Unit/DependencyInjection/TwigComponentExtensionTest.php b/src/TwigComponent/tests/Unit/DependencyInjection/TwigComponentExtensionTest.php
@@ -98,7 +98,7 @@ private function createContainer()
             'kernel.debug' => true,
             'kernel.project_dir' => __DIR__,
             'kernel.bundles' => [
-                'TwigBundle' => new class() {},
+                'TwigBundle' => new class {},
                 'TwigComponentBundle' => TwigComponentBundle::class,
             ],
         ]));

diff --git a/ux.symfony.com/.php-cs-fixer.dist.php b/ux.symfony.com/.php-cs-fixer.dist.php
@@ -30,9 +30,6 @@
         '@Symfony' => true,
         '@Symfony:risky' => true,
         'header_comment' => ['header' => $fileHeaderComment],
-        'nullable_type_declaration' => true,
-        // TODO: Remove once the "compiler_optimized" set includes "sprintf"
-        'native_function_invocation' => ['include' => ['@compiler_optimized', 'sprintf'], 'scope' => 'namespaced', 'strict' => true],
         'trailing_comma_in_multiline' => ['elements' => ['arrays', 'match', 'parameters']],
     ])
     ->setRiskyAllowed(true)