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

Site generation exception for french language, error with simple quote in translation "S'inscrire" for mailing list

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.1
    • Component/s: None
    • Labels:
      None
    • Environment:
      linux Debian sarge
      jdk 1.6
      maven 2.0.4
    • Number of attachments :
      1

      Description

      When we try to generate site in french <locales>fr</locales> in site plugin we have exception:

      [ERROR] FATAL ERROR
      [INFO] ------------------------------------------------------------------------
      [INFO] Unmatched braces in the pattern in '

      {S'incrire, http://lists.labs.libre-entreprise.org/mailman/listinfo/lutinutil-cvscommit}

      '
      [INFO] ------------------------------------------------------------------------
      [INFO] Trace
      java.lang.IllegalArgumentException: Unmatched braces in the pattern.
      at org.apache.maven.reporting.AbstractMavenReportRenderer.applyPattern(AbstractMavenReportRenderer.java:619)
      at org.apache.maven.reporting.AbstractMavenReportRenderer.linkPatternedText(AbstractMavenReportRenderer.java:353)
      at org.apache.maven.reporting.AbstractMavenReportRenderer.tableCell(AbstractMavenReportRenderer.java:213)
      at org.apache.maven.reporting.AbstractMavenReportRenderer.tableCell(AbstractMavenReportRenderer.java:193)
      at org.apache.maven.reporting.AbstractMavenReportRenderer.tableRow(AbstractMavenReportRenderer.java:225)
      at org.apache.maven.report.projectinfo.MailingListsReport$MailingListsRenderer.renderBody(MailingListsReport.java:238)
      at org.apache.maven.reporting.AbstractMavenReportRenderer.render(AbstractMavenReportRenderer.java:65)
      at org.apache.maven.report.projectinfo.MailingListsReport.executeReport(MailingListsReport.java:65)
      at org.apache.maven.reporting.AbstractMavenReport.generate(AbstractMavenReport.java:101)
      ...

      in applyPattern some test is done to know if simple quote \' are opened but this test is not good for some translation as: report.mailing-lists.column.subscribe = S'incrire

      because this become:

      {S'incrire, http://lists.labs.libre-entreprise.org/mailman/listinfo/lutinutil-cvscommit}

      and applyPattern method think that quote is nether close.
      quick fix is to put two simple quote: S''incrire
      but this render two simple quote in html page.
      I think applyPattern must be rewritten to correct this problem

        Issue Links

          Activity

          Hide
          Fabrice Daugan added a comment -

          I've also the same error with the site goal into French :
          java.lang.IllegalArgumentException: Unmatched braces in the pattern.
          at org.apache.maven.reporting.AbstractMavenReportRenderer.applyPattern(AbstractMavenReportRenderer.java:619)
          at org.apache.maven.reporting.AbstractMavenReportRenderer.linkPatternedText(AbstractMavenReportRenderer.java:353)
          at org.apache.maven.reporting.AbstractMavenReportRenderer.tableCell(AbstractMavenReportRenderer.java:213)
          at org.apache.maven.reporting.AbstractMavenReportRenderer.tableCell(AbstractMavenReportRenderer.java:193)
          at org.apache.maven.reporting.AbstractMavenReportRenderer.tableRow(AbstractMavenReportRenderer.java:225)
          at org.apache.maven.report.projectinfo.MailingListsReport$MailingListsRenderer.renderBody(MailingListsReport.java:238)
          at org.apache.maven.reporting.AbstractMavenReportRenderer.render(AbstractMavenReportRenderer.java:65)
          at org.apache.maven.report.projectinfo.MailingListsReport.executeReport(MailingListsReport.java:65)
          at org.apache.maven.reporting.AbstractMavenReport.generate(AbstractMavenReport.java:101)
          at org.apache.maven.plugins.site.ReportDocumentRenderer.renderDocument(ReportDocumentRenderer.java:67)
          at org.apache.maven.doxia.siterenderer.DefaultSiteRenderer.renderModule(DefaultSiteRenderer.java:239)
          at org.apache.maven.doxia.siterenderer.DefaultSiteRenderer.render(DefaultSiteRenderer.java:115)
          at org.apache.maven.plugins.site.SiteMojo.renderLocale(SiteMojo.java:124)
          at org.apache.maven.plugins.site.SiteMojo.execute(SiteMojo.java:92)
          ...

          Remove the mailling list info from my pom make thr site to be correctly generated.

          Show
          Fabrice Daugan added a comment - I've also the same error with the site goal into French : java.lang.IllegalArgumentException: Unmatched braces in the pattern. at org.apache.maven.reporting.AbstractMavenReportRenderer.applyPattern(AbstractMavenReportRenderer.java:619) at org.apache.maven.reporting.AbstractMavenReportRenderer.linkPatternedText(AbstractMavenReportRenderer.java:353) at org.apache.maven.reporting.AbstractMavenReportRenderer.tableCell(AbstractMavenReportRenderer.java:213) at org.apache.maven.reporting.AbstractMavenReportRenderer.tableCell(AbstractMavenReportRenderer.java:193) at org.apache.maven.reporting.AbstractMavenReportRenderer.tableRow(AbstractMavenReportRenderer.java:225) at org.apache.maven.report.projectinfo.MailingListsReport$MailingListsRenderer.renderBody(MailingListsReport.java:238) at org.apache.maven.reporting.AbstractMavenReportRenderer.render(AbstractMavenReportRenderer.java:65) at org.apache.maven.report.projectinfo.MailingListsReport.executeReport(MailingListsReport.java:65) at org.apache.maven.reporting.AbstractMavenReport.generate(AbstractMavenReport.java:101) at org.apache.maven.plugins.site.ReportDocumentRenderer.renderDocument(ReportDocumentRenderer.java:67) at org.apache.maven.doxia.siterenderer.DefaultSiteRenderer.renderModule(DefaultSiteRenderer.java:239) at org.apache.maven.doxia.siterenderer.DefaultSiteRenderer.render(DefaultSiteRenderer.java:115) at org.apache.maven.plugins.site.SiteMojo.renderLocale(SiteMojo.java:124) at org.apache.maven.plugins.site.SiteMojo.execute(SiteMojo.java:92) ... Remove the mailling list info from my pom make thr site to be correctly generated.
          Hide
          Herve Boutemy added a comment -

          Here is a patch for french translation, with much more fixes than this quote problem.

          For the quote problem, I just wrote two simple quote, which leads to reported problem: both quotes are rendered in the report. But at least, there is no exception any more.

          I looked at quote handling in applyPattern, and got to the same conclusion: quote handling is broken. I opened a specific issue for this: MNG-2855. There is a patch, but its impact is rather important. Perhaps we'll have to live for a moment with these two quotes in french reports

          Show
          Herve Boutemy added a comment - Here is a patch for french translation, with much more fixes than this quote problem. For the quote problem, I just wrote two simple quote, which leads to reported problem: both quotes are rendered in the report. But at least, there is no exception any more. I looked at quote handling in applyPattern, and got to the same conclusion: quote handling is broken. I opened a specific issue for this: MNG-2855 . There is a patch, but its impact is rather important. Perhaps we'll have to live for a moment with these two quotes in french reports
          Hide
          Herve Boutemy added a comment -

          quick fix is to put two simple quote: S''inscrire
          but this render two simple quote in html page.
          I think applyPattern must be rewritten to correct this problem

          Show
          Herve Boutemy added a comment - quick fix is to put two simple quote: S''inscrire but this render two simple quote in html page. I think applyPattern must be rewritten to correct this problem
          Hide
          Arnaud Heritier added a comment -

          Patch applied. Snapshot deployed (2.1-20070401.230737-6).
          I replaced all ' characters by \u0092.

          Show
          Arnaud Heritier added a comment - Patch applied. Snapshot deployed (2.1-20070401.230737-6). I replaced all ' characters by \u0092.
          Hide
          Rémy Sanlaville added a comment -

          For information, it does not work with UTF-8 encoding
          Cf . MPIR-136

          Show
          Rémy Sanlaville added a comment - For information, it does not work with UTF-8 encoding Cf . MPIR-136

            People

            • Assignee:
              Arnaud Heritier
              Reporter:
              Benjamin POUSSIN
            • Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: