Squiz/NonExecutableCode: bug fix - expressions in ternary

The `T_EXIT` token can be used in ternaries without affecting code after the ternary expression.

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

Fixed now.

Includes unit test.

Fixes 2857

Author: jrfnl

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

@@ -68,6 +68,11 @@ public function process(File $phpcsFile, $stackPtr)
             if (isset(Tokens::$booleanOperators[$tokens[$prev]['code']]) === true) {
                 return;
             }
+
+            // Expressions are allowed in the `else` clause of ternaries.
+            if ($tokens[$prev]['code'] === T_INLINE_THEN || $tokens[$prev]['code'] === T_INLINE_ELSE) {
+                return;
+            }
         }
 
         // Check if this token is actually part of a one-line IF or ELSE statement.

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

@@ -323,5 +323,17 @@ function exitExpressionsWithLogicalOperators() {
     echo 'still executable as exit, in all of the above cases, is used as part of an expression';
 }
 
+// Inline expressions are allowed in ternaries.
+function exitExpressionsInTernary() {
+    $value = $myValue ? $myValue : exit();
+    $value = $myValue ?: exit();
+    $value = $var == 'foo' ? 'bar' : die( 'world' );
+
+    $value = (!$myValue ) ? exit() : $myValue;
+    $value = $var != 'foo' ? die( 'world' ) : 'bar';
+
+    echo 'still executable';
+}
+
 // Intentional syntax error.
 return array_map(