You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Alastair Maw (JIRA)" <ji...@apache.org> on 2007/06/04 13:37:37 UTC

[jira] Resolved: (WICKET-612) HeaderContributor forces relative URL

     [ https://issues.apache.org/jira/browse/WICKET-612?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alastair Maw resolved WICKET-612.
---------------------------------

    Resolution: Fixed

Yep, I think this is a reasonable use-case, even if it might confuse some users. I've updated the javadoc appropriately, so hopefully this will be mitigated.

Fixed in r544124.

> HeaderContributor forces relative URL
> -------------------------------------
>
>                 Key: WICKET-612
>                 URL: https://issues.apache.org/jira/browse/WICKET-612
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.3.0-beta1
>            Reporter: Jeremy Thomerson
>            Assignee: Alastair Maw
>            Priority: Minor
>             Fix For: 1.3.0-beta2
>
>         Attachments: wicket-patch-612.patch
>
>
> I consider this a bug, and was wondering if you do as well...  If so, I will open a JIRA and attach a patch for it.  Just let me know.
> In my 1.2.6 app, I have the following line:
> add(HeaderContributor.forCss("/resources/styles/global.css"));
> which generates this in the markup (correctly):
> <link rel ="stylesheet" type="text/css" href="/resources/css/global.css" ></link>
> In 1.3, it generates this in the markup (incorrectly):
> <link rel ="stylesheet" type="text/css" href="../../..//resources/styles/global.css" ></link>
> This is because of the change to have everything with relative URLs, I am assuming.  But, I don't want a relative URL, I want an absolute one.  I'm thinking that the following change would be appropriate:
> HeaderContributor.java (26)
> if (location.startsWith("http://") || location.startsWith("https://"))
> changed to:
> if (location.startsWith("http://") || location.startsWith("https://") || location.startsWith("/"))

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.