Initial support for null representations in Binder

Change-Id: If40bfa28764d1399b5ed4d5928988560e9989dce

1 files changed, 8 insertions, 15 deletions

diff --git a/server/src/main/java/com/vaadin/ui/AbstractTextField.java b/server/src/main/java/com/vaadin/ui/AbstractTextField.java
index b5119ec92c..9c2720efd8 100644
--- a/server/src/main/java/com/vaadin/ui/AbstractTextField.java
+++ b/server/src/main/java/com/vaadin/ui/AbstractTextField.java
@@ -17,6 +17,7 @@
 package com.vaadin.ui;
 
 import java.util.Collection;
+import java.util.Objects;
 
 import org.jsoup.nodes.Attributes;
 import org.jsoup.nodes.Element;
@@ -80,11 +81,8 @@ public abstract class AbstractTextField extends AbstractField<String>
 
     @Override
     public void setValue(String value) {
-        if (value == null) {
-            setValue("", false);
-        } else {
-            setValue(value, false);
-        }
+        Objects.requireNonNull(value, "Null value not supported");
+        setValue(value, false);
     }
 
     /**
@@ -270,16 +268,6 @@ public abstract class AbstractTextField extends AbstractField<String>
         setValue("");
     }
 
-    /**
-     * Checks if the field is empty.
-     *
-     * @return <code>true</code> if the field value is an empty string,
-     *         <code>false</code> otherwise
-     */
-    public boolean isEmpty() {
-        return "".equals(getValue());
-    }
-
     @Override
     public void writeDesign(Element design, DesignContext designContext) {
         super.writeDesign(design, designContext);
@@ -299,4 +287,9 @@ public abstract class AbstractTextField extends AbstractField<String>
         customAttributes.add("cursor-position");
         return customAttributes;
     }
+
+    @Override
+    public String getEmptyValue() {
+        return "";
+    }
 }