add php-cs-fixer rule "single_line_empty_body" (qossmic#1217)

.php-cs-fixer.dist.php

@@ -12,6 +12,7 @@
         '@Symfony' => true,
         'array_syntax' => ['syntax' => 'short'],
         'array_indentation' => true,
+        'single_line_empty_body' => true,
         'global_namespace_import' => [
             'import_constants' => true,
             'import_functions' => true,

src/Contract/Analyser/PostProcessEvent.php

@@ -14,9 +14,7 @@
  */
 final class PostProcessEvent extends Event
 {
-    public function __construct(private AnalysisResult $result)
-    {
-    }
+    public function __construct(private AnalysisResult $result) {}
 
     public function getResult(): AnalysisResult
     {

src/Contract/Analyser/ProcessEvent.php

@@ -25,8 +25,7 @@ public function __construct(
         public readonly TokenReferenceInterface $dependentReference,
         public readonly array $dependentLayers,
         private AnalysisResult $result = new AnalysisResult()
-    ) {
-    }
+    ) {}
 
     public function getResult(): AnalysisResult
     {

src/Contract/Ast/AstFileAnalysedEvent.php

@@ -11,7 +11,5 @@
  */
 final class AstFileAnalysedEvent extends Event
 {
-    public function __construct(public readonly string $file)
-    {
-    }
+    public function __construct(public readonly string $file) {}
 }

src/Contract/Ast/AstFileSyntaxErrorEvent.php

@@ -14,6 +14,5 @@ final class AstFileSyntaxErrorEvent extends Event
     public function __construct(
         public readonly string $file,
         public readonly string $syntaxError
-    ) {
-    }
+    ) {}
 }

src/Contract/Ast/FileOccurrence.php

@@ -14,6 +14,5 @@ final class FileOccurrence
     public function __construct(
         public readonly string $filepath,
         public readonly int $line
-    ) {
-    }
+    ) {}
 }

src/Contract/Ast/PostCreateAstMapEvent.php

@@ -9,6 +9,4 @@
 /**
  * Event triggered after the AST map of all files has been created.
  */
-final class PostCreateAstMapEvent extends Event
-{
-}
+final class PostCreateAstMapEvent extends Event {}

src/Contract/Ast/PreCreateAstMapEvent.php

@@ -13,6 +13,5 @@ final class PreCreateAstMapEvent extends Event
 {
     public function __construct(
         public readonly int $expectedFileCount
-    ) {
-    }
+    ) {}
 }

src/Contract/Config/Collector/BoolConfig.php

@@ -15,9 +15,7 @@ final class BoolConfig extends CollectorConfig
     /** @var array<CollectorConfig> */
     private array $must = [];
 
-    private function __construct()
-    {
-    }
+    private function __construct() {}
 
     /**
      * @param array<CollectorConfig> $must

src/Contract/Config/Collector/ComposerConfig.php

@@ -15,8 +15,7 @@ final class ComposerConfig extends CollectorConfig
     private function __construct(
         private readonly string $composerPath,
         private readonly string $composerLockPath,
-    ) {
-    }
+    ) {}
 
     /**
      * @param list<string> $packages

src/Contract/Config/Collector/SuperGlobalConfig.php

@@ -14,8 +14,7 @@ final class SuperGlobalConfig extends CollectorConfig
      */
     private function __construct(
         protected array $config,
-    ) {
-    }
+    ) {}
 
     public static function create(string ...$config): self
     {

src/Contract/Config/ConfigurableCollectorConfig.php

@@ -97,8 +97,7 @@ abstract class ConfigurableCollectorConfig extends CollectorConfig
 
     final protected function __construct(
         protected string $config,
-    ) {
-    }
+    ) {}
 
     public static function create(string $config): self
     {

src/Contract/Config/Formatter/CodeclimateConfig.php

@@ -10,8 +10,7 @@ private function __construct(
         private CodeclimateLevelEnum $failure,
         private CodeclimateLevelEnum $skipped,
         private CodeclimateLevelEnum $uncovered,
-    ) {
-    }
+    ) {}
 
     public static function create(
         CodeclimateLevelEnum $failure = CodeclimateLevelEnum::BLOCKER,

src/Contract/Config/Formatter/GraphvizConfig.php

@@ -17,9 +17,7 @@ final class GraphvizConfig implements FormatterConfigInterface
     /** @var array<string, Layer[]> */
     private array $groups = [];
 
-    private function __construct()
-    {
-    }
+    private function __construct() {}
 
     public static function create(): self
     {

src/Contract/Dependency/PostEmitEvent.php

@@ -9,6 +9,4 @@
 /**
  * Event triggered after all the dependencies have been resolved.
  */
-final class PostEmitEvent extends Event
-{
-}
+final class PostEmitEvent extends Event {}

src/Contract/Dependency/PostFlattenEvent.php

@@ -11,6 +11,4 @@
  *
  * This occurs when all dependencies caused by class inheritance have been resolved.
  */
-final class PostFlattenEvent extends Event
-{
-}
+final class PostFlattenEvent extends Event {}

src/Contract/Dependency/PreEmitEvent.php

@@ -13,6 +13,5 @@ final class PreEmitEvent extends Event
 {
     public function __construct(
         public readonly string $emitterName
-    ) {
-    }
+    ) {}
 }

src/Contract/Dependency/PreFlattenEvent.php

@@ -11,6 +11,4 @@
  *
  * This occurs when all dependencies caused by class inheritance have been resolved.
  */
-final class PreFlattenEvent extends Event
-{
-}
+final class PreFlattenEvent extends Event {}

src/Contract/Layer/LayerProvider.php

@@ -12,9 +12,7 @@ final class LayerProvider
     /**
      * @param array<string, list<string>> $allowedLayers source layer -> target layers
      */
-    public function __construct(private readonly array $allowedLayers)
-    {
-    }
+    public function __construct(private readonly array $allowedLayers) {}
 
     /**
      * @return list<string>

src/Contract/OutputFormatter/OutputFormatterInput.php

@@ -14,6 +14,5 @@ public function __construct(
         public readonly bool $reportSkipped,
         public readonly bool $reportUncovered,
         public readonly bool $failOnUncovered
-    ) {
-    }
+    ) {}
 }

src/Contract/Result/Allowed.php

@@ -17,8 +17,7 @@ public function __construct(
         private readonly DependencyInterface $dependency,
         private readonly string $dependerLayer,
         private readonly string $dependentLayer
-    ) {
-    }
+    ) {}
 
     public function getDependency(): DependencyInterface
     {

src/Contract/Result/Error.php

@@ -11,9 +11,7 @@
  */
 final class Error implements Stringable
 {
-    public function __construct(private readonly string $message)
-    {
-    }
+    public function __construct(private readonly string $message) {}
 
     public function __toString()
     {

src/Contract/Result/OutputResult.php

@@ -24,8 +24,7 @@ private function __construct(
         public readonly array $rules,
         public readonly array $errors,
         public readonly array $warnings
-    ) {
-    }
+    ) {}
 
     public static function fromAnalysisResult(AnalysisResult $analysisResult): self
     {

src/Contract/Result/SkippedViolation.php

@@ -13,9 +13,7 @@
  */
 final class SkippedViolation implements CoveredRuleInterface
 {
-    public function __construct(private readonly DependencyInterface $dependency, private readonly string $dependerLayer, private readonly string $dependentLayer)
-    {
-    }
+    public function __construct(private readonly DependencyInterface $dependency, private readonly string $dependerLayer, private readonly string $dependentLayer) {}
 
     public function getDependency(): DependencyInterface
     {

src/Contract/Result/Uncovered.php

@@ -16,8 +16,7 @@ final class Uncovered implements RuleInterface
     public function __construct(
         private readonly DependencyInterface $dependency,
         public readonly string $layer
-    ) {
-    }
+    ) {}
 
     public function getDependency(): DependencyInterface
     {

src/Contract/Result/Violation.php

@@ -19,8 +19,7 @@ public function __construct(
         private readonly string $dependerLayer,
         private readonly string $dependentLayer,
         private readonly ViolationCreatingInterface $violationCreatingRule
-    ) {
-    }
+    ) {}
 
     public function getDependency(): DependencyInterface
     {

src/Contract/Result/Warning.php

@@ -14,9 +14,7 @@
  */
 final class Warning implements Stringable
 {
-    public function __construct(private readonly string $message)
-    {
-    }
+    public function __construct(private readonly string $message) {}
 
     /**
      * @param string[] $layerNames

src/Core/Analyser/DependencyLayersAnalyser.php

@@ -30,8 +30,7 @@ public function __construct(
         private readonly TokenResolver $tokenResolver,
         private readonly LayerResolverInterface $layerResolver,
         private readonly EventDispatcherInterface $eventDispatcher
-    ) {
-    }
+    ) {}
 
     /**
      * @throws AnalyserException

src/Core/Analyser/EventHandler/DependsOnDisallowedLayer.php

@@ -17,9 +17,7 @@
  */
 class DependsOnDisallowedLayer implements ViolationCreatingInterface
 {
-    public function __construct(private readonly EventHelper $eventHelper)
-    {
-    }
+    public function __construct(private readonly EventHelper $eventHelper) {}
 
     public static function getSubscribedEvents()
     {

src/Core/Analyser/EventHandler/DependsOnInternalToken.php

@@ -14,9 +14,7 @@
  */
 class DependsOnInternalToken implements ViolationCreatingInterface
 {
-    public function __construct(private readonly EventHelper $eventHelper)
-    {
-    }
+    public function __construct(private readonly EventHelper $eventHelper) {}
 
     public static function getSubscribedEvents()
     {

src/Core/Analyser/EventHandler/DependsOnPrivateLayer.php

@@ -13,9 +13,7 @@
  */
 class DependsOnPrivateLayer implements ViolationCreatingInterface
 {
-    public function __construct(private readonly EventHelper $eventHelper)
-    {
-    }
+    public function __construct(private readonly EventHelper $eventHelper) {}
 
     public static function getSubscribedEvents()
     {

src/Core/Analyser/EventHandler/UncoveredDependentHandler.php

@@ -15,9 +15,7 @@
  */
 class UncoveredDependentHandler implements EventSubscriberInterface
 {
-    public function __construct(private readonly bool $ignoreUncoveredInternalClasses)
-    {
-    }
+    public function __construct(private readonly bool $ignoreUncoveredInternalClasses) {}
 
     public function invoke(ProcessEvent $event): void
     {

src/Core/Analyser/EventHandler/UnmatchedSkippedViolations.php

@@ -16,9 +16,7 @@
  */
 class UnmatchedSkippedViolations implements EventSubscriberInterface
 {
-    public function __construct(private readonly EventHelper $eventHelper)
-    {
-    }
+    public function __construct(private readonly EventHelper $eventHelper) {}
 
     public function handleUnmatchedSkipped(PostProcessEvent $event): void
     {

src/Core/Analyser/LayerDependenciesAnalyser.php

@@ -23,8 +23,7 @@ public function __construct(
         private readonly TokenResolver $tokenResolver,
         private readonly DependencyResolver $dependencyResolver,
         private readonly LayerResolverInterface $layerResolver
-    ) {
-    }
+    ) {}
 
     /**
      * @return array<string, list<Uncovered>>

src/Core/Analyser/LayerForTokenAnalyser.php

@@ -26,8 +26,7 @@ public function __construct(
         private readonly AstMapExtractor $astMapExtractor,
         private readonly TokenResolver $tokenResolver,
         private readonly LayerResolverInterface $layerResolver
-    ) {
-    }
+    ) {}
 
     /**
      * @return array<string, string[]>

src/Core/Analyser/RulesetUsageAnalyser.php

@@ -29,8 +29,7 @@ public function __construct(
         private readonly DependencyResolver $dependencyResolver,
         private readonly TokenResolver $tokenResolver,
         private readonly array $layers
-    ) {
-    }
+    ) {}
 
     /**
      * @return array<string, array<string, int>>

src/Core/Ast/AstLoader.php

@@ -18,8 +18,7 @@ class AstLoader
     public function __construct(
         private readonly ParserInterface $parser,
         private readonly EventDispatcherInterface $eventDispatcher
-    ) {
-    }
+    ) {}
 
     /**
      * @param list<string> $files

src/Core/Ast/AstMap/AstInherit.php

@@ -25,8 +25,7 @@ public function __construct(
         public readonly FileOccurrence $fileOccurrence,
         public readonly AstInheritType $type,
         private readonly array $path = []
-    ) {
-    }
+    ) {}
 
     /**
      * @return AstInherit[]

src/Core/Ast/AstMap/ClassLike/ClassLikeToken.php

@@ -8,9 +8,7 @@
 
 final class ClassLikeToken implements TokenInterface
 {
-    private function __construct(private readonly string $className)
-    {
-    }
+    private function __construct(private readonly string $className) {}
 
     public static function fromFQCN(string $className): self
     {

src/Core/Ast/AstMap/DependencyToken.php

@@ -17,6 +17,5 @@ public function __construct(
         public readonly TokenInterface $token,
         public readonly FileOccurrence $fileOccurrence,
         public readonly DependencyType $type
-    ) {
-    }
+    ) {}
 }

src/Core/Ast/AstMap/Function/FunctionReference.php

@@ -21,8 +21,7 @@ public function __construct(
         private readonly FunctionToken $functionName,
         public readonly array $dependencies = [],
         private readonly ?FileReference $fileReference = null
-    ) {
-    }
+    ) {}
 
     public function withFileReference(FileReference $astFileReference): self
     {