Don't throw UnsupportedOperationException in Label.toString (#9175)

diff --git a/src/com/vaadin/ui/Label.java b/src/com/vaadin/ui/Label.java
index e7ca26faf846c88bacf9d37799f531b347c88bb3..00137f67e1b82dbe41f681fcd963fbd3791280f9 100644 (file)
--- a/src/com/vaadin/ui/Label.java
+++ b/src/com/vaadin/ui/Label.java
@@ -5,6 +5,7 @@
 package com.vaadin.ui;
 
 import java.lang.reflect.Method;
+import java.util.logging.Logger;
 
 import com.vaadin.data.Property;
 import com.vaadin.data.util.converter.Converter;
@@ -41,6 +42,9 @@ public class Label extends AbstractComponent implements Property<String>,
         Property.Viewer, Property.ValueChangeListener,
         Property.ValueChangeNotifier, Comparable<Label> {
 
+    private static final Logger logger = Logger
+            .getLogger(Label.class.getName());
+
     /**
      * @deprecated From 7.0, use {@link ContentMode#TEXT} instead
      */
@@ -184,14 +188,20 @@ public class Label extends AbstractComponent implements Property<String>,
     }
 
     /**
+     * Returns the value displayed by this label.
+     * 
      * @see java.lang.Object#toString()
-     * @deprecated Use {@link #getValue()} instead
+     * @deprecated As of 7.0.0, use {@link #getValue()} to get the value of the
+     *             label or {@link #getPropertyDataSource()} .getValue() to get
+     *             the value of the data source.
      */
     @Deprecated
     @Override
     public String toString() {
-        throw new UnsupportedOperationException(
-                "Use getValue() instead of Label.toString()");
+        logger.warning("You are using Label.toString() to get the value for a "
+                + getClass().getSimpleName()
+                + ". This is not recommended and will not be supported in future versions.");
+        return getValue();
     }
 
     /**