From 7569ac879c82e3dd6e67797fce4f6a7fac76adce Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gabriel=20Ostroluck=C3=BD?= <gabriel.ostrolucky@gmail.com>
Date: Wed, 23 Oct 2024 23:43:09 +0200
Subject: [PATCH] fix: improve handling of comments in
 line_break_between_method_arguments

---
 .../LineBreakBetweenMethodArgumentsFixer.php  | 12 ++++-
 .../Regression/Case10.php                     | 52 +++++++++++++++++++
 2 files changed, 63 insertions(+), 1 deletion(-)
 create mode 100644 tests/UseCase/LineBreakBetweenMethods/Regression/Case10.php

diff --git a/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php b/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php
index 988f750..2345ea7 100644
--- a/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php
+++ b/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php
@@ -235,10 +235,20 @@ private function mergeArgs(Tokens $tokens, $index): void
         $closeBraceIndex = $this->analyze($tokens)->getClosingParenthesis($openBraceIndex);
 
         foreach ($tokens->findGivenKind(T_WHITESPACE, $openBraceIndex, $closeBraceIndex) as $spaceIndex => $spaceToken) {
+            if ($tokens[$tokens->getPrevNonWhitespace($spaceIndex)]->isGivenKind([T_COMMENT, T_DOC_COMMENT])) {
+                continue;
+            }
+
+            if ($tokens[$tokens->getNextNonWhitespace($spaceIndex)]->isGivenKind([T_COMMENT, T_DOC_COMMENT])) {
+                continue;
+            }
+
             $tokens[$spaceIndex] = new Token([T_WHITESPACE, ' ']);
         }
 
-        $tokens->removeTrailingWhitespace($openBraceIndex);
+        if (!$tokens[$tokens->getNextNonWhitespace($openBraceIndex)]->isGivenKind([T_COMMENT, T_DOC_COMMENT])) {
+            $tokens->removeTrailingWhitespace($openBraceIndex);
+        }
         $tokens->removeLeadingWhitespace($closeBraceIndex);
 
         $end = $tokens->getNextTokenOfKind($closeBraceIndex, [';', '{']);
diff --git a/tests/UseCase/LineBreakBetweenMethods/Regression/Case10.php b/tests/UseCase/LineBreakBetweenMethods/Regression/Case10.php
new file mode 100644
index 0000000..ca23439
--- /dev/null
+++ b/tests/UseCase/LineBreakBetweenMethods/Regression/Case10.php
@@ -0,0 +1,52 @@
+<?php
+
+declare(strict_types=1);
+
+namespace tests\UseCase\LineBreakBetweenMethods\Regression;
+
+use PedroTroller\CS\Fixer\CodingStyle\LineBreakBetweenMethodArgumentsFixer;
+use tests\UseCase;
+
+/**
+ * https://github.com/PedroTroller/PhpCSFixer-Custom-Fixers/issues/208
+ * https://github.com/PedroTroller/PhpCSFixer-Custom-Fixers/issues/214.
+ */
+final class Case10 implements UseCase
+{
+    public function getFixers(): iterable
+    {
+        yield new LineBreakBetweenMethodArgumentsFixer();
+    }
+
+    public function getRawScript(): string
+    {
+        return <<<'PHP'
+            <?php
+
+            readonly class Foo
+            {
+                public function __construct(
+                    // simple comment
+                    public ?string $simpleCommentAbove = null,
+                    // public string|null $commentedOutLine = null,
+                    public ?string $detailAndEdit = null,
+                    /**
+                     * @var list<string>
+                     */
+                    public array $groups = [])
+                {
+                }
+            }
+            PHP;
+    }
+
+    public function getExpectation(): string
+    {
+        return $this->getRawScript();
+    }
+
+    public function getMinSupportedPhpVersion(): int
+    {
+        return 80000;
+    }
+}