Maven JXR

JXR report html malformed

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0-beta-1
  • Fix Version/s: 1.0
  • Component/s: jxr
  • Labels:
    None
  • Number of attachments :
    0

Description

Here's a sample of the HTML generated. Note the invalid links and the unresolved $name variable:

<html xml:lang="en" lang="en">
<head>
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1" />
<title>WSF TripleStore SPI 2.2.0-SNAPSHOT Reference Package $name</title>
<link rel="stylesheet" type="text/css" href="\.\./\.\./\.\./\.\./\.\./stylesheet.css" title="style" />
</head>
<body>
<div class="overview">

<ul>
<li>
<a href="\.\./\.\./\.\./\.\./\.\./overview-summary.html">Overview</a>
</li>
<li class="selected">Package</li>
</ul>
</div>
<div class="framenoframe">

<ul>
<li>
<a href="\.\./\.\./\.\./\.\./\.\./index.html" target="_top">FRAMES</a>
</li>

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Mike Perham added a comment -

Where is the maven-jxr source code? I'll fix this and submit a patch if only I knew where it was...

Show
Mike Perham added a comment - Where is the maven-jxr source code? I'll fix this and submit a patch if only I knew where it was...
Hide
Permalink
Emmanuel Venisse added a comment -

https://svn.apache.org/repos/asf/maven/jxr/trunk

Show
Emmanuel Venisse added a comment - https://svn.apache.org/repos/asf/maven/jxr/trunk
Hide
Permalink
Mike Perham added a comment -

Here's an ugly but working patch:

Index: src/main/java/org/apache/maven/jxr/DirectoryIndexer.java
===================================================================
— src/main/java/org/apache/maven/jxr/DirectoryIndexer.java (revision 354452)
+++ src/main/java/org/apache/maven/jxr/DirectoryIndexer.java (working copy)
@@ -347,6 +347,8 @@
String pkgName = pkg.getName();
String pkgDir = perl.substitute( "s/\\./
//g", pkgName );
String rootRef = perl.substitute( "s/[^\\.]*(\\.|$)/\\.\\.
//g", pkgName );
+ // TODO Ugly hack to remove spurious backslashes, see JXR-5
+ rootRef = perl.substitute( "s/\\\\//g", rootRef );

// special case for the default package
// javadoc doesn't deal with it, but it's easy for us

Show
Mike Perham added a comment - Here's an ugly but working patch: Index: src/main/java/org/apache/maven/jxr/DirectoryIndexer.java =================================================================== — src/main/java/org/apache/maven/jxr/DirectoryIndexer.java (revision 354452) +++ src/main/java/org/apache/maven/jxr/DirectoryIndexer.java (working copy) @@ -347,6 +347,8 @@ String pkgName = pkg.getName(); String pkgDir = perl.substitute( "s/\\./ //g", pkgName ); String rootRef = perl.substitute( "s/ [^\\.] *(\\.|$)/\\.\\. //g", pkgName ); + // TODO Ugly hack to remove spurious backslashes, see JXR-5 + rootRef = perl.substitute( "s/\\\\//g", rootRef ); // special case for the default package // javadoc doesn't deal with it, but it's easy for us
Hide
Permalink
fabrizio giustina added a comment -

Committed fix for the unresolved $name variable, link generation has already been fixed

Show
fabrizio giustina added a comment - Committed fix for the unresolved $name variable, link generation has already been fixed

People

  • Assignee:
    fabrizio giustina
    Reporter:
    Mike Perham
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: