Generic/InlineControlStructure: fix bug when handling elseif/if/foreach without body

In PHP, `elseif`, `if` and `foreach` can be declared without a body.
The sniff was improperly flagging those as inline control structures.
This commit changes that and now the sniff does not trigger an error
anymore for elseif/if/foreach without a body. This is consistent with
the behavior of the sniff for `while` and `for` statements without a
body.

It is likely that this problem was not found before because it is hard
to imagine a case in which a `elseif`, `if` or `foreach` without a body
would be useful.

Author: rodrigoprimo

src/Standards/Generic/Sniffs/ControlStructures/InlineControlStructureSniff.php

@@ -80,8 +80,8 @@ public function process(File $phpcsFile, $stackPtr)
             }
         }
 
-        if ($tokens[$stackPtr]['code'] === T_WHILE || $tokens[$stackPtr]['code'] === T_FOR) {
-            // This could be from a DO WHILE, which doesn't have an opening brace or a while/for without body.
+        if (in_array($tokens[$stackPtr]['code'], [T_ELSEIF, T_IF, T_FOR, T_FOREACH, T_WHILE], true) === true) {
+            // This could be from a DO WHILE, which doesn't have an opening brace, or a else/if/for/foreach/while without body.
             if (isset($tokens[$stackPtr]['parenthesis_closer']) === true) {
                 $afterParensCloser = $phpcsFile->findNext(Tokens::$emptyTokens, ($tokens[$stackPtr]['parenthesis_closer'] + 1), null, true);
                 if ($afterParensCloser === false) {

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.1.inc

@@ -276,3 +276,16 @@ function testFinally()
 if ($something) {
     echo 'hello';
 } else /* comment */ if ($somethingElse) echo 'hi';
+
+if ($shouldNotTriggerSniff);
+
+if (false) {
+} elseif ($shouldNotTriggerSniff);
+
+if (false) {
+} else if ($shouldNotTriggerSniff);
+
+if (false) {
+} else ($shouldTriggerSniff);
+
+foreach ($array as $shouldNotTriggerSniff);

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.1.inc.fixed

@@ -312,3 +312,17 @@ if ($something) {
     echo 'hello';
 } else /* comment */ if ($somethingElse) { echo 'hi';
 }
+
+if ($shouldNotTriggerSniff);
+
+if (false) {
+} elseif ($shouldNotTriggerSniff);
+
+if (false) {
+} else if ($shouldNotTriggerSniff);
+
+if (false) {
+} else { ($shouldTriggerSniff);
+}
+
+foreach ($array as $shouldNotTriggerSniff);

src/Standards/Generic/Tests/ControlStructures/InlineControlStructureUnitTest.php

@@ -79,6 +79,7 @@ public function getErrorList($testFile='')
                 260 => 1,
                 269 => 1,
                 278 => 1,
+                289 => 1,
             ];
 
         case 'InlineControlStructureUnitTest.1.js':