[PEAR-BUG] Bug #18709 [Opn->Csd]: Code sniffer sniffs file if even if it's in --ignore

[email protected]
Newsgroups php.pear.bugs
Message-ID <[email protected]>
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.