Skip to content

Commit 14fadc2

Browse files
committed
Renamed this error code because the code didn't reflect the error at all
1 parent b10354a commit 14fadc2

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

package.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
104104
- Squiz.Commenting.BlockComment no longer requires blank line before comment if it's the first content after the PHP open tag
105105
-- Thanks to Juliette Reinders Folmer for the patch
106106
- Squiz.Functions.FunctionDeclarationArgumentSpacing now has more accurate error messages
107+
-- This includes renaming the SpaceAfterDefault error code to SpaceAfterEquals, which reflects the real error
107108
- Squiz.Functions.FunctionDeclarationArgumentSpacing now checks for no space after a reference operator
108109
-- If you don't want this new behaviour, exclude the SpacingAfterReference error message in a ruleset.xml file
109110
- Squiz.Functions.FunctionDeclarationArgumentSpacing now checks for no space after a variadic operator

src/Standards/Squiz/Sniffs/Functions/FunctionDeclarationArgumentSpacingSniff.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@ public function processBracket($phpcsFile, $openBracket)
213213
$spacesAfter,
214214
];
215215

216-
$fix = $phpcsFile->addFixableError($error, $equalToken, 'SpaceAfterDefault', $data);
216+
$fix = $phpcsFile->addFixableError($error, $equalToken, 'SpaceAfterEquals', $data);
217217
if ($fix === true) {
218218
$padding = str_repeat(' ', $this->equalsSpacing);
219219
if ($spacesAfter === 0) {

0 commit comments

Comments
 (0)