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;


Reply via email to