Browse Source

Make single_line_comment_style runs after no_useless_return

Kuba Werłos 4 years ago
parent
commit
c262d5c795

+ 10 - 0
src/Fixer/Comment/SingleLineCommentStyleFixer.php

@@ -98,6 +98,16 @@ $c = 3;
         );
     }
 
+    /**
+     * {@inheritdoc}
+     *
+     * Must run after NoUselessReturnFixer.
+     */
+    public function getPriority()
+    {
+        return -19;
+    }
+
     /**
      * {@inheritdoc}
      */

+ 1 - 1
src/Fixer/ReturnNotation/NoUselessReturnFixer.php

@@ -55,7 +55,7 @@ function example($b) {
     /**
      * {@inheritdoc}
      *
-     * Must run before BlankLineBeforeReturnFixer, BlankLineBeforeStatementFixer, NoExtraBlankLinesFixer, NoWhitespaceInBlankLineFixer.
+     * Must run before BlankLineBeforeReturnFixer, BlankLineBeforeStatementFixer, NoExtraBlankLinesFixer, NoWhitespaceInBlankLineFixer, SingleLineCommentStyleFixer.
      * Must run after NoEmptyStatementFixer, NoUnneededCurlyBracesFixer, NoUselessElseFixer, SimplifiedNullReturnFixer.
      */
     public function getPriority()

+ 1 - 0
tests/AutoReview/FixerFactoryTest.php

@@ -192,6 +192,7 @@ final class FixerFactoryTest extends TestCase
             [$fixers['no_useless_return'], $fixers['blank_line_before_statement']],
             [$fixers['no_useless_return'], $fixers['no_extra_blank_lines']],
             [$fixers['no_useless_return'], $fixers['no_whitespace_in_blank_line']],
+            [$fixers['no_useless_return'], $fixers['single_line_comment_style']],
             [$fixers['no_useless_sprintf'], $fixers['method_argument_space']],
             [$fixers['no_useless_sprintf'], $fixers['native_function_casing']],
             [$fixers['no_useless_sprintf'], $fixers['no_empty_statement']],

+ 17 - 0
tests/Fixtures/Integration/priority/no_useless_return,single_line_comment_style.test

@@ -0,0 +1,17 @@
+--TEST--
+Integration of fixers: no_useless_return,single_line_comment_style.
+--RULESET--
+{ "no_useless_return": true, "single_line_comment_style": true }
+--EXPECT--
+<?php
+function foo()
+{
+    // foo
+}
+
+--INPUT--
+<?php
+function foo()
+{
+    return/* foo */;
+}