Fix end docblock (#42)

src/DocBlock/UselessDocBlockCleaner.php

@@ -15,9 +15,14 @@ final class UselessDocBlockCleaner
     private const CLEANING_REGEXES = [
         self::TODO_COMMENT_BY_PHPSTORM_REGEX,
         self::TODO_IMPLEMENT_METHOD_COMMENT_BY_PHPSTORM_REGEX,
+        self::COMMENT_CONSTRUCTOR_CLASS_REGEX,
+
+        // must run first
+        self::STANDALONE_DOCBLOCK_CLASS_REGEX,
+        // then this one
         self::STANDALONE_COMMENT_CLASS_REGEX,
+        // then this one
         self::INLINE_COMMENT_CLASS_REGEX,
-        self::COMMENT_CONSTRUCTOR_CLASS_REGEX,
     ];
 
     /**
@@ -36,7 +41,13 @@ final class UselessDocBlockCleaner
      * @see https://regex101.com/r/RzTdFH/4
      * @var string
      */
-    private const STANDALONE_COMMENT_CLASS_REGEX = '#(\/\*{2}\s+?)?(\*|\/\/)\s+[cC]lass\s+[^\s]*(\s+\*\/)?$#';
+    private const STANDALONE_DOCBLOCK_CLASS_REGEX = '#(\/\*\*\s+)\*\s+[cC]lass\s+[^\s]*(\s+\*\/)$#';
+
+    /**
+     * @see https://regex101.com/r/RzTdFH/4
+     * @var string
+     */
+    private const STANDALONE_COMMENT_CLASS_REGEX = '#\/\/\s+[cC]lass\s+\w+$#';
 
     /**
      * @see https://regex101.com/r/RzTdFH/4

src/Fixer/Commenting/RemoveUselessDefaultCommentFixer.php

@@ -64,6 +64,7 @@ public function fix(SplFileInfo $fileInfo, Tokens $tokens): void
             }
 
             $cleanedDocContent = $this->uselessDocBlockCleaner->clearDocTokenContent($token);
+
             if ($cleanedDocContent === '') {
                 // remove token
                 $tokens->clearTokenAndMergeSurroundingWhitespace($index);

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/useless_class_comment.php.inc → tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/class_docblock.php.inc

@@ -16,22 +16,6 @@ class SomeClass2
 {
 }
 
-/**
- * class SomeClass
- */
-class SomeClass3
-{
-}
-
-// class SomeClass
-class SomeClass4
-{
-}
-
-// class        SomeClass
-class SomeClass5
-{
-}
 ?>
 -----
 <?php
@@ -48,18 +32,4 @@ class SomeClass2
 {
 }
 
-
-class SomeClass3
-{
-}
-
-
-class SomeClass4
-{
-}
-
-
-class SomeClass5
-{
-}
 ?>

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/remove_from_bottom_line.php.inc

@@ -0,0 +1,28 @@
+<?php
+
+namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
+
+/**
+ * @property SomeItem $item
+ * Class RemoveFromPropertyTest
+ */
+#[\AllowDynamicProperties]
+final class RemoveFromBottomLine
+{
+}
+
+?>
+-----
+<?php
+
+namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
+
+/**
+ * @property SomeItem $item
+ */
+#[\AllowDynamicProperties]
+final class RemoveFromBottomLine
+{
+}
+
+?>

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/simple_inline_comment.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
+
+// class SomeClass
+class SomeClass4
+{
+}
+
+// class        SomeClass
+class SomeClass5
+{
+}
+
+?>
+-----
+<?php
+
+namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
+
+
+class SomeClass4
+{
+}
+
+
+class SomeClass5
+{
+}
+
+?>

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/skip_1_extra_word_class_method_comment.php.inc

@@ -2,15 +2,12 @@
 
 namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
 
-class SomeClass
+final class Skip1ExtraWordClassMethodComment
 {
-/**
- * Get special translator
- */
-	public function getTranslator()
-	{
-
-	}
+    /**
+    * Get special translator
+    */
+    public function getTranslator()
+    {
+    }
 }
-
-?>

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/skip_inline_set.php.inc

@@ -4,9 +4,9 @@ namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCom
 
 class SkipInlineSet
 {
-	public function getTranslator()
-	{
+    public function getTranslator()
+    {
         // set value
         $value = 1000;
-	}
+    }
 }

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/skip_usefull_class_comment.php.inc

@@ -5,6 +5,6 @@ namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCom
 /**
  * class to provide something
  */
-class SomeClass5
+class SkipUsefullClassComment
 {
 }

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/skip_usefull_class_constructor_comment.php.inc

@@ -2,13 +2,12 @@
 
 namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
 
-class SomeClass
+class SkipUsefullClassConstructorComment
 {
-	/**
+    /**
      * A usefull comment.
      */
-	public function __construct()
-	{
-
-	}
+    public function __construct()
+    {
+    }
 }

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/todo_comment.php.inc

@@ -1,25 +1,28 @@
 <?php
 
-namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemovePHPStormTodoImplementMethodCommentFixer\Fixture;
+namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
 
 class PackagesData
 {
     public function getIterator()
     {
         // TODO: Implement getIterator() method.
+        $value = 100;
     }
 }
+
 ?>
 -----
 <?php
 
-namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemovePHPStormTodoImplementMethodCommentFixer\Fixture;
+namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
 
 class PackagesData
 {
     public function getIterator()
     {
-
+        $value = 100;
     }
 }
+
 ?>

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/Fixture/useless_class_constructor_comment.php.inc

@@ -2,86 +2,42 @@
 
 namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
 
-class SomeClass
+final class UselessClassConstructorComment
 {
-/**
- * SomeClass constructor.
- */
-	public function __construct()
-	{
-
-	}
-}
-
-class SomeClass
-{
-/**
- * SomeClass Constructor.
- */
-	public function __construct()
-	{
-
-	}
+    /**
+     * SomeClass constructor.
+     */
+    public function __construct()
+    {
+    }
 }
 
 class SomeClass
 {
-/**
- * SomeClass Constructor
- */
-	public function __construct()
-	{
-
-	}
+    // SomeClass constructor
+    public function __construct()
+    {
+    }
 }
 
-class SomeClass
-{
-// SomeClass constructor
-	public function __construct()
-	{
-
-	}
-}
 ?>
 -----
 <?php
 
 namespace Symplify\CodingStandard\Tests\Fixer\Commenting\RemoveUselessDefaultCommentFixer\Fixture;
 
-class SomeClass
-{
-
-	public function __construct()
-	{
-
-	}
-}
-
-class SomeClass
+final class UselessClassConstructorComment
 {
-
-	public function __construct()
-	{
-
-	}
+    public function __construct()
+    {
+    }
 }
 
 class SomeClass
 {
-
-	public function __construct()
-	{
-
-	}
+    public function __construct()
+    {
+    }
 }
 
-class SomeClass
-{
-
-	public function __construct()
-	{
-
-	}
-}
 ?>

tests/Fixer/Commenting/RemoveUselessDefaultCommentFixer/config/configured_rule.php

@@ -1,10 +1,14 @@
 <?php
 
 declare(strict_types=1);
+use PhpCsFixer\Fixer\Basic\BracesFixer;
 
 use Symplify\CodingStandard\Fixer\Commenting\RemoveUselessDefaultCommentFixer;
 use Symplify\EasyCodingStandard\Config\ECSConfig;
 
 return static function (ECSConfig $ecsConfig): void {
-    $ecsConfig->rule(RemoveUselessDefaultCommentFixer::class);
+    $ecsConfig->rules([
+        RemoveUselessDefaultCommentFixer::class,
+        BracesFixer::class,
+    ]);
 };