StandaloneLinePromotedPropertyFixer: properly deal with attributes.

Author: gnutix

src/TokenRunner/Analyzer/FixerAnalyzer/TokenSkipper.php

@@ -57,15 +57,45 @@ public function skipBlocksReversed(Tokens $tokens, int $position): int
     {
         /** @var Token $token */
         $token = $tokens[$position];
-        if (! $token->isGivenKind(CT::T_ARRAY_SQUARE_BRACE_CLOSE) && ! $token->equals(')')) {
+        if (! $token->isGivenKind(CT::T_ARRAY_SQUARE_BRACE_CLOSE) && ! $token->equals(')') && ! $token->isGivenKind(CT::T_ATTRIBUTE_CLOSE)) {
             return $position;
         }
 
+        // Check if this is an attribute closing bracket
+        if ($token->isGivenKind(CT::T_ATTRIBUTE_CLOSE)) {
+            $attributeStartPosition = $this->findAttributeStart($tokens, $position);
+            if ($attributeStartPosition !== null) {
+                return $attributeStartPosition;
+            }
+        }
+
         $blockInfo = $this->blockFinder->findInTokensByEdge($tokens, $position);
         if (! $blockInfo instanceof BlockInfo) {
             throw new ShouldNotHappenException();
         }
 
         return $blockInfo->getStart();
     }
+
+    /**
+     * @param Tokens<Token> $tokens
+     */
+    private function findAttributeStart(Tokens $tokens, int $closingBracketPosition): ?int
+    {
+        // Search backwards for T_ATTRIBUTE token (#[)
+        for ($i = $closingBracketPosition - 1; $i >= 0; --$i) {
+            $currentToken = $tokens[$i];
+
+            if ($currentToken->isGivenKind(T_ATTRIBUTE)) {
+                return $i;
+            }
+
+            // If we hit another ] or reach a statement boundary, stop searching
+            if ($currentToken->equals(']') || $currentToken->equals(';') || $currentToken->equals('{') || $currentToken->equals('}')) {
+                break;
+            }
+        }
+
+        return null;
+    }
 }

src/TokenRunner/Transformer/FixerTransformer/TokensNewliner.php

@@ -51,11 +51,11 @@ public function breakItems(BlockInfo $blockInfo, Tokens $tokens, int $kind): voi
 
         // again, from the bottom to the top
         for ($i = $blockInfo->getEnd() - 1; $i >= $blockInfo->getStart(); --$i) {
+            $i = $this->tokenSkipper->skipBlocksReversed($tokens, $i);
+
             /** @var Token $currentToken */
             $currentToken = $tokens[$i];
 
-            $i = $this->tokenSkipper->skipBlocksReversed($tokens, $i);
-
             // 2. new line after each comma ",", instead of just space
             if ($currentToken->getContent() === ',') {
                 if ($this->isLastItem($tokens, $i)) {

tests/Fixer/Spacing/StandaloneLinePromotedPropertyFixer/Fixture/skip_attributes.php.inc

@@ -0,0 +1,96 @@
+<?php
+
+namespace Symplify\CodingStandard\Tests\Fixer\Spacing\StandaloneLinePromotedPropertyFixer\Fixture;
+
+final class SkipAttributes
+{
+    public function __construct(#[JMS\Expose, JMS\Type('integer')] private int $id, private string $name) {}
+}
+
+final class SkipAttributes2
+{
+    public function __construct(
+        #[JMS\Expose, JMS\Type('integer')]
+        private int $id,
+        private string $name,
+    ) {}
+}
+
+final class SkipAttributes3
+{
+    public function __construct(
+        #[JMS\Expose, JMS\Type('integer')] private int $id,
+        private string $name,
+    ) {}
+}
+
+final class SkipAttributes4
+{
+    public function __construct(
+        #[JMS\Expose]
+        #[JMS\Type('integer')] private int $id,
+        private string $name,
+    ) {}
+}
+
+final class SkipAttributes5
+{
+    public function __construct(
+        #[JMS\Expose]
+        #[JMS\Type('integer')]
+        private int $id,
+        private string $name,
+    ) {}
+}
+
+?>
+-----
+<?php
+
+namespace Symplify\CodingStandard\Tests\Fixer\Spacing\StandaloneLinePromotedPropertyFixer\Fixture;
+
+final class SkipAttributes
+{
+    public function __construct(
+        #[JMS\Expose, JMS\Type('integer')] private int $id,
+        private string $name
+    ) {}
+}
+
+final class SkipAttributes2
+{
+    public function __construct(
+        #[JMS\Expose, JMS\Type('integer')]
+        private int $id,
+        private string $name,
+    ) {}
+}
+
+final class SkipAttributes3
+{
+    public function __construct(
+        #[JMS\Expose, JMS\Type('integer')] private int $id,
+        private string $name,
+    ) {}
+}
+
+final class SkipAttributes4
+{
+    public function __construct(
+        #[JMS\Expose]
+        #[JMS\Type('integer')] private int $id,
+        private string $name,
+    ) {}
+}
+
+final class SkipAttributes5
+{
+    public function __construct(
+        #[JMS\Expose]
+        #[JMS\Type('integer')]
+        private int $id,
+        private string $name,
+    ) {}
+}
+
+?>