Skip to content

Commit 77b2aa7

Browse files
committed
Fix only when fixer is enabled
1 parent 9f8d6a6 commit 77b2aa7

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

src/Standards/Squiz/Sniffs/ControlStructures/InlineIfDeclarationSniff.php

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -107,10 +107,12 @@ public function process(File $phpcsFile, $stackPtr)
107107
$error = 'Inline shorthand IF statement requires 1 space after THEN; %s found';
108108
$data = array($spaceAfter);
109109
$fix = $phpcsFile->addFixableError($error, $stackPtr, 'SpacingAfterThen', $data);
110-
if ($spaceAfter === 0) {
111-
$phpcsFile->fixer->addContent($stackPtr, ' ');
112-
} else {
113-
$phpcsFile->fixer->replaceToken(($stackPtr + 1), ' ');
110+
if ($fix === true) {
111+
if ($spaceAfter === 0) {
112+
$phpcsFile->fixer->addContent($stackPtr, ' ');
113+
} else {
114+
$phpcsFile->fixer->replaceToken(($stackPtr + 1), ' ');
115+
}
114116
}
115117
}
116118

@@ -138,10 +140,12 @@ public function process(File $phpcsFile, $stackPtr)
138140
$error = 'Inline shorthand IF statement requires 1 space after ELSE; %s found';
139141
$data = array($spaceAfter);
140142
$fix = $phpcsFile->addFixableError($error, $inlineElse, 'SpacingAfterElse', $data);
141-
if ($spaceAfter === 0) {
142-
$phpcsFile->fixer->addContent($inlineElse, ' ');
143-
} else {
144-
$phpcsFile->fixer->replaceToken(($inlineElse + 1), ' ');
143+
if ($fix === true) {
144+
if ($spaceAfter === 0) {
145+
$phpcsFile->fixer->addContent($inlineElse, ' ');
146+
} else {
147+
$phpcsFile->fixer->replaceToken(($inlineElse + 1), ' ');
148+
}
145149
}
146150
}
147151

0 commit comments

Comments
 (0)