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

Request #17227 Allow multipart file extensions
Submitted: 2010-03-16 02:00 UTC
From: ashnazg Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.2.2)
PHP Version: 5.2.4 OS: RHEL4
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 : 46 + 3 = ?

 
 [2010-03-16 02:00 UTC] ashnazg (Chuck Burgess)
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.

Comments

 [2010-03-16 02:02 UTC] ashnazg (Chuck Burgess)
 [2010-03-16 02:13 UTC] ashnazg (Chuck Burgess)
Original patch by Doug Hanks at Progress Rail Services, only slightly edited by me...
 [2011-04-08 10:11 UTC] squiz (Greg Sherwood)
-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.