Edit report at http://pear.php.net/bugs/bug.php?id=18709&edit=1
ID: 18709
Updated by: [email protected]
Reported By: biozshock at gmail dot com
Summary: Code sniffer sniffs file if even if it's in --ignore
-Status: Open
+Status: Closed
Type: Bug
Package: PHP_CodeSniffer
Operating System: Ubuntu
Package Version: SVN
PHP Version: 5.3.6
-Assigned To:
+Assigned To: squiz
Roadmap Versions:
New Comment:
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: squiz
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.
Thanks for the patch.
Previous Comments:
------------------------------------------------------------------------
[2011-08-04 11:17:40] biozshock
Added #patch
bug:18709;patch:exclude_if_ine_file.patch;revision:1312453060;.
------------------------------------------------------------------------
[2011-08-04 11:16:49] biozshock
Description:
------------
Code sniffer sniffs file if one file given and this file in --ignore
pattern.
Test script:
---------------
phpcs --ignore=*system/application/views/*
--standard=./phpcs_standards/ProjectGeneric
system/application/views/widgets/movies_widget.php
Expected result:
----------------
phpcs --ignore=*system/application/views/*
--standard=./phpcs_standards/ProjectGeneric
system/application/views/widgets/movies_widget.php
Time: 0 seconds, Memory: 3.75Mb
Actual result:
--------------
phpcs --ignore=*system/application/views/*
--standard=./phpcs_standards/ProjectGeneric
system/application/views/widgets/movies_widget.php
FILE:
...ormProjects/project/system/application/views/widgets/movies_widget.php
--------------------------------------------------------------------------------
FOUND 18 ERROR(S) AND 2 WARNING(S) AFFECTING 9 LINE(S)
--------------------------------------------------------------------------------
6 | ERROR | Line indented incorrectly; expected 0 spaces, found 24
9 | ERROR | Line indented incorrectly; expected 0 spaces, found 24
11 | ERROR | Line indented incorrectly; expected 4 spaces, found 28
16 | ERROR | Line indented incorrectly; expected 0 spaces, found 24
.....
------------------------------------------------------------------------
--
Edit this bug report at http://pear.php.net/bugs/bug.php?id=18709&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.