BadFunctions/EasyRFI: bug fix - fix detecting of start/end of the statement [3]

An `include`/`require` statement can be used within template files to include another template and doesn't need a closing semi-colon in that case.

That situation was so far not being considered by the sniff and the sniff could in that case search way to far and report on completely unrelated statements after the include.

Author: jrfnl

Security/Sniffs/BadFunctions/EasyRFISniff.php

@@ -38,7 +38,7 @@ public function register() {
 	* @return void
 	*/
 	public function process(File $phpcsFile, $stackPtr) {
-		$closer = $phpcsFile->findNext(T_SEMICOLON, ($stackPtr + 1));
+		$closer = $phpcsFile->findNext(array(T_SEMICOLON, T_CLOSE_TAG), ($stackPtr + 1));
 		if ($closer === false) {
 			// Live coding or parse error.
 			return;

Security/Tests/BadFunctions/EasyRFIUnitTest.0.inc

@@ -16,5 +16,10 @@ include arg(2) . drupal_get_query_parameters()['param'];
 // Prevent false positives on safe $_SERVER variables.
 include $_SERVER['DOCUMENT_ROOT'] . '/filename.php';
 
+?>
+<?php include $_POST['path'] ?><!-- Error. -->
+<?php
+echo function_call($param);
+
 // Intentional parse error. This should be the last test in the file.
 require_once

Security/Tests/BadFunctions/EasyRFIUnitTest.1.inc

@@ -16,5 +16,10 @@ include arg(2) . drupal_get_query_parameters()['param']; // Warning x 2.
 // Prevent false positives on safe $_SERVER variables.
 include $_SERVER['DOCUMENT_ROOT'] . '/filename.php'; // Error.
 
+?>
+<?php include $_POST['path'] ?><!-- Error. -->
+<?php
+echo function_call($param);
+
 // Intentional parse error. This should be the last test in the file.
 require $_GET['path']

Security/Tests/BadFunctions/EasyRFIUnitTest.php

@@ -29,13 +29,15 @@ public function getErrorList($testFile = '')
 				return [
 					8  => 1,
 					10 => 1,
+					20 => 1,
 				];
 
 			case 'EasyRFIUnitTest.1.inc':
 				return [
 					8  => 1,
 					10 => 1,
 					17 => 1,
+					20 => 1,
 				];
 
 			case 'EasyRFIUnitTest.Drupal7.1.inc':