[
http://jira.codehaus.org/browse/MNG-2855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jason van Zyl closed MNG-2855.
------------------------------
Resolution: Fixed
Patch applied. Thanks.
> [PATCH] quote handling in linkPatternedText seems broken
> --------------------------------------------------------
>
> Key: MNG-2855
> URL: http://jira.codehaus.org/browse/MNG-2855
> Project: Maven 2
> Issue Type: Bug
> Components: Multiple Language Support, Sites & Reporting
> Affects Versions: 2.0.5
> Reporter: Herve Boutemy
> Attachments: AbstractMavenReportRendererTest.java,
> maven-reporting-impl-quote.patch, maven-reporting-impl-quote2.patch
>
>
> MPIR-59 shows a problem on a quote for french translation : an easy
> workaround is to double the quote '', but these 2 quotes are rendered in the
> final report, which is not what is expected.
> Then I checked the correponding code, and found inconsistencies in the way
> quotes are handled (apparently to protect braces from being interpreted).
> I wrote a JUnit test to show these inconsistencies (which in addition showed
> some bugs on special situations).
> To get consistent results, I think that single quotes should be removed when
> found: that's what is done in the patch.
> But doing so, there will be problems with existing texts that contain only
> one single quote: what used to work won't work any more, because the quote
> should be doubled (and not let without a closing quote).
> Should this patch be applied or not ? I really don't know: without it, you
> can have weird results (showed in the JUnit test). But when applying it,
> prepare yourself to some problems with existing texts: I can't say how much
> content it will break...
> At least, adding the JUnit test to the codebase will help to deal with the
> current situation.
--
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
