Skip to content

Commit 88d5f5f

Browse files
committed
Autoreview the PHPDoc of *Fixer::getPriority based on the priority map
1 parent 10642ed commit 88d5f5f

File tree

130 files changed

+754
-103
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

130 files changed

+754
-103
lines changed

src/Fixer/Alias/BacktickToShellExecFixer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,10 +55,11 @@ public function getDefinition()
5555

5656
/**
5757
* {@inheritdoc}
58+
*
59+
* Must run before EscapeImplicitBackslashesFixer, ExplicitStringVariableFixer.
5860
*/
5961
public function getPriority()
6062
{
61-
// Should run before escape_implicit_backslashes
6263
return 2;
6364
}
6465

src/Fixer/Alias/NoAliasFunctionsFixer.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,16 @@ public function getDefinition()
154154
);
155155
}
156156

157+
/**
158+
* {@inheritdoc}
159+
*
160+
* Must run before ImplodeCallFixer, PhpUnitDedicateAssertFixer.
161+
*/
162+
public function getPriority()
163+
{
164+
return 0;
165+
}
166+
157167
/**
158168
* {@inheritdoc}
159169
*/

src/Fixer/Alias/NoMixedEchoPrintFixer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,10 +75,11 @@ public function getDefinition()
7575

7676
/**
7777
* {@inheritdoc}
78+
*
79+
* Must run after NoShortEchoTagFixer.
7880
*/
7981
public function getPriority()
8082
{
81-
// should run after NoShortEchoTagFixer.
8283
return -10;
8384
}
8485

src/Fixer/Alias/PowToExponentiationFixer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,11 @@ public function getDefinition()
5353

5454
/**
5555
* {@inheritdoc}
56+
*
57+
* Must run before BinaryOperatorSpacesFixer, MethodArgumentSpaceFixer, NativeFunctionCasingFixer, NoSpacesAfterFunctionNameFixer, NoSpacesInsideParenthesisFixer.
5658
*/
5759
public function getPriority()
5860
{
59-
// must be run before BinaryOperatorSpacesFixer
6061
return 3;
6162
}
6263

src/Fixer/ArrayNotation/ArraySyntaxFixer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,10 +68,11 @@ public function getDefinition()
6868

6969
/**
7070
* {@inheritdoc}
71+
*
72+
* Must run before BinaryOperatorSpacesFixer, TernaryOperatorSpacesFixer.
7173
*/
7274
public function getPriority()
7375
{
74-
// should be run before the BinaryOperatorSpacesFixer and TernaryOperatorSpacesFixer.
7576
return 1;
7677
}
7778

src/Fixer/ArrayNotation/NoMultilineWhitespaceAroundDoubleArrowFixer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,11 @@ public function getDefinition()
3838

3939
/**
4040
* {@inheritdoc}
41+
*
42+
* Must run before BinaryOperatorSpacesFixer, TrailingCommaInMultilineArrayFixer.
4143
*/
4244
public function getPriority()
4345
{
44-
// should be run before the TrailingCommaInMultilineArrayFixer and BinaryOperatorSpacesFixer.
4546
return 1;
4647
}
4748

src/Fixer/ArrayNotation/TrailingCommaInMultilineArrayFixer.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,16 @@ public function getDefinition()
6161
);
6262
}
6363

64+
/**
65+
* {@inheritdoc}
66+
*
67+
* Must run after NoMultilineWhitespaceAroundDoubleArrowFixer.
68+
*/
69+
public function getPriority()
70+
{
71+
return 0;
72+
}
73+
6474
/**
6575
* {@inheritdoc}
6676
*/

src/Fixer/Basic/BracesFixer.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,10 +123,12 @@ public function bar($baz)
123123

124124
/**
125125
* {@inheritdoc}
126+
*
127+
* Must run before ArrayIndentationFixer, MethodChainingIndentationFixer.
128+
* Must run after ClassAttributesSeparationFixer, ElseifFixer, LineEndingFixer, MethodSeparationFixer, NoAlternativeSyntaxFixer, NoEmptyStatementFixer, NoUselessElseFixer, SingleTraitInsertPerStatementFixer.
126129
*/
127130
public function getPriority()
128131
{
129-
// should be run after the ElseIfFixer, LineEndingFixer, NoEmptyStatementFixer and NoUselessElseFixer
130132
return -25;
131133
}
132134

src/Fixer/Casing/NativeFunctionCasingFixer.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,16 @@ public function getDefinition()
3535
);
3636
}
3737

38+
/**
39+
* {@inheritdoc}
40+
*
41+
* Must run after FunctionToConstantFixer, PowToExponentiationFixer.
42+
*/
43+
public function getPriority()
44+
{
45+
return 0;
46+
}
47+
3848
/**
3949
* {@inheritdoc}
4050
*/

src/Fixer/CastNotation/CastSpacesFixer.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,10 +63,11 @@ public function getDefinition()
6363

6464
/**
6565
* {@inheritdoc}
66+
*
67+
* Must run after NoShortBoolCastFixer.
6668
*/
6769
public function getPriority()
6870
{
69-
// should be run after the NoShortBoolCastFixer
7071
return -10;
7172
}
7273

0 commit comments

Comments
 (0)