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;