[jira] Commented: (MPPMD-20) ruleset names

["Sivanandan Srilakshmanan (JIRA)" <ji...@codehaus.org>]

    [ http://jira.codehaus.org/browse/MPPMD-20?page=comments#action_57904 ] 

Sivanandan Srilakshmanan commented on MPPMD-20:
-----------------------------------------------

Please allow me to explain the difficulty I face when using PMD Report.

Firstly
-------
If I need to suppress "OnlyOneReturn" under the Controversial Rules I need to specify it as <exclude name="OnlyOneReturnRule"/>

Where as when I need to suppress "CallSuperInConstructor" under the Controversial Rules I need to specify it as <exclude name="CallSuperInConstructor"/>

As you can see there is no document/consistency on the naming of the actual rules.
In "OnlyOneReturn" I need to actually exclude "OnlyOneReturnRule".
On the other hand for CallSuperInConstructor it is not a problem as the rule itself is called "CallSuperInConstructor".

Secondly
--------
Controversial Rules is contained in rulesets/controversial.xml Would Security Code Guidelines be in rulesets/SecurityCodeGuidelines.xml


If I am incorrect then I am sorry for creating the confusion and would much appreciate clarification on the correct usage or a methodical approach to finding the rules and rule set.

If I am correct, then I feel that the "rulesets/SecurityCodeGuidelines.xml" should be included as part of the heading. For example "Security Code Guidelines : rulesets/SecurityCodeGuidelines.xml"
and the rule to be the name itself. ie instead of calling OnlyOneReturn" under the Controversial Rules, call it OnlyOneReturnRule.

Thanks
Lakshman


> ruleset names
> -------------
>
>          Key: MPPMD-20
>          URL: http://jira.codehaus.org/browse/MPPMD-20
>      Project: maven-pmd-plugin
>         Type: Improvement

>  Environment: Web browser
>     Reporter: Sivanandan Srilakshmanan
>     Priority: Minor
>      Fix For: 1.8

>
>
> Web page http://pmd.sourceforge.net/rules/index.html
> The PMD index should include the name of the ruleset for example 
> Braces Rules is called rulesets/braces.xml
> Jakarta Commons Logging Rules is called ??

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org