Improve error message when using allowOnly

squizlabs · Sep 29, 2020 · 01370ba · 01370ba
1 parent 861acb1
commit 01370ba
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/Standards/PSR12/Sniffs/ControlStructures/BooleanOperatorPlacementSniff.php b/src/Standards/PSR12/Sniffs/ControlStructures/BooleanOperatorPlacementSniff.php
@@ -154,8 +154,18 @@ public function process(File $phpcsFile, $stackPtr)
             return;
         }
 
-        $error = 'Boolean operators between conditions must be at the beginning or end of the line, but not both';
-        $fix   = $phpcsFile->addFixableError($error, $stackPtr, 'FoundMixed');
+        switch ($this->allowOnly) {
+        case 'first':
+            $error = 'Boolean operators between conditions must be at the beginning of the line';
+            break;
+        case 'last':
+            $error = 'Boolean operators between conditions must be at the end of the line';
+            break;
+        default:
+            $error = 'Boolean operators between conditions must be at the beginning or end of the line, but not both';
+        }
+
+        $fix = $phpcsFile->addFixableError($error, $stackPtr, 'FoundMixed');
         if ($fix === false) {
             return;
         }