Fixed bug #3075 : PSR12.ControlStructures.BooleanOperatorPlacement false positive when operator is the only content on line

Author: gsherwood

package.xml

@@ -61,6 +61,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
     -- Thanks to Sergei Morozov for the patch
   - Fixed bug #3066 : No support for namespace operator used in type declarations
     -- Thanks to Juliette Reinders Folmer for the patch
+  - Fixed bug #3075 : PSR12.ControlStructures.BooleanOperatorPlacement false positive when operator is the only content on line
   - Fixed bug #3099 : Squiz.WhiteSpace.OperatorSpacing false positive when exiting with negative number
     -- Thanks to Sergei Morozov for the patch
   - Fixed bug #3124 : PSR-12 not reporting error for empty lines with only whitespace

src/Standards/PSR12/Sniffs/ControlStructures/BooleanOperatorPlacementSniff.php

@@ -90,44 +90,63 @@ public function process(File $phpcsFile, $stackPtr)
                 break;
             }
 
-            $operators[] = $operator;
-
             $prev = $phpcsFile->findPrevious(T_WHITESPACE, ($operator - 1), $parenOpener, true);
             if ($prev === false) {
                 // Parse error.
                 return;
             }
 
+            $next = $phpcsFile->findNext(T_WHITESPACE, ($operator + 1), $parenCloser, true);
+            if ($next === false) {
+                // Parse error.
+                return;
+            }
+
+            $firstOnLine = false;
+            $lastOnLine  = false;
+
             if ($tokens[$prev]['line'] < $tokens[$operator]['line']) {
                 // The boolean operator is the first content on the line.
-                if ($position === null) {
-                    $position = 'first';
-                }
+                $firstOnLine = true;
+            }
 
-                if ($position !== 'first') {
-                    $error = true;
-                }
+            if ($tokens[$next]['line'] > $tokens[$operator]['line']) {
+                // The boolean operator is the last content on the line.
+                $lastOnLine = true;
+            }
 
+            if ($firstOnLine === true && $lastOnLine === true) {
+                // The operator is the only content on the line.
+                // Don't record it because we can't determine
+                // placement information from looking at it.
                 continue;
             }
 
-            $next = $phpcsFile->findNext(T_WHITESPACE, ($operator + 1), $parenCloser, true);
-            if ($next === false) {
-                // Parse error.
-                return;
+            $operators[] = $operator;
+
+            if ($firstOnLine === false && $lastOnLine === false) {
+                // It's in the middle of content, so we can't determine
+                // placement information from looking at it, but we may
+                // still need to process it.
+                continue;
             }
 
-            if ($tokens[$next]['line'] > $tokens[$operator]['line']) {
-                // The boolean operator is the last content on the line.
+            if ($firstOnLine === true) {
+                if ($position === null) {
+                    $position = 'first';
+                }
+
+                if ($position !== 'first') {
+                    $error = true;
+                }
+            } else {
                 if ($position === null) {
                     $position = 'last';
                 }
 
                 if ($position !== 'last') {
                     $error = true;
                 }
-
-                continue;
             }
         } while ($operator !== false);
 

src/Standards/PSR12/Tests/ControlStructures/BooleanOperatorPlacementUnitTest.inc

@@ -108,3 +108,13 @@ if (
 ) {
     // elseif body
 }
+
+if (
+    ($value == 1 ||
+        $value == 2)
+    &&
+    ($value == 3 ||
+        $value == 4)
+) {
+    return 5;
+}

src/Standards/PSR12/Tests/ControlStructures/BooleanOperatorPlacementUnitTest.inc.fixed

@@ -118,3 +118,13 @@ if (
 ) {
     // elseif body
 }
+
+if (
+    ($value == 1 ||
+        $value == 2)
+    &&
+    ($value == 3 ||
+        $value == 4)
+) {
+    return 5;
+}