From: Artur Signell <artur@vaadin.com>
Date: Tue, 22 Nov 2011 14:47:20 +0000 (+0200)
Subject: FIXME comment for the future
X-Git-Tag: 7.0.0.alpha1~244^2~6
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e1f4a1215aa6cc65822e7968cfe385fe0b1b695c;p=vaadin-framework.git

FIXME comment for the future
---

diff --git a/src/com/vaadin/ui/CheckBox.java b/src/com/vaadin/ui/CheckBox.java
index bb3a0cc66a..11d9619c8c 100644
--- a/src/com/vaadin/ui/CheckBox.java
+++ b/src/com/vaadin/ui/CheckBox.java
@@ -122,6 +122,9 @@ public class CheckBox extends AbstractField {
      */
     @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 = (Boolean) getValue();
         return (null == value) ? false : value.booleanValue();
     }