Browse Source

Fix priority, fix comments with new fixer names.

SpacePossum 8 years ago
parent
commit
19a23c760e

+ 1 - 1
src/Fixer/ArrayNotation/ShortArraySyntaxFixer.php

@@ -62,7 +62,7 @@ final class ShortArraySyntaxFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should be run before the BinaryOperatorSpacesFixer and TernarySpacesFixer.
+        // should be run before the BinaryOperatorSpacesFixer and TernaryOperatorSpacesFixer.
         return 1;
     }
 }

+ 2 - 1
src/Fixer/ClassNotation/OrderedClassElementsFixer.php

@@ -185,7 +185,8 @@ final class OrderedClassElementsFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // must run before MethodSeparationFixer, NoBlankLinesAfterClassOpeningFixer and SpaceAfterSemicolonFixer
+        // must run before MethodSeparationFixer, NoBlankLinesAfterClassOpeningFixer and SpaceAfterSemicolonFixer.
+        // must run after ProtectedToPrivateFixer.
         return 65;
     }
 

+ 6 - 0
src/Fixer/ClassNotation/ProtectedToPrivateFixer.php

@@ -60,6 +60,12 @@ final class ProtectedToPrivateFixer extends AbstractFixer
         return 'Converts protected variables and methods to private where possible.';
     }
 
+    public function getPriority()
+    {
+        // must run before OrderedClassElementsFixer.
+        return 66;
+    }
+
     /**
      * @param Tokens $tokens
      * @param int    $classIndex

+ 1 - 1
src/Fixer/Comment/NoEmptyCommentFixer.php

@@ -58,7 +58,7 @@ final class NoEmptyCommentFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should be run after PhpdocToCommentFixer and before ExtraEmptyLinesFixer, TrailingSpacesFixer and WhitespacyLinesFixer.
+        // should be run after PhpdocToCommentFixer and before NoExtraConsecutiveBlankLinesFixer, NoTrailingWhitespaceFixer and NoWhitespaceInBlankLineFixer.
         return 2;
     }
 

+ 1 - 1
src/Fixer/ControlStructure/NoUselessElseFixer.php

@@ -67,7 +67,7 @@ final class NoUselessElseFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should be run before NoDuplicateSemicolonsFixer, NoWhitespaceInBlankLineFixer, NoExtraConsecutiveBlankLinesFixer and BracesFixer.
+        // should be run before NoWhitespaceInBlankLineFixer, NoExtraConsecutiveBlankLinesFixer, BracesFixer and after NoEmptyStatementFixer.
         return 25;
     }
 

+ 1 - 1
src/Fixer/Import/SingleImportPerStatementFixer.php

@@ -63,7 +63,7 @@ final class SingleImportPerStatementFixer extends AbstractFixer
 
     public function getPriority()
     {
-        // must be run before RemoveLeadingSlashUseFuxer, NoSinglelineWhitespaceBeforeSemicolonsFixer, SpaceAfterSemicolonFixer, MultilineSpacesBeforeSemicolonFixer, NoLeadingImportSlashFixer
+        // must be run before NoLeadingImportSlashFixer, NoSinglelineWhitespaceBeforeSemicolonsFixer, SpaceAfterSemicolonFixer, NoMultilineWhitespaceBeforeSemicolonsFixer, NoLeadingImportSlashFixer.
         return 1;
     }
 

+ 1 - 1
src/Fixer/LanguageConstruct/CombineConsecutiveUnsetsFixer.php

@@ -88,7 +88,7 @@ final class CombineConsecutiveUnsetsFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should ran before SpacesAfterSemicolonFixer, WhitespacyLinesFixer, TrailingSpacesFixer and ExtraEmptyLinesFixer and after NoEmptyStatementFixer.
+        // should ran before SpaceAfterSemicolonFixer, NoWhitespaceInBlankLineFixer, NoTrailingWhitespaceFixer and NoExtraConsecutiveBlankLinesFixer and after NoEmptyStatementFixer.
         return 24;
     }
 

+ 1 - 1
src/Fixer/Phpdoc/NoEmptyPhpdocFixer.php

@@ -57,7 +57,7 @@ final class NoEmptyPhpdocFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should be run before NoExtraConsecutiveBlankLinesFixer, NoTrailingSpacesFixer, NoWhitespaceInBlankLineFixer and
+        // should be run before NoExtraConsecutiveBlankLinesFixer, NoTrailingWhitespaceFixer, NoWhitespaceInBlankLineFixer and
         // after PhpdocNoAccessFixer, PhpdocNoEmptyReturnFixer, PhpdocNoPackageFixer.
         return 5;
     }

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

@@ -58,7 +58,7 @@ final class NoUselessReturnFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should be run before BlankLineBeforeReturnFixer, NoExtraConsecutiveBlankLinesFixer, NoWhitespaceInBlankLineFixer and after SimplifiedNullReturnFixer and NoDuplicateSemicolonFixer.
+        // should be run before BlankLineBeforeReturnFixer, NoExtraConsecutiveBlankLinesFixer, NoWhitespaceInBlankLineFixer and after SimplifiedNullReturnFixer and NoEmptyStatementFixer.
         return -18;
     }
 

+ 1 - 1
src/Fixer/Whitespace/NoTrailingWhitespaceFixer.php

@@ -75,7 +75,7 @@ final class NoTrailingWhitespaceFixer extends AbstractFixer
      */
     public function getPriority()
     {
-        // should be run after NoEmptyPhpdocFixer, UnneededControlParenthesesFixer, ClassDefinitionFixer, CombineConsecutiveUnsetsFixer, NoEmptyStatementFixer and NoUselessElseFixer.
+        // should be run after NoEmptyPhpdocFixer, NoUnneededControlParenthesesFixer, ClassDefinitionFixer, CombineConsecutiveUnsetsFixer, NoEmptyStatementFixer and NoUselessElseFixer.
         return 0;
     }
 }

Some files were not shown because too many files changed in this diff