Maven Site Plugin
  1. Maven Site Plugin
  2. MSITE-265

filename with dot are considered as duplicate

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta-6
    • Fix Version/s: 2.0-beta-6
    • Component/s: doxia integration
    • Labels:
      None

      Description

      I have two files :
      src/site/xdoc/releases/release1.6.3.xml
      src/site/xdoc/releases/release1.6.xml

      The output is : Embedded error: Files 'releases\release1.6.xml' clashes with existing 'c:\projets\ServicesLayerServer\src\site\xdoc\releases\release1.6.3.xml'.
      As I see this comes from org.apache.maven.doxia.module.xhtml.decoration.render.RenderingContext.
      There is

       
      this.outputName = document.substring( 0, document.indexOf( "." ) ).replace( '\\', '/' ) + ".html";
      

      IMHO it should be :

       
      this.outputName = document.substring( 0, document.lastIndexOf( "." ) ).replace( '\\', '/' ) + ".html";
      

      I have to try to update dependency in maven-site-plugin to last doxia to test this fix but I have some compilation error due to some changes in doxia

      c:\maven-svn\plugins\maven-site-plugin\src\main\java\org\apache\maven\plugins\site\ReportDocumentRenderer.java:[139,26] generate(org.codehaus.doxia.sink.Sink,java.util.Locale) in org.apache.maven.reporting.MavenReport cannot be applied to (org.apache.maven.doxia.siterenderer.sink.SiteRendererSink,java.util.Locale)

      This works with 2.0-beta-5.

        Issue Links

          Activity

          Olivier Lamy (*$^¨%`£) created issue -
          Olivier Lamy (*$^¨%`£) made changes -
          Field Original Value New Value
          Description I have two files :
          src/site/xdoc/releases/release1.6.3.xml
          src/site/xdoc/releases/release1.6.xml

          The output is : Embedded error: Files 'releases\release1.6.xml' clashes with existing 'c:\projets\ServicesLayerServer\src\site\xdoc\releases\release1.6.3.xml'.
          As I see this comes from org.apache.maven.doxia.module.xhtml.decoration.render.RenderingContext.
          There is
          this.outputName = document.substring( 0, document.indexOf( "." ) ).replace( '\\', '/' ) + ".html";
          IMHO it should be :
          this.outputName = document.substring( 0, document.lastIndexOf( "." ) ).replace( '\\', '/' ) + ".html";
          I have to try to update dependency in maven-site-plugin to last doxia to test this fix but I have some compilation error due to some changes in doxia

          c:\maven-svn\plugins\maven-site-plugin\src\main\java\org\apache\maven\plugins\site\ReportDocumentRenderer.java:[139,26] generate(org.codehaus.doxia.sink.Sink,java.util.Locale) in org.apache.maven.reporting.MavenReport cannot be applied to (org.apache.maven.doxia.siterenderer.sink.SiteRendererSink,java.util.Locale)

          This works with 2.0-beta-5.
          I have two files :
          src/site/xdoc/releases/release1.6.3.xml
          src/site/xdoc/releases/release1.6.xml

          The output is : Embedded error: Files 'releases\release1.6.xml' clashes with existing 'c:\projets\ServicesLayerServer\src\site\xdoc\releases\release1.6.3.xml'.
          As I see this comes from org.apache.maven.doxia.module.xhtml.decoration.render.RenderingContext.
          There is
          {noformat}
          this.outputName = document.substring( 0, document.indexOf( "." ) ).replace( '\\', '/' ) + ".html";
          {noformat}
          IMHO it should be :
          {noformat}
          this.outputName = document.substring( 0, document.lastIndexOf( "." ) ).replace( '\\', '/' ) + ".html";
          {noformat}
          I have to try to update dependency in maven-site-plugin to last doxia to test this fix but I have some compilation error due to some changes in doxia

          c:\maven-svn\plugins\maven-site-plugin\src\main\java\org\apache\maven\plugins\site\ReportDocumentRenderer.java:[139,26] generate(org.codehaus.doxia.sink.Sink,java.util.Locale) in org.apache.maven.reporting.MavenReport cannot be applied to (org.apache.maven.doxia.siterenderer.sink.SiteRendererSink,java.util.Locale)

          This works with 2.0-beta-5.
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          This need only a new release of doxia-module-xhtml.
          Whats about a branch starting from 1.0-alpha-9 and release a doxia-module-xhtml-1.0-alpha-10.

          Thoughts ?

          Show
          Olivier Lamy (*$^¨%`£) added a comment - This need only a new release of doxia-module-xhtml. Whats about a branch starting from 1.0-alpha-9 and release a doxia-module-xhtml-1.0-alpha-10. Thoughts ?
          Olivier Lamy (*$^¨%`£) made changes -
          Fix Version/s 2.0-beta-6 [ 12151 ]
          Olivier Lamy (*$^¨%`£) made changes -
          Link This issue depends upon DOXIA-161 [ DOXIA-161 ]
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          fix in rev 586801.
          upgrade doxia version to 1.0-alpha-10-SNAPSHOT

          Show
          Olivier Lamy (*$^¨%`£) added a comment - fix in rev 586801. upgrade doxia version to 1.0-alpha-10-SNAPSHOT
          Olivier Lamy (*$^¨%`£) made changes -
          Resolution Fixed [ 1 ]
          Assignee Olivier Lamy [ olamy ]
          Status Open [ 1 ] Closed [ 6 ]
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          As DOXIA-161 has been reopened, this must be reopened.
          The it must be completed with a test on file like download.apt.vm.

          Show
          Olivier Lamy (*$^¨%`£) added a comment - As DOXIA-161 has been reopened, this must be reopened. The it must be completed with a test on file like download.apt.vm.
          Olivier Lamy (*$^¨%`£) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Hide
          Olivier Lamy (*$^¨%`£) added a comment -

          fix in doxia-branches and in doxia trunks.
          look linked issue.

          Show
          Olivier Lamy (*$^¨%`£) added a comment - fix in doxia-branches and in doxia trunks. look linked issue.
          Olivier Lamy (*$^¨%`£) made changes -
          Resolution Fixed [ 1 ]
          Status Reopened [ 4 ] Closed [ 6 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 13:16:22 UTC 2015 [ 1428239782088 ]
          Mark Thomas made changes -
          Workflow jira [ 12726650 ] Default workflow, editable Closed status [ 12757329 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 01:19:02 UTC 2015 [ 1428283142430 ]
          Mark Thomas made changes -
          Workflow jira [ 12964726 ] Default workflow, editable Closed status [ 13001731 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          12h 46m 1 Olivier Lamy (*$^¨%`£) 20/Oct/07 17:12
          Closed Closed Reopened Reopened
          2d 1h 24m 1 Olivier Lamy (*$^¨%`£) 22/Oct/07 18:36
          Reopened Reopened Closed Closed
          1d 23h 27m 1 Olivier Lamy (*$^¨%`£) 24/Oct/07 18:03

            People

            • Assignee:
              Olivier Lamy (*$^¨%`£)
              Reporter:
              Olivier Lamy (*$^¨%`£)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development