Squiz/NonExecutableCode: bug fix - allow for all logic operators

The sniff, as it was, only allowed for `or` and `||` logical operators before a termination expression, not for `and`, `&&` or `xor`, while PHP allows these too.

See: https://3v4l.org/OWS4n#veol

Fixed now.

Includes unit tests.

Author: jrfnl

src/Standards/Squiz/Sniffs/PHP/NonExecutableCodeSniff.php

@@ -63,9 +63,9 @@ public function process(File $phpcsFile, $stackPtr)
 
         // Tokens which can be used in inline expressions need special handling.
         if (isset($this->expressionTokens[$tokens[$stackPtr]['code']]) === true) {
-            // If this token is preceded with an "or", it only relates to one line
+            // If this token is preceded by a logical operator, it only relates to one line
             // and should be ignored. For example: fopen() or die().
-            if ($tokens[$prev]['code'] === T_LOGICAL_OR || $tokens[$prev]['code'] === T_BOOLEAN_OR) {
+            if (isset(Tokens::$booleanOperators[$tokens[$prev]['code']]) === true) {
                 return;
             }
         }

src/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.1.inc

@@ -308,5 +308,20 @@ function parseError2() {
     echo 'unreachable';
 }
 
+// All logical operators are allowed with inline expressions (but this was not correctly handled by the sniff).
+function exitExpressionsWithLogicalOperators() {
+    $condition = false;
+    $condition || exit();
+    $condition or die();
+
+    $condition = true;
+    $condition && die();
+    $condition and exit;
+
+    $condition xor die();
+
+    echo 'still executable as exit, in all of the above cases, is used as part of an expression';
+}
+
 // Intentional syntax error.
 return array_map(