Edit report at http://pear.php.net/bugs/bug.php?id=16182&edit=1
ID: 16182
Updated by: [email protected]
Reported By: hm2k at php dot net
Summary: Pear Doc standards.sample.php fails PHP_CodeSniffer
validation
-Status: Open
+Status: Closed
Type: Bug
Package: Documentation
PHP Version: 5.2.9
-Assigned To:
+Assigned To: cweiske
Roadmap Versions:
New Comment:
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: cweiske
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.
Previous Comments:
------------------------------------------------------------------------
[2009-05-05 13:46:36] hm2k
Description:
------------
The sample file found here:
http://pear.php.net/manual/en/standards.sample.php
Does not appear to be valid according to PHP_CodeSniffer as per the
notes.
I have checked this myself, and it is still not valid according to
PHP_CodeSniffer.
This means, that either PHP_CodeSniffer is wrong, or the documentation
is, one MUST comply.
Rather than adding another note, I have reported this as a bug so it
can be tracked.
Test script:
---------------
[root@blade php]# phpcs pear_sample_file.phps
--------------------------------------------------------------------------------
FOUND 3 ERROR(S) AND 1 WARNING(S) AFFECTING 3 LINE(S)
--------------------------------------------------------------------------------
74 | WARNING | Line exceeds 85 characters; contains 105 characters
150 | ERROR | There must be exactly one blank line before the tags
in
| | function comment
197 | ERROR | Parameters must appear immediately after the comment
197 | ERROR | Expected 1 space after the longest variable name
--------------------------------------------------------------------------------
------------------------------------------------------------------------
--
Edit this bug report at http://pear.php.net/bugs/bug.php?id=16182&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.