Maven 1.x XDoc Plugin

[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

Ascending order - Click to sort in descending order
Hide
Permalink
Rafal Krzewski added a comment -

patch fixing the problem

Show
Rafal Krzewski added a comment - patch fixing the problem
Hide
Permalink
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
Permalink
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
Permalink
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
Permalink
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
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: