svn commit: r979800 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckGroupSelector.java

[mgrigorov]

Author: mgrigorov
Date: Tue Jul 27 18:38:51 2010
New Revision: 979800

URL: http://svn.apache.org/viewvc?rev=979800&view=rev
Log:
WICKET-2844 Make CheckGroupSelector Ajax friendly

Use checkbox.click() instead of checkbox.checked=bool. 
#click() will trigger any registered event listeners (e.g. registered by 
AjaxFormChoiceComponentUpdatingBehavior)


Modified:
    
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckGroupSelector.java

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckGroupSelector.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckGroupSelector.java?rev=979800&r1=979799&r2=979800&view=diff
==============================================================================
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckGroupSelector.java
 (original)
+++ 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckGroupSelector.java
 Tue Jul 27 18:38:51 2010
@@ -87,7 +87,7 @@ public class CheckGroupSelector extends 
                        "onclick",
                        "var cb=this.form['" +
                                group.getInputName() +
-                               "']; if (cb!=null) { if (!isNaN(cb.length)) { 
for(var i=0;i<cb.length;i++) { cb[i].checked=this.checked; } } else { 
cb.checked=this.checked; } }");
+                               "']; if (cb!=null) { if (!isNaN(cb.length)) { 
for(var i=0;i<cb.length;i++) { if (cb[i].checked != this.checked) 
{cb[i].click();} } } else { if (cb.checked != this.checked) {cb.click();} } }");
 
                super.onComponentTag(tag);
        }