Author: jkuhnert
Date: Sat Nov 11 18:55:12 2006
New Revision: 473855

URL: http://svn.apache.org/viewvc?view=rev&rev=473855
Log:
Fixes absolute url bug in AjaxShellDelegate with https (in some instances).

Modified:
    
tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/dojo/AjaxShellDelegate.java
    
tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/dojo/AjaxShellDelegateTest.java

Modified: 
tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/dojo/AjaxShellDelegate.java
URL: 
http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/dojo/AjaxShellDelegate.java?view=diff&rev=473855&r1=473854&r2=473855
==============================================================================
--- 
tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/dojo/AjaxShellDelegate.java
 (original)
+++ 
tapestry/tapestry4/trunk/tapestry-framework/src/java/org/apache/tapestry/dojo/AjaxShellDelegate.java
 Sat Nov 11 18:55:12 2006
@@ -88,7 +88,7 @@
         // The key to resolving everything out of the asset service
         
         dojoConfig.put("baseRelativePath", 
-                _assetService.getLink(true, 
_dojoPath.getResourceLocation().getPath()).getAbsoluteURL());
+                _assetService.getLink(true, 
_dojoPath.getResourceLocation().getPath()).getURL());
         
         dojoConfig.put("preventBackButtonFix", _preventBackButtonFix);
         dojoConfig.put("parseWidgets", _parseWidgets);
@@ -116,14 +116,14 @@
         str.append("<script type=\"text/javascript\" src=\"")
         .append(_assetService.getLink(true,
                 _dojoSource.getResourceLocation()
-                .getPath()).getAbsoluteURL()).append("\"></script>");
+                .getPath()).getURL()).append("\"></script>");
         
         // include core tapestry.js package
         
         str.append("<script type=\"text/javascript\" src=\"")
         .append(_assetService.getLink(true,
                 _tapestrySource.getResourceLocation()
-                .getPath()).getAbsoluteURL()).append("\"></script>");
+                .getPath()).getURL()).append("\"></script>");
         
         String logRequire = _consoleEnabled ? 
"dojo.require(\"dojo.debug.console\");\n"
                 : "dojo.require(\"dojo.logging.Logger\");\n";

Modified: 
tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/dojo/AjaxShellDelegateTest.java
URL: 
http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/dojo/AjaxShellDelegateTest.java?view=diff&rev=473855&r1=473854&r2=473855
==============================================================================
--- 
tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/dojo/AjaxShellDelegateTest.java
 (original)
+++ 
tapestry/tapestry4/trunk/tapestry-framework/src/test/org/apache/tapestry/dojo/AjaxShellDelegateTest.java
 Sat Nov 11 18:55:12 2006
@@ -48,7 +48,7 @@
         ILink link = newLink();
         
         expect(engine.getLink(Boolean.TRUE, path)).andReturn(link);
-        expect(link.getAbsoluteURL()).andReturn("http://"; + path);
+        expect(link.getURL()).andReturn("http://"; + path);
     }
     
     void trainPageLocale(IRequestCycle cycle, Locale locale)


Reply via email to