Author: ivaynberg Date: Sat Dec 15 10:44:08 2007 New Revision: 604485 URL: http://svn.apache.org/viewvc?rev=604485&view=rev Log: WICKET-1221 required check doesnt really make sense for checkboxes
Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java Modified: wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java URL: http://svn.apache.org/viewvc/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java?rev=604485&r1=604484&r2=604485&view=diff ============================================================================== --- wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java (original) +++ wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/html/form/CheckBox.java Sat Dec 15 10:44:08 2007 @@ -191,8 +191,8 @@ // NOTE: do not encode the url as that would give invalid // JavaScript tag.put("onclick", "window.location.href='" + url + - (url.toString().indexOf('?') > -1 ? "&" : "?") + getInputName() + - "=' + this.checked;"); + (url.toString().indexOf('?') > -1 ? "&" : "?") + getInputName() + + "=' + this.checked;"); } } @@ -222,7 +222,15 @@ catch (StringValueConversionException e) { throw new ConversionException("Invalid boolean input value posted \"" + getInput() + - "\"", e).setTargetType(Boolean.class); + "\"", e).setTargetType(Boolean.class); } + } + + /** + * @see org.apache.wicket.markup.html.form.FormComponent#checkRequired() + */ + public boolean checkRequired() + { + return true; } }