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

Bug #19324 Setting show_warnings configuration option does not work
Submitted: 2012-03-09 14:20 UTC
From: damd Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.3.3)
PHP Version: 5.3.10 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 : 23 - 13 = ?

 
 [2012-03-09 14:20 UTC] damd (Deniz Dogan)
Description: ------------ The documentation clearly states: To hide warnings by default, set the show_warnings configuration option to 0. phpcs --config-set show_warnings 0 This doesn't actually hide the warnings I get in my report about line lengths. Expected result: ---------------- I expect the output to not contain any WARNING lines. Actual result: -------------- I get e.g. 38 | WARNING | Line exceeds 85 characters; contains 94 characters

Comments

 [2012-03-12 05:54 UTC] squiz (Greg Sherwood)
-Summary: Documentation confusing/incorrect regarding configuring warnings +Summary: Setting show_warnings configuration option does not work -Type: Documentation Problem +Type: Bug -Assigned To: +Assigned To: squiz
The documentation is correct, but a code change broke that functionality.
 [2012-03-12 05:56 UTC] squiz (Greg Sherwood)
-Status: Assigned +Status: Closed
Fixed in git repo: https://github.com/squizlabs/PHP_CodeSniffer/commit/c8ac067ea5519c5e4d7fb8017 22eb2c7b858e844