[
https://issues.apache.org/jira/browse/TAPESTRY-2033?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kevin Menard updated TAPESTRY-2033:
-----------------------------------
Attachment: TAPESTRY-2033.patch
This code removes a "-1" check. Presumably this code was in there in order to
prevent an AIOOB exception, but the rest of the code makes that impossible
anyway.
The problem, using the example above, is that I would have the following:
requestTerms[0] == ""
requestTerms[1] == "manager"
pathTerms[0] == ""
pathTerms[1] == "manager"
pathTerms[2] == "assets"
.
.
.
The first path through the loop would detect that "" was duplicated, but the
second pass would not detect that "manager" was duplicated because the first
conditional short-circuited the whole loop (i == 1; requestTerms.length = 2
- 1 == 1)
Making this change fixed my app. Relative URLs seem to work throughout with
"../" being added as appropriate.
> TAPESTRY-1502 breaks non-root contexts.
> ---------------------------------------
>
> Key: TAPESTRY-2033
> URL: https://issues.apache.org/jira/browse/TAPESTRY-2033
> Project: Tapestry
> Issue Type: Bug
> Affects Versions: 5.0.8
> Reporter: Kevin Menard
> Attachments: TAPESTRY-2033.patch
>
>
> TAPESTRY-1502 changed the way that URLs were generated. The comments on the
> issue indicate it works great for a non-root context, but my experience is
> indicating otherwise. It is possible I've done something wrong, but without
> changing a single line of code, all of my pagelinks, actionlinks,
> includescripts, includestylesheets, and so forth have broken.
> In this case, I have my app mapped to the "/manager" context. I access it
> locally via http://localhost:8080/manager/.
> The generated URLs are all of the form:
> <script src="manager/assets/scriptaculous/prototype.js"
> type="text/javascript"></script>
> As you can see, the context is being included in the path but done so
> relatively. The above would resolve to
> http://localhost:8080/manager/manager/assets/scriptaculous/prototype.js,
> which is incorrect. It should be
> http://localhost:8080/manager/assets/scriptaculous/prototype.js
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
