Maven 2.x and 3.x Site Plugin

poweredBy Logo links don't work

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Duplicate
  • Affects Version/s: 2.0-beta-5
  • Fix Version/s: None
  • Component/s: relative links
  • Labels:
    None
  • Number of attachments :
    1

Description

I added the following poweredBy Item to my site:

<poweredBy>
<logo name="Apache Velocity"
href="http://velocity.apache.org"
img="/images/pbv90x30.png" />
</poweredBy>

This gets rendered out to

<a href="" title="Apache Velocity" id="poweredBy">
<img alt="Apache Velocity" src="images/pbv90x30.png" />
</a>

The href link is empty. When debugging, it seems that any URL given to the logo href attribute gets silently removed. Using href="foobar" works, href="http://www.apache.org/" does not.

Issue Links

Activity

Hide
Dennis Lundberg added a comment -

I guess that you have this in your pom as well:

<url>http://velocity.apache.org</url>

This has to do with the converting of absolute URL:s to relative links.

Show
Dennis Lundberg added a comment - I guess that you have this in your pom as well: <url>http://velocity.apache.org</url> This has to do with the converting of absolute URL:s to relative links.
Hide
Dennis Lundberg added a comment -

Here's a small project that displays this behavior. The default config has a href="" on the poweredBy logo, because the url in the pom is the same as the url for the poweredBy logo.

If you change the url in the pom to the maven.apache.org one that is currently commented out, you will see that the href on the poweredBy logo will be different.

Show
Dennis Lundberg added a comment - Here's a small project that displays this behavior. The default config has a href="" on the poweredBy logo, because the url in the pom is the same as the url for the poweredBy logo. If you change the url in the pom to the maven.apache.org one that is currently commented out, you will see that the href on the poweredBy logo will be different.
Hide
Dennis Lundberg added a comment -

Another try, this time without the generated site in it.

Show
Dennis Lundberg added a comment - Another try, this time without the generated site in it.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: