feat: Add single_line_empty_body to @PhpCsFixer (#7266)

doc/list.rst

@@ -3008,6 +3008,8 @@ List of Available Rules
 
    Empty body of class, interface, trait, enum or function must be abbreviated as ``{}`` and placed on the same line as the previous symbol, separated by a single space.
 
+   Part of rule set `@PhpCsFixer <./ruleSets/PhpCsFixer.rst>`_
+
    `Source PhpCsFixer\\Fixer\\Basic\\SingleLineEmptyBodyFixer <./../src/Fixer/Basic/SingleLineEmptyBodyFixer.php>`_
 -  `single_line_throw <./rules/function_notation/single_line_throw.rst>`_
 

doc/ruleSets/PhpCsFixer.rst

@@ -58,6 +58,7 @@ Rules
 - `return_assignment <./../rules/return_notation/return_assignment.rst>`_
 - `self_static_accessor <./../rules/class_notation/self_static_accessor.rst>`_
 - `single_line_comment_style <./../rules/comment/single_line_comment_style.rst>`_
+- `single_line_empty_body <./../rules/basic/single_line_empty_body.rst>`_
 - `whitespace_after_comma_in_array <./../rules/array_notation/whitespace_after_comma_in_array.rst>`_ with config:
 
   ``['ensure_single_space' => true]``

doc/rules/basic/single_line_empty_body.rst

@@ -22,3 +22,11 @@ Example #1
    -{
    -}
    +) {}
+
+Rule sets
+---------
+
+The rule is part of the following rule set:
+
+- `@PhpCsFixer <./../../ruleSets/PhpCsFixer.rst>`_
+

src/Cache/NullCacheManager.php

@@ -26,7 +26,5 @@ public function needFixing(string $file, string $fileContent): bool
         return true;
     }
 
-    public function setFile(string $file, string $fileContent): void
-    {
-    }
+    public function setFile(string $file, string $fileContent): void {}
 }

src/ConfigurationException/InvalidForEnvFixerConfigurationException.php

@@ -19,6 +19,4 @@
  *
  * @internal
  */
-final class InvalidForEnvFixerConfigurationException extends InvalidFixerConfigurationException
-{
-}
+final class InvalidForEnvFixerConfigurationException extends InvalidFixerConfigurationException {}

src/ConfigurationException/RequiredFixerConfigurationException.php

@@ -19,6 +19,4 @@
  *
  * @internal
  */
-final class RequiredFixerConfigurationException extends InvalidFixerConfigurationException
-{
-}
+final class RequiredFixerConfigurationException extends InvalidFixerConfigurationException {}

src/Console/Output/Progress/NullOutput.php

@@ -21,11 +21,7 @@
  */
 final class NullOutput implements ProgressOutputInterface
 {
-    public function printLegend(): void
-    {
-    }
+    public function printLegend(): void {}
 
-    public function onFixerFileProcessed(FixerFileProcessedEvent $event): void
-    {
-    }
+    public function onFixerFileProcessed(FixerFileProcessedEvent $event): void {}
 }

src/Fixer/PhpUnit/PhpUnitTargetVersion.php

@@ -39,9 +39,7 @@ final class PhpUnitTargetVersion
     public const VERSION_8_4 = '8.4';
     public const VERSION_NEWEST = 'newest';
 
