Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.8.6

Request #14791 Improve PHPCS Results
Submitted: 2008-10-14 10:22 UTC
From: doconnor Assigned: amir
Status: Closed Package: Validate (version CVS)
PHP Version: 5.2.6 OS:
Roadmaps: (Not assigned)    
Subscription  


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 17 + 42 = ?

 
 [2008-10-14 10:22 UTC] doconnor (Daniel O'Connor)
Description: ------------ FILE: /home/clockwerx/pear/Validate/Validate.php -------------------------------------------------------------------------------- FOUND 134 ERROR(S) AND 65 WARNING(S) AFFECTING 165 LINE(S) See also: http://pear.cweiske.de/showstats.php?p=Validate http://pear.cweiske.de/codingstandards/Validate.html

Comments

 [2008-10-14 10:54 UTC] doconnor (Daniel O'Connor)
Ugh, lets hope that you can simply join those two back together neatly to get a fuller patch. Addresses all of the PHPCS errors apart from the unfixable ones.
 [2008-10-26 03:28 UTC] doconnor (Daniel O'Connor)
See also Bug #14377
 [2008-10-26 07:56 UTC] amir (Amir Mohammad Saied)
Progressing! FILE: /home/amir/vc/checkouts/cvs/pear/Validate/Validate.php -------------------------------------------------------------------------------- FOUND 28 ERROR(S) AND 48 WARNING(S) AFFECTING 74 LINE(S) --------------------------------------------------------------------------------
 [2008-12-06 14:45 UTC] doconnor (Daniel O'Connor)
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. Part of the commit for Bug #14931