You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@click.apache.org by sa...@apache.org on 2008/11/17 20:44:10 UTC
svn commit: r718346 - in /incubator/click/trunk/click:
documentation/docs/roadmap-changes.html framework/src/net/sf/click/Page.java
Author: sabob
Date: Mon Nov 17 11:44:09 2008
New Revision: 718346
URL: http://svn.apache.org/viewvc?rev=718346&view=rev
Log:
fixed setRedirect. CLK-468
Modified:
incubator/click/trunk/click/documentation/docs/roadmap-changes.html
incubator/click/trunk/click/framework/src/net/sf/click/Page.java
Modified: incubator/click/trunk/click/documentation/docs/roadmap-changes.html
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/documentation/docs/roadmap-changes.html?rev=718346&r1=718345&r2=718346&view=diff
==============================================================================
--- incubator/click/trunk/click/documentation/docs/roadmap-changes.html (original)
+++ incubator/click/trunk/click/documentation/docs/roadmap-changes.html Mon Nov 17 11:44:09 2008
@@ -68,6 +68,10 @@
Fixed WebappLoader to reload Velocity global macro's
[<a target='blank' href="http://www.avoka.com/jira/browse/CLK-467">467</a>].
</li>
+ <li class="change">
+ Fixed Page.setRedirect to check if redirect path starts with context path
+ [<a target='blank' href="http://www.avoka.com/jira/browse/CLK-468">468</a>].
+ </li>
</ul>
</dd>
Modified: incubator/click/trunk/click/framework/src/net/sf/click/Page.java
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/framework/src/net/sf/click/Page.java?rev=718346&r1=718345&r2=718346&view=diff
==============================================================================
--- incubator/click/trunk/click/framework/src/net/sf/click/Page.java (original)
+++ incubator/click/trunk/click/framework/src/net/sf/click/Page.java Mon Nov 17 11:44:09 2008
@@ -984,7 +984,7 @@
String contextPath = context.getRequest().getContextPath();
// Guard against adding duplicate context path
- if (!location.startsWith(contextPath)) {
+ if (!location.startsWith(contextPath + '/')) {
location = contextPath + location;
}
}