Edit report at http://pear.php.net/bugs/bug.php?id=18191&edit=1

ID: 18191
Comment by: fmazerolle
Reported By: fmazerolle at idealtechnology dot net
Summary: Squiz.PHP.LowercasePHPFunctions does not work with "new
Status: Closed
Type: Bug
Package: PHP_CodeSniffer
Package Version: 1.3.0RC1
PHP Version: Irrelevant
Assigned To: squiz
New Comment:

I've copied the file from the GIT repository, and replaced the file (
) of my current installation, and I still have the issue.

Also, I see nothing in the code that check if an object exist, check if
the "function" is preceeded by a new tags, or an exclude list.

Are you sure it's fixed, what's the patch ?

Previous Comments:

[2011-01-13 23:23:38] squiz

<div id="changeset">
<span class="removed">-Status: Open</span>
<span class="added">+Status: Closed</span>
<span class="removed">-Assigned To:</span>
<span class="added">+Assigned To: squiz</span>
</div>This bug has been fixed in SVN.

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.


[2011-01-13 10:09:55] buchi

Squiz.PHP.LowercasePHPFunctions denotes an error when instantiating the
PEAR class Date: "new Date()"

Error message:
Calls to inbuilt PHP functions must be lowercase; expected
"date" but found "Date"

This is wrong becuse this is no function call of a built-in PHP


Search Discussions

Discussion Posts

Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 1 of 2 | next ›
Discussion Overview
grouppear-bugs @
postedNov 15, '11 at 6:11p
activeNov 21, '11 at 5:14a

2 users in discussion

Fmazerolle: 1 post Gsherwood: 1 post



site design / logo © 2021 Grokbase