Maven 1.x XDoc Plugin
  1. Maven 1.x XDoc Plugin
  2. MPXDOC-102

[trivial, patch] all reports that don't specify target parameter in their <doc:registerReport> invocation open in new window

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.7.1
    • Labels:
      None
    • Environment:
      MAVEN-1_0 branch + HEAD of maven-plugins
    • Number of attachments :
      1

      Description

      A silly, but extremely annoying bug.

      All of the reports that are unaware of the new target functionality (almost all ATM) end up with broken links:

      <a href="..." target="$

      {report.target}

      ">...

      Attached pathc fixes the issue.

        Activity

        Hide
        Rafal Krzewski added a comment -

        patch fixing the problem

        Show
        Rafal Krzewski added a comment - patch fixing the problem
        Hide
        Arnaud Heritier added a comment -

        I applied the fix for the reports overview page.
        The modification in the navigation isn't usefull because we already test it.

        Show
        Arnaud Heritier added a comment - I applied the fix for the reports overview page. The modification in the navigation isn't usefull because we already test it.
        Hide
        Rafal Krzewski added a comment -

        Did you really test if after making the changes? For me, both changes were necessary to fix the problem. If you don't get the target="$

        {report.target}

        " stuff in your navigation without the patch, the problem needs further investigation.

        Show
        Rafal Krzewski added a comment - Did you really test if after making the changes? For me, both changes were necessary to fix the problem. If you don't get the target="$ {report.target} " stuff in your navigation without the patch, the problem needs further investigation.
        Hide
        Arnaud Heritier added a comment -

        yes I tested it
        Can I send you the new plugin or can you update it from CVS ?

        Show
        Arnaud Heritier added a comment - yes I tested it Can I send you the new plugin or can you update it from CVS ?
        Hide
        Arnaud Heritier added a comment -

        must change the affects version

        Show
        Arnaud Heritier added a comment - must change the affects version

          People

          • Assignee:
            Arnaud Heritier
            Reporter:
            Rafal Krzewski
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: