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

ID: 17227
Updated by: gsherwood@squiz.net
Reported By: demon dot gene at gmail dot com
Summary: Allow multipart file extensions
-Status: Open
+Status: Closed
Type: Feature/Change Request
Package: PHP_CodeSniffer
Operating System: RHEL4
Package Version: 1.2.2
PHP Version: 5.2.4
-Assigned To:
+Assigned To: squiz
Roadmap Versions:
New Comment:

-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: squiz
I did it in a slightly different way, but the result is the same.
Committed to SVN.


Previous Comments:
------------------------------------------------------------------------

[2010-03-15 22:13:51] ashnazg

Original patch by Doug Hanks at Progress Rail Services, only slightly
edited by me...

------------------------------------------------------------------------

[2010-03-15 22:02:09] ashnazg

Added #patch
bug:17227;patch:allow-multipart-extensions;revision:1268690529;.

------------------------------------------------------------------------

[2010-03-15 22:00:59] ashnazg

Description:
------------
The --extensions argument should allow multipart file extensions, e.g.
for file foo.class.php, the extension of class.php is recognizable.

The attached patch implements this.

Test script:
---------------
Files:
- foo.class.php
- class.php
- bar.php

Run:
phpcs --extensions=class.php ./

Expected result:
----------------
Report only includes foo.class.php

Actual result:
--------------
No files match the extension mask, so report is empty.

------------------------------------------------------------------------

Search Discussions

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedApr 8, '11 at 5:10a
activeApr 8, '11 at 5:10a
posts1
users1
websitepear.php.net

1 user in discussion

Gsherwood: 1 post

People

Translate

site design / logo © 2022 Grokbase