-    private function __construct()
-    {
-    }
+    private function __construct() {}
 
     public static function fulfills(string $candidate, string $target): bool
     {

src/FixerConfiguration/InvalidOptionsForEnvException.php

@@ -21,6 +21,4 @@
  *
  * @internal
  */
-final class InvalidOptionsForEnvException extends InvalidOptionsException
-{
-}
+final class InvalidOptionsForEnvException extends InvalidOptionsException {}

src/Linter/LintingException.php

@@ -21,6 +21,4 @@
  *
  * @TODO 4.0 make class "final"
  */
-class LintingException extends \RuntimeException
-{
-}
+class LintingException extends \RuntimeException {}

src/Linter/UnavailableLinterException.php

@@ -23,6 +23,4 @@
  *
  * @TODO 4.0 make class "final"
  */
-class UnavailableLinterException extends \RuntimeException
-{
-}
+class UnavailableLinterException extends \RuntimeException {}

src/PregException.php

@@ -21,6 +21,4 @@
  *
  * @internal
  */
-final class PregException extends \RuntimeException
-{
-}
+final class PregException extends \RuntimeException {}

src/RuleSet/AbstractRuleSetDescription.php

@@ -19,9 +19,7 @@
  */
 abstract class AbstractRuleSetDescription implements RuleSetDescriptionInterface
 {
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 
     public function getName(): string
     {

src/RuleSet/Sets/PhpCsFixerSet.php

@@ -115,6 +115,7 @@ public function getRules(): array
             'return_assignment' => true,
             'self_static_accessor' => true,
             'single_line_comment_style' => true,
+            'single_line_empty_body' => true,
             'single_line_throw' => false,
             'whitespace_after_comma_in_array' => ['ensure_single_space' => true],
             'yield_from_array_to_yields' => true,

src/StdinFileInfo.php

@@ -165,11 +165,7 @@ public function openFile($openMode = 'r', $useIncludePath = false, $context = nu
         throw new \BadMethodCallException(sprintf('Method "%s" is not implemented.', __METHOD__));
     }
 
-    public function setFileClass($className = null): void
-    {
-    }
+    public function setFileClass($className = null): void {}
 
-    public function setInfoClass($className = null): void
-    {
-    }
+    public function setInfoClass($className = null): void {}
 }

src/Tokenizer/CT.php

@@ -59,9 +59,7 @@ final class CT
     public const T_DYNAMIC_CLASS_CONSTANT_FETCH_CURLY_BRACE_OPEN = 10038;
     public const T_DYNAMIC_CLASS_CONSTANT_FETCH_CURLY_BRACE_CLOSE = 10039;
 
-    private function __construct()
-    {
-    }
+    private function __construct() {}
 
     /**
      * Get name for custom token.

tests/Fixtures/Integration/misc/PHP8_0.test

@@ -41,16 +41,12 @@ $country = $session?->user?->getAddress()?->country;
 class PostsController
 {
     #[Route('/api/posts/{id}', methods: ['GET'])]
-    public function get($id): void
-    {
-    }
+    public function get($id): void {}
 }
 
 $object = new #[ExampleAttribute] class() {
     #[ExampleAttribute]
-    public function foo(#[ExampleAttribute] $bar): void
-    {
-    }
+    public function foo(#[ExampleAttribute] $bar): void {}
 };
 
 // https://wiki.php.net/rfc/union_types_v2
@@ -75,22 +71,16 @@ class Number
 // https://wiki.php.net/rfc/static_return_type
 class T
 {
-    public function Foo(object $A): ?static
-    {
-    }
+    public function Foo(object $A): ?static {}
 
-    public function something(): static
-    {
-    }
+    public function something(): static {}
 }
 
 // https://wiki.php.net/rfc/variable_syntax_tweaks
 // nothing we can do
 
 // https://wiki.php.net/rfc/trailing_comma_in_parameter_list
-function foo(string $param = null): void
-{
-}
+function foo(string $param = null): void {}
 call_user_func('foo', 1);
 
 // https://wiki.php.net/rfc/trailing_comma_in_closure_use_list
@@ -101,9 +91,7 @@ $foo2 = static function () use ($bar, $baz): void { echo $bar.$baz; };
 // https://wiki.php.net/rfc/mixed_type_v2
 class T_Mixed
 {
-    public function Foo(mixed $a): mixed
-    {
-    }
+    public function Foo(mixed $a): mixed {}
 }
 
 // https://wiki.php.net/rfc/throw_expression
@@ -130,8 +118,7 @@ class Point
         protected float $y = 0.0,
         private float $z = 0.0,
         private ?string $desc = null,
-    ) {
-    }
+    ) {}
 }
 
 // https://wiki.php.net/rfc/match_expression_v2

tests/Fixtures/Integration/misc/PHP8_1.test

@@ -52,8 +52,7 @@ class TestNewWithInitialiers
 {
     public function __construct(
         private Logger $logger = new NullLogger(),
-    ) {
-    }
+    ) {}
 }
 
 --INPUT--

tests/Fixtures/Integration/misc/PHP8_2.test

@@ -25,9 +25,7 @@ enum A: string
 }
 
 #[Attr(A::B->name)]
-class E
-{
-}
+class E {}
 
 // https://wiki.php.net/rfc/null-false-standalone-types
 class FalseNull
@@ -38,8 +36,7 @@ class FalseNull
     public function __construct(
         private false $falsy = false,
         private null $nully = null
-    ) {
-    }
+    ) {}
 
     public function falsy(): false
     {
@@ -58,9 +55,7 @@ class FalseNull
     }
 }
 
-function falsyNull(null $n, false $f): null|false
-{
-}
+function falsyNull(null $n, false $f): null|false {}
 
 array_filter([], fn (null $n) => null === $n);
 array_filter([], fn (false $n) => false === $n);
@@ -72,8 +67,7 @@ class TrueType
 
     public function __construct(
         private true $havingABud = true
-    ) {
-    }
+    ) {}
 
     public function havingABud(): true
     {

tests/Fixtures/Integration/misc/issue_3879.test

@@ -12,9 +12,7 @@ Integration of fixers: align_multiline_comment and PHPDoc rules.
  *
  * @return int the number of runs needed to fix this is 2 :(
  */
-function fixMe($str)
-{
-}
+function fixMe($str) {}
 
 --INPUT--
 <?php
@@ -26,6 +24,4 @@ function fixMe($str)
 
  * @return int The number of runs needed to fix this is 2 :( .
  */
-function fixMe($str)
-{
-}
+function fixMe($str) {}

tests/RuleSet/Sets/DoctrineAnnotationSetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\DoctrineAnnotationSet
  */
-final class DoctrineAnnotationSetTest extends AbstractSetTestCase
-{
-}
+final class DoctrineAnnotationSetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PERCS1x0RiskySetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PERCS1x0RiskySet
  */
-final class PERCS1x0RiskySetTest extends AbstractSetTestCase
-{
-}
+final class PERCS1x0RiskySetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PERCS1x0SetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PERCS1x0Set
  */
-final class PERCS1x0SetTest extends AbstractSetTestCase
-{
-}
+final class PERCS1x0SetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PERRiskySetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PERRiskySet
  */
-final class PERRiskySetTest extends AbstractSetTestCase
-{
-}
+final class PERRiskySetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PERSetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PERSet
  */
-final class PERSetTest extends AbstractSetTestCase
-{
-}
+final class PERSetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP54MigrationSetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP54MigrationSet
  */
-final class PHP54MigrationSetTest extends AbstractSetTestCase
-{
-}
+final class PHP54MigrationSetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP56MigrationRiskySetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP56MigrationRiskySet
  */
-final class PHP56MigrationRiskySetTest extends AbstractSetTestCase
-{
-}
+final class PHP56MigrationRiskySetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP70MigrationRiskySetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP70MigrationRiskySet
  */
-final class PHP70MigrationRiskySetTest extends AbstractSetTestCase
-{
-}
+final class PHP70MigrationRiskySetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP70MigrationSetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP70MigrationSet
  */
-final class PHP70MigrationSetTest extends AbstractSetTestCase
-{
-}
+final class PHP70MigrationSetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP71MigrationRiskySetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP71MigrationRiskySet
  */
-final class PHP71MigrationRiskySetTest extends AbstractSetTestCase
-{
-}
+final class PHP71MigrationRiskySetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP71MigrationSetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP71MigrationSet
  */
-final class PHP71MigrationSetTest extends AbstractSetTestCase
-{
-}
+final class PHP71MigrationSetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP73MigrationSetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP73MigrationSet
  */
-final class PHP73MigrationSetTest extends AbstractSetTestCase
-{
-}
+final class PHP73MigrationSetTest extends AbstractSetTestCase {}

tests/RuleSet/Sets/PHP74MigrationRiskySetTest.php

@@ -19,6 +19,4 @@
  *
  * @covers \PhpCsFixer\RuleSet\Sets\PHP74MigrationRiskySet
  */
-final class PHP74MigrationRiskySetTest extends AbstractSetTestCase
-{
-}
+final class PHP74MigrationRiskySetTest extends AbstractSetTestCase {}