Maven JXR

aggregate goal creates blank top-level report

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.3
  • Fix Version/s: 2.4
  • Component/s: maven2 jxr plugin
  • Labels:
    None
  • Testcase included:
    yes
  • Number of attachments :
    0

Description

Sadly, the IT for the aggregate goal does not check that there is any useful content in the file xref/index.html. It checks that individual html files are created for individual classes, but that's all. In fact, the front page is empty. Simply looking at xref/index.html from the aggregate IT is all you need to demonstrate this.

Issue Links

depends upon

Improvement - An improvement or enhancement to an existing feature or task. MSHARED-204 AbstractMavenReport should check isExternalReport() before trying to output a Doxia document

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is duplicated by

Bug - A problem which impairs or prevents the functions of the product. JXR-92 ${project.version} not substituted in xref/index.html for aggregate goal

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Benson Margulies added a comment -

The correct xref/index.html gets creared, and then overwritten by AbstractMavenReport.

Show
Benson Margulies added a comment - The correct xref/index.html gets creared, and then overwritten by AbstractMavenReport.
Hide
Permalink
Benson Margulies added a comment -

Javadoc's JavadocReport class has an execute method. JXR does not. It has an executeReport. Apparently this is the source of the distress.

Show
Benson Margulies added a comment - Javadoc's JavadocReport class has an execute method. JXR does not. It has an executeReport. Apparently this is the source of the distress.
Hide
Permalink
Benson Margulies added a comment -

I don't feel done.

Show
Benson Margulies added a comment - I don't feel done.
Hide
Permalink
Benson Margulies added a comment -

This was actually a bug in the support for jxr as a command-line mojo; it lacked an 'execute' method. In fact, it has always lacked it, and I stumbled on it.

Show
Benson Margulies added a comment - This was actually a bug in the support for jxr as a command-line mojo; it lacked an 'execute' method. In fact, it has always lacked it, and I stumbled on it.
Hide
Permalink
Herve Boutemy added a comment -

the fix should be done in AbstractMavenReport, to be available for every external report without coding: see MSHARED-204

Show
Herve Boutemy added a comment - the fix should be done in AbstractMavenReport, to be available for every external report without coding: see MSHARED-204

People

  • Assignee:
    Benson Margulies
    Reporter:
    Benson Margulies
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: