Maven 2.x Project Info Reports Plugin

French properties file reference inexistent unicode char

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.1
  • Fix Version/s: 2.1.2
  • Component/s: None
  • Labels:
    None
  • Environment:
    Any
  • Number of attachments :
    1

Description

French property file (http://svn.apache.org/repos/asf/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/resources/project-info-report_fr.properties)
contains unexistant unicode char : \u0092

Should be : ' or \u0027, but test fails with that char

Issue Links

This issue is duplicated by:
MPIR-156 Bad unicode character for quote in french translation Trivial Closed
This issue is related to:
MPIR-59 Site generation exception for french language, error with simple quote in translation "S'inscrire" for mailing list Blocker Closed
This issue depends upon:
MSHARED-130 [PATCH] quote handling in linkPatternedText seems broken Major Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Herve Boutemy added a comment -

current maven-reporting-impl version is 2.0.4.1, without the fix
we'll need a new version before fixing quotes in french translation

Show
Herve Boutemy added a comment - current maven-reporting-impl version is 2.0.4.1, without the fix we'll need a new version before fixing quotes in french translation
Hide
Permalink
Eric Chatellier added a comment -

Still exist in 2.1.1

Show
Eric Chatellier added a comment - Still exist in 2.1.1
Hide
Permalink
Rémy Sanlaville added a comment -

One proposition is to use \u00B4 instead.
It's not exactly quote char but it looks like
It works with UTF-8 encoding but I don't know for others encoding.

Show
Rémy Sanlaville added a comment - One proposition is to use \u00B4 instead. It's not exactly quote char but it looks like It works with UTF-8 encoding but I don't know for others encoding.
Hide
Permalink
Rémy Sanlaville added a comment -

patch attached for the proposition to use \u00B4 instead.

In the meantime a better solution is found, is it possible to apply the patch and deploy a new version ?

Thanks

Show
Rémy Sanlaville added a comment - patch attached for the proposition to use \u00B4 instead. In the meantime a better solution is found, is it possible to apply the patch and deploy a new version ? Thanks
Hide
Permalink
Herve Boutemy added a comment -

what is the output encoding you're using, for which this char does not exist?

Unicode Character 'PRIVATE USE TWO' (U+0092) should be available, since it's avaiable in every ISO-8859 encodings
I only find Windows CP1252 where it is not available.
But since maven-site-plugin's default encoding was ISO-8859-1 before 2.0 and is UTF-8 since 2.0 (see Reporting Encoding Configuration), this should be a problem only if you explicitely configured maven-site-plugin tu use CP1252. Or perhaps the font used does not have this char.

proposed Unicode Character 'ACUTE ACCENT' (U+00B4) does not have this problem since it is both in ISO-8859 and in CP1252.
This could be a workaround: thanks for the proposal Rémy.

But in fact it would be a workaround of the workaround, since the real problem is [quote handling|MNG-2855].
I'm actually working to release maven-reporting-impl 2.0.4.2 to fix quote handling, then I'll integrate it in MPIR. Stay tuned...

Show
Herve Boutemy added a comment - what is the output encoding you're using, for which this char does not exist? Unicode Character 'PRIVATE USE TWO' (U+0092) should be available, since it's avaiable in every ISO-8859 encodings I only find Windows CP1252 where it is not available. But since maven-site-plugin's default encoding was ISO-8859-1 before 2.0 and is UTF-8 since 2.0 (see Reporting Encoding Configuration), this should be a problem only if you explicitely configured maven-site-plugin tu use CP1252. Or perhaps the font used does not have this char. proposed Unicode Character 'ACUTE ACCENT' (U+00B4) does not have this problem since it is both in ISO-8859 and in CP1252. This could be a workaround: thanks for the proposal Rémy. But in fact it would be a workaround of the workaround, since the real problem is [quote handling|MNG-2855]. I'm actually working to release maven-reporting-impl 2.0.4.2 to fix quote handling, then I'll integrate it in MPIR. Stay tuned...
Hide
Permalink
Herve Boutemy added a comment -

fixed in r771130: now, normal quotes are used, no more exotic Unicode chars
please test and open a Jira issue if problems persist for some texts (I have checked but may have missed some)

Show
Herve Boutemy added a comment - fixed in r771130: now, normal quotes are used, no more exotic Unicode chars please test and open a Jira issue if problems persist for some texts (I have checked but may have missed some)
Hide
Permalink
Rémy Sanlaville added a comment -

We try it and it works.
Could you deploy a new snapshot ?

Thanks

Show
Rémy Sanlaville added a comment - We try it and it works. Could you deploy a new snapshot ? Thanks
Hide
Permalink
Olivier Lamy added a comment -

done.

[INFO] Retrieving previous build number from apache.snapshots.https
Uploading: https://repository.apache.org/content/repositories/snapshots/org/apache/maven/plugins/maven-project-info-reports-plugin/2.1.2-SNAPSHOT/maven-project-info-reports-plugin-2.1.2-20090528.203528-2.jar
212K uploaded  (maven-project-info-reports-plugin-2.1.2-20090528.203528-2.jar)
Show
Olivier Lamy added a comment - done. 
[INFO] Retrieving previous build number from apache.snapshots.https
Uploading: https://repository.apache.org/content/repositories/snapshots/org/apache/maven/plugins/maven-project-info-reports-plugin/2.1.2-SNAPSHOT/maven-project-info-reports-plugin-2.1.2-20090528.203528-2.jar
212K uploaded  (maven-project-info-reports-plugin-2.1.2-20090528.203528-2.jar)

People

Vote (6)
Watch (5)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.