You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2010/01/13 03:43:08 UTC

svn commit: r898631 - /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js

Author: hlship
Date: Wed Jan 13 02:43:07 2010
New Revision: 898631

URL: http://svn.apache.org/viewvc?rev=898631&view=rev
Log:
TAP5-959: Ajax event handler methods that return a page instance, page class or page name should cause the client to redirect to that page, but doesn't work in IE7 or 8 if the URL has a query param

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js?rev=898631&r1=898630&r2=898631&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tapestry.js Wed Jan 13 02:43:07 2010
@@ -281,14 +281,7 @@
 
         if (redirectURL)
         {
-            // Check for complete URL.
-            if (/^https?:/.test(redirectURL))
-            {
-                window.location = redirectURL;
-                return;
-            }
-
-            window.location.pathname = redirectURL;
+            window.location.href = redirectURL;
 
             // Don't bother loading scripts or invoking the callback.