maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dennis Lundberg (JIRA)" <j...@codehaus.org>
Subject [jira] Closed: (MCHECKSTYLE-84) Disabled "HiddenField" check
Date Sat, 02 Feb 2008 18:51:58 GMT

     [ 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

        

Mime
View raw message