You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Dennis Lundberg (JIRA)" <ji...@codehaus.org> on 2008/02/02 19:51:58 UTC

[jira] Closed: (MCHECKSTYLE-84) Disabled "HiddenField" check

     [ http://jira.codehaus.org/browse/MCHECKSTYLE-84?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dennis Lundberg closed MCHECKSTYLE-84.
--------------------------------------

         Assignee: Dennis Lundberg
       Resolution: Fixed
    Fix Version/s: 2.2

Patch applied. Thanks!

> Disabled "HiddenField" check
> ----------------------------
>
>                 Key: MCHECKSTYLE-84
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-84
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Wish
>    Affects Versions: 2.1
>            Reporter: Benjamin Bentmann
>            Assignee: Dennis Lundberg
>            Priority: Trivial
>             Fix For: 2.2
>
>         Attachments: hidden-field-check.patch
>
>
> Now, isn't something like
> {code:java}
> public class MyBean {
>   private String property;
>   public MyBean(String property) {
>     setProperty(property);
>   }
>   public void setProperty(String property) {
>     this.property = property;
>   }
> }
> {code}
> common among Maven code? The current {{maven_checks.xml}} will complain about that twice ("'property' hides a field").
> If you agree that this is common coding, there is no need to dump errors or even warnings because these messages would become boilerplate text in the CheckStyle report and would reduce the developer's focus on the real problems.

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