Author: mgrigorov
Date: Fri Feb 11 16:27:22 2011
New Revision: 1069860

URL: http://svn.apache.org/viewvc?rev=1069860&view=rev
Log:
WICKET-3323 TreeTable should call attachUpdate javascript on domready event.

Render 'Wicket.TreeTable.attachUpdate()' when DOM is ready, because otherwise 
wicket-ajax.js could not be available yet

Modified:
    
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.html
    
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.java

Modified: 
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.html
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.html?rev=1069860&r1=1069859&r2=1069860&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.html
 (original)
+++ 
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.html
 Fri Feb 11 16:27:22 2011
@@ -45,10 +45,6 @@
 
 </div>
 
-<script type="text/javascript" wicket:id="attachJavascript">
-       /* Will be replaced on runtime; 
Wicket.TreeTable.attachUpdate("treeTable"); */
-</script>
-
 </wicket:panel>
 
 <wicket:fragment wicket:id="fragment">

Modified: 
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.java
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.java?rev=1069860&r1=1069859&r2=1069860&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.java
 (original)
+++ 
wicket/branches/wicket-1.4.x/wicket-extensions/src/main/java/org/apache/wicket/extensions/markup/html/tree/table/TreeTable.java
 Fri Feb 11 16:27:22 2011
@@ -27,13 +27,13 @@ import org.apache.wicket.behavior.Abstra
 import org.apache.wicket.extensions.markup.html.tree.DefaultAbstractTree;
 import 
org.apache.wicket.extensions.markup.html.tree.table.ColumnLocation.Alignment;
 import org.apache.wicket.markup.ComponentTag;
+import org.apache.wicket.markup.html.IHeaderResponse;
 import org.apache.wicket.markup.html.WebMarkupContainer;
 import org.apache.wicket.markup.html.basic.Label;
 import org.apache.wicket.markup.html.panel.Fragment;
 import org.apache.wicket.markup.html.tree.AbstractTree;
 import org.apache.wicket.model.AbstractReadOnlyModel;
 import org.apache.wicket.model.IModel;
-import org.apache.wicket.model.Model;
 
 
 /**
@@ -422,15 +422,16 @@ public class TreeTable extends DefaultAb
                // scroll together with body. The body contains vertical 
scrollbar. The
                // header width must be same as body content width, so that the 
columns
                // are properly aligned.
-               add(new Label("attachJavascript", new Model<String>()
+               add(new AbstractBehavior()
                {
                        private static final long serialVersionUID = 1L;
 
                        @Override
-                       public String getObject()
+                       public void renderHead(final IHeaderResponse response)
                        {
-                               return "Wicket.TreeTable.attachUpdate(\"" + 
getMarkupId() + "\");";
+                               
response.renderOnDomReadyJavascript("Wicket.TreeTable.attachUpdate(\"" +
+                                       getMarkupId() + "\");");
                        }
-               }).setEscapeModelStrings(false));
+               });
        }
 }
\ No newline at end of file


Reply via email to