Description: ------------ The Code Sniffer is treating content inside of a docblock as PHP code and sniffing it. Test script: --------------- <?php $test = <<<TEST function test(foo, bar) { alert(foo + bar); } TEST; ?> Expected result: ---------------- No errors produced aside from docblock warnings Actual result: -------------- -------------------------------------------------------------------------------- FOUND 4 ERROR(S) AND 0 WARNING(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 4 | ERROR | Constants must be uppercase; expected FOO but found foo 4 | ERROR | Constants must be uppercase; expected BAR but found bar 5 | ERROR | Constants must be uppercase; expected FOO but found foo 5 | ERROR | Constants must be uppercase; expected BAR but found bar --------------------------------------------------------------------------------

That should be "inside of a heredoc"

I actually can't get the test case to reproduce, only a very large file with 2000-3000 lines of code that I can not legally upload. I'll working on getting a real test case.

Real test case: <?php function foo() { $output = <<<EJS ads = '{$redirect_context}'; i = i+1; EJS; } ?>

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. The reason for this is because token_get_all actually treats code within heredocs as valid PHP code. I've changed PHP_CodeSniffer to join all the tokens together and treat them as T_HEREDOC tokens (a new custom token) so they are ignored by all existing sniffs. The start and end heredoc token remain the same.