You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by jk...@apache.org on 2007/06/13 19:34:16 UTC

svn commit: r546971 - /tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js

Author: jkuhnert
Date: Wed Jun 13 10:34:15 2007
New Revision: 546971

URL: http://svn.apache.org/viewvc?view=rev&rev=546971
Log:
Fixes TAPESTRY-1206 "for real" this time.  The node.outerHTML call wasn't making previously disabled nodes enabled when the disabled attribute had been removed on the server side.  Following a hunch I re looked up the node element again after replacing its contents and the attributes magically came over.

Modified:
    tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js

Modified: tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js?view=diff&rev=546971&r1=546970&r2=546971
==============================================================================
--- tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js (original)
+++ tapestry/tapestry4/trunk/tapestry-framework/src/js/tapestry/core.js Wed Jun 13 10:34:15 2007
@@ -236,12 +236,13 @@
     	
     	var content=tapestry.html.getContentAsString(element);
     	if (djConfig["isDebug"]) {
-    		dojo.log.debug("Received element content for id <" + id + "> of: ", content);
+    		dojo.log.debug("Received element content for id <" + id + "> of: " + content);
     	}
 
         // fix for IE - setting innerHTML does not work for SELECTs
         if (tapestry.isIE && node.outerHTML && node.nodeName == "SELECT") {
             node.outerHTML = node.outerHTML.replace(/(<SELECT[^<]*>).*(<\/SELECT>)/, '$1' + content + '$2');
+            node=dojo.byId(id);
         } else {
             if (content && content.length > 0) {
                 node.innerHTML=content;