Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 3.7.2
Request #20121 Overloading of -v option generates unwanted output for svnblame report
Submitted: 2013-11-05 21:03 UTC
From: happydog Assigned:
Status: Open Package: PHP_CodeSniffer (version 1.4.7)
PHP Version: Irrelevant OS: N/A
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 : 37 + 34 = ?

 
 [2013-11-05 21:03 UTC] happydog (Mark Clements)
Description: ------------ Currently, if you want to include all committers in the output of the svnblame report, as opposed to just committers who are responsible for 1 or more sniff errors, then you do this by specifying the -v option. However, the -v option also controls the general verbosity of code-sniffer. This therefore generates a lot of unwanted output. Test script: --------------- phpcs --report=svnblame -v Expected result: ---------------- PHP CODE SNIFFER SVN BLAME SUMMARY -------------------------------------------------------------------------------- AUTHOR SOURCE (Author %) (Overall %) COUNT -------------------------------------------------------------------------------- author1 (0.11) (100) 27 author2 (0) (0) 0 -------------------------------------------------------------------------------- A TOTAL OF 27 SNIFF VIOLATION(S) WERE COMMITTED BY 2 AUTHOR(S) -------------------------------------------------------------------------------- Actual result: -------------- Getting SVN blame info for classCache.php... DONE Getting SVN blame info for classCacheBackend_filesystem.php... DONE Getting SVN blame info for classCacheBackend_nocache.php... DONE Getting SVN blame info for ErrorConstants.php... DONE Getting SVN blame info for classlibError.php... DONE Getting SVN blame info for classlibErrorHandling.php... DONE Getting SVN blame info for Input.php... DONE Getting SVN blame info for InputConstants.php... DONE Getting SVN blame info for classInput.php... DONE Getting SVN blame info for classInputCLI.php... DONE Getting SVN blame info for classInputWeb.php... DONE [SNIP] PHP CODE SNIFFER SVN BLAME SUMMARY -------------------------------------------------------------------------------- AUTHOR SOURCE (Author %) (Overall %) COUNT -------------------------------------------------------------------------------- author1 (0.11) (100) 27 author2 (0) (0) 0 -------------------------------------------------------------------------------- A TOTAL OF 27 SNIFF VIOLATION(S) WERE COMMITTED BY 2 AUTHOR(S) -------------------------------------------------------------------------------- The solution would be to provide an alternative argument for svnblame so that showing all users and showing verbose information are decoupled, e.g. phpcs --report=svnblame --all-authors

Comments