[ http://jira.codehaus.org/browse/MCHECKSTYLE-55?page=all ]

Vincent Siveton closed MCHECKSTYLE-55.
--------------------------------------

         Assignee: Vincent Siveton
       Resolution: Fixed
    Fix Version/s: 2.2

Fixed.
Remove single quote in msg due to java.text.MessageFormat specification.



> Wrong rules summary for GenericIllegalRegexp when message contains a quote
> --------------------------------------------------------------------------
>
>                 Key: MCHECKSTYLE-55
>                 URL: http://jira.codehaus.org/browse/MCHECKSTYLE-55
>             Project: Maven 2.x Checkstyle Plugin
>          Issue Type: Bug
>    Affects Versions: 2.1
>         Environment: Windows XP - Maven  2.0.4
>            Reporter: Olivier Vierlinck
>         Assigned To: Vincent Siveton
>            Priority: Minor
>             Fix For: 2.2
>
>         Attachments: MavenCheckstyleReportBug.zip
>
>
> if a GenericIllegalRegexp check is defined and its 'message' property 
> contains a quote, the check is executed ok and failures are listed in the 
> Details part of the HTML report.
> But the Rules summary part is wrong: it show the rules with 0 violations.
> For example, this check
>       <module name="GenericIllegalRegexp">
>          <property name="format" value="printStackTrace" /> <property 
> name="message" value="Don't do this" />
>       </module>
> will no be reported as violated in the rules summary (but the message text is 
> show correctly)
> But will be listed in the errors on the file (but with the quote removed from 
> the message ;-) )
> See checkstyle.xml and target/site/checkstyle.html in attachment (and sample 
> code in src\main\java\MCTest\App.java)

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

        

Reply via email to