PhpdocTagNoNamedArgumentsFixer - do not add @no-named-arguments to attribute classes

Author: kubawerlos

.dev-tools/psalm.xml

@@ -15,6 +15,10 @@
         <directory name='../src' />
     </projectFiles>
 
+    <stubs>
+        <file name='./psalm_stub.php' preloadClasses="true" />
+    </stubs>
+
     <issueHandlers>
         <DeprecatedClass errorLevel='suppress' />
         <InaccessibleMethod errorLevel='suppress' />

.dev-tools/psalm_stub.php

@@ -0,0 +1,18 @@
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of PHP CS Fixer: custom fixers.
+ *
+ * (c) 2018 Kuba Werłos
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace PhpCsFixer\Tokenizer {
+    final class FCT
+    {
+        public const int T_READONLY = \T_READONLY;
+    }
+    final class OtherClassSoTheNameOfClassAboveIsNotChanged {}
+}

README.md

@@ -5,7 +5,7 @@
 [![Latest stable version](https://img.shields.io/packagist/v/kubawerlos/php-cs-fixer-custom-fixers.svg?label=current%20version)](https://packagist.org/packages/kubawerlos/php-cs-fixer-custom-fixers)
 [![PHP version](https://img.shields.io/packagist/php-v/kubawerlos/php-cs-fixer-custom-fixers.svg)](https://php.net)
 [![License](https://img.shields.io/github/license/kubawerlos/php-cs-fixer-custom-fixers.svg)](LICENSE)
-![Tests](https://img.shields.io/badge/tests-3802-brightgreen.svg)
+![Tests](https://img.shields.io/badge/tests-3804-brightgreen.svg)
 [![Downloads](https://img.shields.io/packagist/dt/kubawerlos/php-cs-fixer-custom-fixers.svg)](https://packagist.org/packages/kubawerlos/php-cs-fixer-custom-fixers)
 
 [![CI status](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/actions/workflows/ci.yaml/badge.svg)](https://github.com/kubawerlos/php-cs-fixer-custom-fixers/actions/workflows/ci.yaml)

src/Fixer/PhpdocTagNoNamedArgumentsFixer.php

@@ -22,6 +22,11 @@
 use PhpCsFixer\FixerDefinition\FixerDefinition;
 use PhpCsFixer\FixerDefinition\FixerDefinitionInterface;
 use PhpCsFixer\Preg;
+use PhpCsFixer\Tokenizer\Analyzer\Analysis\NamespaceUseAnalysis;
+use PhpCsFixer\Tokenizer\Analyzer\AttributeAnalyzer;
+use PhpCsFixer\Tokenizer\Analyzer\FullyQualifiedNameAnalyzer;
+use PhpCsFixer\Tokenizer\CT;
+use PhpCsFixer\Tokenizer\FCT;
 use PhpCsFixer\Tokenizer\Token;
 use PhpCsFixer\Tokenizer\Tokens;
 use PhpCsFixer\WhitespacesFixerConfig;
@@ -131,10 +136,16 @@ public function fix(\SplFileInfo $file, Tokens $tokens): void
             }
 
             $prevIndex = $tokens->getPrevMeaningfulToken($index);
+            \assert(\is_int($prevIndex));
+
             if ($tokens[$prevIndex]->isGivenKind(\T_NEW)) {
                 continue;
             }
 
+            if (self::isAttributeClass($tokens, $prevIndex)) {
+                continue;
+            }
+
             $this->ensureIsDocBlockWithNoNameArgumentsTag($tokens, $index);
 
             $docBlockIndex = $tokens->getPrevTokenOfKind($index + 2, [[\T_DOC_COMMENT]]);
@@ -150,6 +161,31 @@ public function fix(\SplFileInfo $file, Tokens $tokens): void
         }
     }
 
+    private static function isAttributeClass(Tokens $tokens, int $index): bool
+    {
+        while ($tokens[$index]->isGivenKind([\T_ABSTRACT, \T_FINAL, FCT::T_READONLY])) {
+            $index = $tokens->getPrevMeaningfulToken($index);
+            \assert(\is_int($index));
+        }
+
+        if (!$tokens[$index]->isGivenKind(CT::T_ATTRIBUTE_CLOSE)) {
+            return false;
+        }
+
+        $fullyQualifiedNameAnalyzer = new FullyQualifiedNameAnalyzer($tokens);
+
+        foreach (AttributeAnalyzer::collect($tokens, $tokens->findBlockStart(Tokens::BLOCK_TYPE_ATTRIBUTE, $index)) as $attributeAnalysis) {
+            foreach ($attributeAnalysis->getAttributes() as $attribute) {
+                $attributeName = \strtolower($fullyQualifiedNameAnalyzer->getFullyQualifiedName($attribute['name'], $attribute['start'], NamespaceUseAnalysis::TYPE_CLASS));
+                if ($attributeName === 'attribute') {
+                    return true;
+                }
+            }
+        }
+
+        return false;
+    }
+
     private function ensureIsDocBlockWithNoNameArgumentsTag(Tokens $tokens, int $index): void
     {
         /** @var null|callable(WhitespacesFixerConfig, Tokens, int): void $ensureIsDocBlockWithTagNoNameArguments */

tests/Fixer/PhpdocTagNoNamedArgumentsFixerTest.php

@@ -224,5 +224,54 @@ class Foo {}
             [],
             new WhitespacesFixerConfig("\t", "\r\n"),
         ];
+
+        yield 'do not add for attribute class' => [
+            <<<'PHP'
+                <?php
+                #[Attribute(flags: Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
+                final readonly class MyAttributeClass {}
+                PHP,
+        ];
+
+        yield 'do not add for attribute class (used with alias)' => [
+            <<<'PHP'
+                <?php
+                namespace Foo;
+                use Attribute as TheAttributeClass;
+
+                /**
+                 * @no-named-arguments
+                 */
+                #[FooAttribute]
+                final class NotAttributeClass1 {}
+
+                #[TheAttributeClass(flags: Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
+                abstract class MyAttributeClass {}
+
+                /**
+                 * @no-named-arguments
+                 */
+                #[FooAttribute]
+                #[BarAttribute]
+                #[BazAttribute]
+                final class NotAttributeClass1 {}
+                PHP,
+            <<<'PHP'
+                <?php
+                namespace Foo;
+                use Attribute as TheAttributeClass;
+
+                #[FooAttribute]
+                final class NotAttributeClass1 {}
+
+                #[TheAttributeClass(flags: Attribute::TARGET_METHOD | Attribute::IS_REPEATABLE)]
+                abstract class MyAttributeClass {}
+
+                #[FooAttribute]
+                #[BarAttribute]
+                #[BazAttribute]
+                final class NotAttributeClass1 {}
+                PHP,
+        ];
     }
 }