Default value for a CheckBox is now false and not null. Removed

old booleanValue as getValue() returns true/false now for all basic
cases

1 files changed, 2 insertions, 16 deletions

diff --git a/src/com/vaadin/ui/CheckBox.java b/src/com/vaadin/ui/CheckBox.java
index 358e2a62c6..250ab5f095 100644
--- a/src/com/vaadin/ui/CheckBox.java
+++ b/src/com/vaadin/ui/CheckBox.java
@@ -21,6 +21,7 @@ public class CheckBox extends AbstractField<Boolean> {
      * Creates a new checkbox.
      */
     public CheckBox() {
+        setValue(Boolean.FALSE);
     }
 
     /**
@@ -68,7 +69,7 @@ public class CheckBox extends AbstractField<Boolean> {
     public void paintContent(PaintTarget target) throws PaintException {
         super.paintContent(target);
 
-        target.addVariable(this, VCheckBox.VARIABLE_STATE, booleanValue());
+        target.addVariable(this, VCheckBox.VARIABLE_STATE, getValue());
     }
 
     @Override
@@ -114,19 +115,4 @@ public class CheckBox extends AbstractField<Boolean> {
 
     }
 
-    /**
-     * Get the boolean value of the checkbox state.
-     * 
-     * @return True iff the checkbox is checked.
-     * @deprecated in Vaadin 7.0.0. Retained to ease migration from Vaadin 6
-     */
-    @Deprecated
-    public boolean booleanValue() {
-        // FIXME: How should null really be handled? A default converter that
-        // converts it to false? The only UI values supported are true and
-        // false.
-        Boolean value = getValue();
-        return (null == value) ? false : value.booleanValue();
-    }
-
 }