Author: ivaynberg
Date: Fri Oct 31 16:49:31 2008
New Revision: 709603

URL: http://svn.apache.org/viewvc?rev=709603&view=rev
Log:
remove value attr from checkboxes

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

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java?rev=709603&r1=709602&r2=709603&view=diff
==============================================================================
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java
 (original)
+++ 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java
 Fri Oct 31 16:49:31 2008
@@ -149,6 +149,10 @@
                        tag.remove("checked");
                }
 
+               // remove value attribute, because it overrides the browser's 
submitted value, eg a [input
+               // type="checkbox" value=""] will always submit as false
+               tag.remove("value");
+
                // Should a roundtrip be made (have onSelectionChanged called) 
when the
                // checkbox is clicked?
                if (wantOnSelectionChangedNotifications())


Reply via email to