| 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) | ||
[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