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

Request #18867 No warning that rhino/jslint paths are not set
Submitted: 2011-09-23 19:40 UTC
From: cweiske Assigned: squiz
Status: Analyzed Package: PHP_CodeSniffer (version 1.3.0)
PHP Version: 5.3.8 OS:
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


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 : 28 + 47 = ?

 
 [2011-09-23 19:40 UTC] cweiske (Christian Weiske)
Description: ------------ When using PHP/CodeSniffer/Standards/Squiz/Sniffs/Debug/JSLintSniff.php, I do not get a warning/notice/error when either of the required paths is not set. I don't have the possibility to find out something is wrong. When someone uses this sniff, phpcs should tell the error and exit with a non-zero exit code.

Comments

 [2011-09-26 08:33 UTC] squiz (Greg Sherwood)
-Status: Open +Status: Analyzed -Type: Bug +Type: Feature/Change Request -Assigned To: +Assigned To: squiz
This isn't a bug - it is completely intentional. But I'll leave it as a feature request, although I'm not exactly sure how it would work. Maybe a config option in the ruleset.xml file. The reason it works like this is that a standard might include everything because it wants access to all these nice external tools, but many developers wont actually use them during day-to-day development. If it threw an error, the standard could never be used until all external tools were installed and functional. On some platforms, this is impossible. So all an error would lead to is a standard that never gets widely used, or at least one that takes a lot of extra time to get working.