File::addMessage(): do not ignore Internal errors when scanning sel…

…ectively When either the `--sniffs=...` CLI parameter is used, or the `--exclude=...` CLI parameter, the `File::addMessage()` method bows out when an error is passed which is not for one of the selected sniffs/is for one of the excluded sniffs. Unfortunately, this "bowing out" did not take `Internal` errors into account, meaning those were now hidden, while those should _always_ be thrown as they generally inform the end-user of something seriously wrong (mixed line endings/no code found etc). Fixed now. Includes updating two test files to allow for seeing internal errors.
squizlabs · Oct 31, 2023 · 2e4ec40 · 2e4ec40
1 parent 7126fad
commit 2e4ec40
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 0 deletions.
diff --git a/src/Files/File.php b/src/Files/File.php
@@ -909,6 +909,7 @@ protected function addMessage($error, $message, $line, $column, $code, $data, $s
         // Filter out any messages for sniffs that shouldn't have run
         // due to the use of the --sniffs command line argument.
         if ($includeAll === false
+            && $parts[0] !== 'Internal'
             && ((empty($this->configCache['sniffs']) === false
             && in_array(strtolower($listenerCode), $this->configCache['sniffs'], true) === false)
             || (empty($this->configCache['exclude']) === false

diff --git a/src/Standards/Generic/Tests/PHP/DisallowAlternativePHPTagsUnitTest.php b/src/Standards/Generic/Tests/PHP/DisallowAlternativePHPTagsUnitTest.php
@@ -89,7 +89,19 @@ public function getErrorList($testFile='')
     public function getWarningList($testFile='')
     {
         if ($testFile === 'DisallowAlternativePHPTagsUnitTest.3.inc') {
+            // Check if the Internal.NoCodeFound error can be expected on line 1.
+            $aspTags = false;
+            if (PHP_VERSION_ID < 70000) {
+                $aspTags = (bool) ini_get('asp_tags');
+            }
+
+            $line1 = 0;
+            if ($aspTags === true) {
+                $line1 = 1;
+            }
+
             return [
+                1 => $line1,
                 3 => 1,
                 4 => 1,
                 5 => 1,

diff --git a/src/Standards/Generic/Tests/PHP/DisallowShortOpenTagUnitTest.php b/src/Standards/Generic/Tests/PHP/DisallowShortOpenTagUnitTest.php
@@ -88,7 +88,14 @@ public function getWarningList($testFile='')
         case 'DisallowShortOpenTagUnitTest.1.inc':
             return [];
         case 'DisallowShortOpenTagUnitTest.3.inc':
+            // Check if the Internal.NoCodeFound error can be expected on line 1.
+            $option = (bool) ini_get('short_open_tag');
+            $line1  = 0;
+            if ($option === true) {
+                $line1 = 1;
+            }
             return [
+                1  => $line1,
                 3  => 1,
                 6  => 1,
                 11 => 1,