-
-
Notifications
You must be signed in to change notification settings - Fork 468
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PHPCS 3.4.0: Add new Generic.WhiteSpace.IncrementDecrementSpacing sniff #1511
Comments
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Apr 22, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Apr 22, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Apr 26, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
May 15, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Jun 17, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Jun 17, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Jun 25, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Jul 22, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Aug 7, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Aug 10, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Aug 12, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Aug 18, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Sep 11, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
jrfnl
added a commit
to jrfnl/WordPress-Coding-Standards
that referenced
this issue
Oct 13, 2022
Related: * `Universal.Namespaces.OneDeclarationPerFile` - upstream Extra WordPress#6 * `Universal.Namespaces.DisallowCurlyBraceSyntax` - upstream Extra WordPress#4 * `Universal.Namespaces.DisallowDeclarationWithoutName` - upstream Extra WordPress#50 * `Universal.UseStatements.NoLeadingBackslash` - upstream Extra WordPress#46 * `Universal.UseStatements.LowercaseFunctionConst` - upstream Extra WordPress#58 * `Universal.Constants.LowercaseClassResolutionKeyword` - upstream Extra WordPress#72 * `Generic.WhiteSpace.IncrementDecrementSpacing` - issue WordPress#1511, upstream PHPCS WordPress#2172, WordPress#2174 * `Universal.Operators.DisallowStandalonePostIncrementDecrement` - upstream Extra WordPress#65 * `Universal.Constants.UppercaseMagicConstants` - upstream Extra WordPress#64
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Rationale
Ever come across code like this:
$a + - $b ++
?Let's at least make the confusion slightly less by prohibiting spaces between a variable and the incrementor/decrementor.
PHPCS 3.4.0 introduces a new
Generic.WhiteSpace.IncrementDecrementSpacing
sniff which does just that, including auto-fixer.I ran that sniff over WP Core when I originally wrote the sniff and at that time, WP Core basically complied with the "no space between incrementor/decrementor and variable" rule, except for 5 instances, which can be auto-fixed by this sniff anyway.
I'd like to suggest adding this sniff to the
Core
ruleset once the minimum PHPCS requirement has been upped to PHPCS 3.4.0./cc @pento
References
Action Checklist
Once PHPCS 3.4.0 has been released and WPCS is ready/willing to up the minimum required PHPCS version:
composer.json
file.Generic.WhiteSpace.IncrementDecrementSpacing
sniff to theWordPress-Core
ruleset.xml
file (in the bottom section/uncovered by handbook).The text was updated successfully, but these errors were encountered: