Maven Project Info Reports Plugin
  1. Maven Project Info Reports Plugin
  2. MPIR-136

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

      1. MPIR-136.patch
        44 kB
        Rémy Sanlaville

        Issue Links

          Activity

          Hide
          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
          Eric Chatellier added a comment -

          Still exist in 2.1.1

          Show
          Eric Chatellier added a comment - Still exist in 2.1.1
          Hide
          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
          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
          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.
          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 . 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
          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
          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
          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

            • Assignee:
              Herve Boutemy
              Reporter:
              Eric Chatellier
            • Votes:
              6 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: