]> source.dussan.org Git - vaadin-framework.git/commitdiff
#6860 introduced AbstractProperty with common functionality of most property implemen...
authorHenri Sara <henri.sara@itmill.com>
Mon, 18 Apr 2011 14:14:35 +0000 (14:14 +0000)
committerHenri Sara <henri.sara@itmill.com>
Mon, 18 Apr 2011 14:14:35 +0000 (14:14 +0000)
svn changeset:18360/svn branch:6.6

src/com/vaadin/data/util/AbstractProperty.java [new file with mode: 0644]
src/com/vaadin/data/util/MethodProperty.java
src/com/vaadin/data/util/ObjectProperty.java
src/com/vaadin/data/util/PropertyFormatter.java
src/com/vaadin/data/util/TextFileProperty.java

diff --git a/src/com/vaadin/data/util/AbstractProperty.java b/src/com/vaadin/data/util/AbstractProperty.java
new file mode 100644 (file)
index 0000000..9db9013
--- /dev/null
@@ -0,0 +1,171 @@
+package com.vaadin.data.util;
+
+import java.util.LinkedList;
+
+import com.vaadin.data.Property;
+
+/**
+ * Abstract base class for {@link Property} implementations.
+ * 
+ * Handles listener management for {@link ValueChangeListener}s and
+ * {@link ReadOnlyStatusChangeListener}s.
+ * 
+ * @since 6.6
+ */
+public abstract class AbstractProperty implements Property,
+        Property.ValueChangeNotifier, Property.ReadOnlyStatusChangeNotifier {
+
+    /**
+     * List of listeners who are interested in the read-only status changes of
+     * the Property
+     */
+    private LinkedList<ReadOnlyStatusChangeListener> readOnlyStatusChangeListeners = null;
+
+    /**
+     * List of listeners who are interested in the value changes of the Property
+     */
+    private LinkedList<ValueChangeListener> valueChangeListeners = null;
+
+    /**
+     * Returns the value of the <code>Property</code> in human readable textual
+     * format. The return value should be assignable to the
+     * <code>setValue</code> method if the Property is not in read-only mode.
+     * 
+     * @return String representation of the value stored in the Property
+     */
+    @Override
+    public String toString() {
+        final Object value = getValue();
+        if (value == null) {
+            return null;
+        }
+        return value.toString();
+    }
+
+    /* Events */
+
+    /**
+     * An <code>Event</code> object specifying the Property whose read-only
+     * status has been changed.
+     */
+    protected class ReadOnlyStatusChangeEvent extends java.util.EventObject
+            implements Property.ReadOnlyStatusChangeEvent {
+
+        /**
+         * Constructs a new read-only status change event for this object.
+         * 
+         * @param source
+         *            source object of the event.
+         */
+        protected ReadOnlyStatusChangeEvent(Property source) {
+            super(source);
+        }
+
+        /**
+         * Gets the Property whose read-only state has changed.
+         * 
+         * @return source Property of the event.
+         */
+        public Property getProperty() {
+            return (Property) getSource();
+        }
+
+    }
+
+    /**
+     * Registers a new read-only status change listener for this Property.
+     * 
+     * @param listener
+     *            the new Listener to be registered.
+     */
+    public void addListener(Property.ReadOnlyStatusChangeListener listener) {
+        if (readOnlyStatusChangeListeners == null) {
+            readOnlyStatusChangeListeners = new LinkedList<ReadOnlyStatusChangeListener>();
+        }
+        readOnlyStatusChangeListeners.add(listener);
+    }
+
+    /**
+     * Removes a previously registered read-only status change listener.
+     * 
+     * @param listener
+     *            the listener to be removed.
+     */
+    public void removeListener(Property.ReadOnlyStatusChangeListener listener) {
+        if (readOnlyStatusChangeListeners != null) {
+            readOnlyStatusChangeListeners.remove(listener);
+        }
+    }
+
+    /**
+     * Sends a read only status change event to all registered listeners.
+     */
+    protected void fireReadOnlyStatusChange() {
+        if (readOnlyStatusChangeListeners != null) {
+            final Object[] l = readOnlyStatusChangeListeners.toArray();
+            final Property.ReadOnlyStatusChangeEvent event = new ReadOnlyStatusChangeEvent(
+                    this);
+            for (int i = 0; i < l.length; i++) {
+                ((Property.ReadOnlyStatusChangeListener) l[i])
+                        .readOnlyStatusChange(event);
+            }
+        }
+    }
+
+    /**
+     * An <code>Event</code> object specifying the Property whose value has been
+     * changed.
+     */
+    private class ValueChangeEvent extends java.util.EventObject implements
+            Property.ValueChangeEvent {
+
+        /**
+         * Constructs a new value change event for this object.
+         * 
+         * @param source
+         *            source object of the event.
+         */
+        protected ValueChangeEvent(Property source) {
+            super(source);
+        }
+
+        /**
+         * Gets the Property whose value has changed.
+         * 
+         * @return source Property of the event.
+         */
+        public Property getProperty() {
+            return (Property) getSource();
+        }
+
+    }
+
+    public void addListener(ValueChangeListener listener) {
+        if (valueChangeListeners == null) {
+            valueChangeListeners = new LinkedList<ValueChangeListener>();
+        }
+        valueChangeListeners.add(listener);
+
+    }
+
+    public void removeListener(ValueChangeListener listener) {
+        if (valueChangeListeners != null) {
+            valueChangeListeners.remove(listener);
+        }
+
+    }
+
+    /**
+     * Sends a value change event to all registered listeners.
+     */
+    protected void fireValueChange() {
+        if (valueChangeListeners != null) {
+            final Object[] l = valueChangeListeners.toArray();
+            final Property.ValueChangeEvent event = new ValueChangeEvent(this);
+            for (int i = 0; i < l.length; i++) {
+                ((Property.ValueChangeListener) l[i]).valueChange(event);
+            }
+        }
+    }
+
+}
index 12b69858dac3138c349665c42665affda3000aab..b04fc77b61428e2fd3038dfe03d2d73beb61bcb8 100644 (file)
@@ -8,7 +8,6 @@ import java.io.IOException;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
-import java.util.LinkedList;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -48,8 +47,7 @@ import com.vaadin.util.SerializerHelper;
  * @since 3.0
  */
 @SuppressWarnings("serial")
-public class MethodProperty<T> implements Property,
-        Property.ValueChangeNotifier, Property.ReadOnlyStatusChangeNotifier {
+public class MethodProperty<T> extends AbstractProperty {
 
     private static final Logger logger = Logger.getLogger(MethodProperty.class
             .getName());
@@ -85,18 +83,6 @@ public class MethodProperty<T> implements Property,
      */
     private transient Class<? extends T> type;
 
-    /**
-     * List of listeners who are interested in the read-only status changes of
-     * the MethodProperty
-     */
-    private LinkedList<ReadOnlyStatusChangeListener> readOnlyStatusChangeListeners = null;
-
-    /**
-     * List of listeners who are interested in the value changes of the
-     * MethodProperty
-     */
-    private LinkedList<ValueChangeListener> valueChangeListeners = null;
-
     /* Special serialization to handle method references */
     private void writeObject(java.io.ObjectOutputStream out) throws IOException {
         out.defaultWriteObject();
@@ -628,22 +614,6 @@ public class MethodProperty<T> implements Property,
         }
     }
 
-    /**
-     * Returns the value of the <code>MethodProperty</code> in human readable
-     * textual format. The return value should be assignable to the
-     * <code>setValue</code> method if the Property is not in read-only mode.
-     * 
-     * @return String representation of the value stored in the Property
-     */
-    @Override
-    public String toString() {
-        final Object value = getValue();
-        if (value == null) {
-            return null;
-        }
-        return value.toString();
-    }
-
     /**
      * <p>
      * Sets the setter method and getter method argument lists.
@@ -852,140 +822,15 @@ public class MethodProperty<T> implements Property,
         }
     }
 
-    /* Events */
-
-    /**
-     * An <code>Event</code> object specifying the Property whose read-only
-     * status has been changed.
-     * 
-     * @author IT Mill Ltd.
-     * @version
-     * @VERSION@
-     * @since 3.0
-     */
-    private class ReadOnlyStatusChangeEvent extends java.util.EventObject
-            implements Property.ReadOnlyStatusChangeEvent {
-
-        /**
-         * Constructs a new read-only status change event for this object.
-         * 
-         * @param source
-         *            source object of the event.
-         */
-        protected ReadOnlyStatusChangeEvent(MethodProperty<T> source) {
-            super(source);
-        }
-
-        /**
-         * Gets the Property whose read-only state has changed.
-         * 
-         * @return source Property of the event.
-         */
-        public Property getProperty() {
-            return (Property) getSource();
-        }
-
-    }
-
-    /**
-     * Registers a new read-only status change listener for this Property.
-     * 
-     * @param listener
-     *            the new Listener to be registered.
-     */
-    public void addListener(Property.ReadOnlyStatusChangeListener listener) {
-        if (readOnlyStatusChangeListeners == null) {
-            readOnlyStatusChangeListeners = new LinkedList<ReadOnlyStatusChangeListener>();
-        }
-        readOnlyStatusChangeListeners.add(listener);
-    }
-
-    /**
-     * Removes a previously registered read-only status change listener.
-     * 
-     * @param listener
-     *            the listener to be removed.
-     */
-    public void removeListener(Property.ReadOnlyStatusChangeListener listener) {
-        if (readOnlyStatusChangeListeners != null) {
-            readOnlyStatusChangeListeners.remove(listener);
-        }
-    }
-
-    /**
-     * Sends a read only status change event to all registered listeners.
-     */
-    private void fireReadOnlyStatusChange() {
-        if (readOnlyStatusChangeListeners != null) {
-            final Object[] l = readOnlyStatusChangeListeners.toArray();
-            final Property.ReadOnlyStatusChangeEvent event = new ReadOnlyStatusChangeEvent(
-                    this);
-            for (int i = 0; i < l.length; i++) {
-                ((Property.ReadOnlyStatusChangeListener) l[i])
-                        .readOnlyStatusChange(event);
-            }
-        }
-    }
-
-    /**
-     * An <code>Event</code> object specifying the Property whose value has been
-     * changed.
-     * 
-     * @author IT Mill Ltd.
-     * @version
-     * @VERSION@
-     * @since 5.3
-     */
-    private class ValueChangeEvent extends java.util.EventObject implements
-            Property.ValueChangeEvent {
-
-        /**
-         * Constructs a new value change event for this object.
-         * 
-         * @param source
-         *            source object of the event.
-         */
-        protected ValueChangeEvent(MethodProperty<T> source) {
-            super(source);
-        }
-
-        /**
-         * Gets the Property whose value has changed.
-         * 
-         * @return source Property of the event.
-         */
-        public Property getProperty() {
-            return (Property) getSource();
-        }
-
-    }
-
-    public void addListener(ValueChangeListener listener) {
-        if (valueChangeListeners == null) {
-            valueChangeListeners = new LinkedList<ValueChangeListener>();
-        }
-        valueChangeListeners.add(listener);
-
-    }
-
-    public void removeListener(ValueChangeListener listener) {
-        if (valueChangeListeners != null) {
-            valueChangeListeners.remove(listener);
-        }
-
-    }
-
     /**
      * Sends a value change event to all registered listeners.
+     * 
+     * Public for backwards compatibility, visibility may be reduced in future
+     * versions.
      */
+    @Override
     public void fireValueChange() {
-        if (valueChangeListeners != null) {
-            final Object[] l = valueChangeListeners.toArray();
-            final Property.ValueChangeEvent event = new ValueChangeEvent(this);
-            for (int i = 0; i < l.length; i++) {
-                ((Property.ValueChangeListener) l[i]).valueChange(event);
-            }
-        }
+        super.fireValueChange();
     }
 
 }
index 6724f1d40f05e4bec76c36bfe072c5f9f28c79e8..ee38bc07974fff20d510891ecbf6dbfa5b7f8dca 100644 (file)
@@ -5,7 +5,6 @@
 package com.vaadin.data.util;
 
 import java.lang.reflect.Constructor;
-import java.util.LinkedList;
 
 import com.vaadin.data.Property;
 
@@ -20,8 +19,7 @@ import com.vaadin.data.Property;
  * @since 3.0
  */
 @SuppressWarnings("serial")
-public class ObjectProperty<T> implements Property,
-        Property.ValueChangeNotifier, Property.ReadOnlyStatusChangeNotifier {
+public class ObjectProperty<T> extends AbstractProperty {
 
     /**
      * A boolean value storing the Property's read-only status information.
@@ -38,16 +36,6 @@ public class ObjectProperty<T> implements Property,
      */
     private final Class<T> type;
 
-    /**
-     * Internal list of registered value change listeners.
-     */
-    private LinkedList<ValueChangeListener> valueChangeListeners = null;
-
-    /**
-     * Internal list of registered read-only status change listeners.
-     */
-    private LinkedList<ReadOnlyStatusChangeListener> readOnlyStatusChangeListeners = null;
-
     /**
      * Creates a new instance of ObjectProperty with the given value. The type
      * of the property is automatically initialized to be the type of the given
@@ -124,24 +112,6 @@ public class ObjectProperty<T> implements Property,
         return value;
     }
 
-    /**
-     * Returns the value of the ObjectProperty in human readable textual format.
-     * The return value should be assignable to the <code>setValue</code> method
-     * if the Property is not in read-only mode.
-     * 
-     * @return <code>String</code> representation of the value stored in the
-     *         ObjectProperty
-     */
-    @Override
-    public String toString() {
-        final Object value = getValue();
-        if (value != null) {
-            return value.toString();
-        } else {
-            return null;
-        }
-    }
-
     /**
      * Tests if the Property is in read-only mode. In read-only mode calls to
      * the method <code>setValue</code> will throw
@@ -214,147 +184,4 @@ public class ObjectProperty<T> implements Property,
         fireValueChange();
     }
 
-    /* Events */
-
-    /**
-     * An <code>Event</code> object specifying the ObjectProperty whose value
-     * has changed.
-     * 
-     * @author IT Mill Ltd.
-     * @version
-     * @VERSION@
-     * @since 3.0
-     */
-    private class ValueChangeEvent extends java.util.EventObject implements
-            Property.ValueChangeEvent {
-
-        /**
-         * Constructs a new value change event for this object.
-         * 
-         * @param source
-         *            the source object of the event.
-         */
-        protected ValueChangeEvent(ObjectProperty<T> source) {
-            super(source);
-        }
-
-        /**
-         * Gets the Property whose read-only state has changed.
-         * 
-         * @return source the Property of the event.
-         */
-        public Property getProperty() {
-            return (Property) getSource();
-        }
-    }
-
-    /**
-     * An <code>Event</code> object specifying the Property whose read-only
-     * status has been changed.
-     * 
-     * @author IT Mill Ltd.
-     * @version
-     * @VERSION@
-     * @since 3.0
-     */
-    private class ReadOnlyStatusChangeEvent extends java.util.EventObject
-            implements Property.ReadOnlyStatusChangeEvent {
-
-        /**
-         * Constructs a new read-only status change event for this object.
-         * 
-         * @param source
-         *            source object of the event
-         */
-        protected ReadOnlyStatusChangeEvent(ObjectProperty<T> source) {
-            super(source);
-        }
-
-        /**
-         * Gets the Property whose read-only state has changed.
-         * 
-         * @return source Property of the event.
-         */
-        public Property getProperty() {
-            return (Property) getSource();
-        }
-    }
-
-    /**
-     * Removes a previously registered value change listener.
-     * 
-     * @param listener
-     *            the listener to be removed.
-     */
-    public void removeListener(Property.ValueChangeListener listener) {
-        if (valueChangeListeners != null) {
-            valueChangeListeners.remove(listener);
-        }
-    }
-
-    /**
-     * Registers a new value change listener for this ObjectProperty.
-     * 
-     * @param listener
-     *            the new Listener to be registered
-     */
-    public void addListener(Property.ValueChangeListener listener) {
-        if (valueChangeListeners == null) {
-            valueChangeListeners = new LinkedList<ValueChangeListener>();
-        }
-        valueChangeListeners.add(listener);
-    }
-
-    /**
-     * Registers a new read-only status change listener for this Property.
-     * 
-     * @param listener
-     *            the new Listener to be registered
-     */
-    public void addListener(Property.ReadOnlyStatusChangeListener listener) {
-        if (readOnlyStatusChangeListeners == null) {
-            readOnlyStatusChangeListeners = new LinkedList<ReadOnlyStatusChangeListener>();
-        }
-        readOnlyStatusChangeListeners.add(listener);
-    }
-
-    /**
-     * Removes a previously registered read-only status change listener.
-     * 
-     * @param listener
-     *            the listener to be removed.
-     */
-    public void removeListener(Property.ReadOnlyStatusChangeListener listener) {
-        if (readOnlyStatusChangeListeners != null) {
-            readOnlyStatusChangeListeners.remove(listener);
-        }
-    }
-
-    /**
-     * Sends a value change event to all registered listeners.
-     */
-    private void fireValueChange() {
-        if (valueChangeListeners != null) {
-            final Object[] l = valueChangeListeners.toArray();
-            final Property.ValueChangeEvent event = new ValueChangeEvent(this);
-            for (int i = 0; i < l.length; i++) {
-                ((Property.ValueChangeListener) l[i]).valueChange(event);
-            }
-        }
-    }
-
-    /**
-     * Sends a read only status change event to all registered listeners.
-     */
-    private void fireReadOnlyStatusChange() {
-        if (readOnlyStatusChangeListeners != null) {
-            final Object[] l = readOnlyStatusChangeListeners.toArray();
-            final Property.ReadOnlyStatusChangeEvent event = new ReadOnlyStatusChangeEvent(
-                    this);
-            for (int i = 0; i < l.length; i++) {
-                ((Property.ReadOnlyStatusChangeListener) l[i])
-                        .readOnlyStatusChange(event);
-            }
-        }
-    }
 }
index 23e4fb324f9a354bf2cd9161ac41a7eabaffe243..c43a4771dccd51dcca839aeae72eb35d53d93027 100644 (file)
@@ -3,8 +3,6 @@
  */
 package com.vaadin.data.util;
 
-import java.util.LinkedList;
-
 import com.vaadin.data.Property;
 
 /**
@@ -35,20 +33,8 @@ import com.vaadin.data.Property;
  * @since 5.3.0
  */
 @SuppressWarnings("serial")
-public abstract class PropertyFormatter implements Property,
-        Property.ValueChangeNotifier, Property.ValueChangeListener,
-        Property.ReadOnlyStatusChangeListener,
-        Property.ReadOnlyStatusChangeNotifier {
-
-    /**
-     * Internal list of registered value change listeners.
-     */
-    private LinkedList<Property.ValueChangeListener> valueChangeListeners = null;
-
-    /**
-     * Internal list of registered read-only status change listeners.
-     */
-    private LinkedList<Property.ReadOnlyStatusChangeListener> readOnlyStatusChangeListeners = null;
+public abstract class PropertyFormatter extends AbstractProperty implements
+        Property.ValueChangeListener, Property.ReadOnlyStatusChangeListener {
 
     /** Datasource that stores the actual value. */
     Property dataSource;
@@ -236,144 +222,6 @@ public abstract class PropertyFormatter implements Property,
         }
     }
 
-    /**
-     * An <code>Event</code> object specifying the ObjectProperty whose value
-     * has changed.
-     * 
-     * @author IT Mill Ltd.
-     * @since 5.3.0
-     */
-    private class ValueChangeEvent extends java.util.EventObject implements
-            Property.ValueChangeEvent {
-
-        /**
-         * Constructs a new value change event for this object.
-         * 
-         * @param source
-         *            the source object of the event.
-         */
-        protected ValueChangeEvent(PropertyFormatter source) {
-            super(source);
-        }
-
-        /**
-         * Gets the Property whose read-only state has changed.
-         * 
-         * @return source the Property of the event.
-         */
-        public Property getProperty() {
-            return (Property) getSource();
-        }
-    }
-
-    /**
-     * An <code>Event</code> object specifying the Property whose read-only
-     * status has been changed.
-     * 
-     * @author IT Mill Ltd.
-     * @since 5.3.0
-     */
-    private class ReadOnlyStatusChangeEvent extends java.util.EventObject
-            implements Property.ReadOnlyStatusChangeEvent {
-
-        /**
-         * Constructs a new read-only status change event for this object.
-         * 
-         * @param source
-         *            source object of the event
-         */
-        protected ReadOnlyStatusChangeEvent(PropertyFormatter source) {
-            super(source);
-        }
-
-        /**
-         * Gets the Property whose read-only state has changed.
-         * 
-         * @return source Property of the event.
-         */
-        public Property getProperty() {
-            return (Property) getSource();
-        }
-    }
-
-    /**
-     * Removes a previously registered value change listener.
-     * 
-     * @param listener
-     *            the listener to be removed.
-     */
-    public void removeListener(Property.ValueChangeListener listener) {
-        if (valueChangeListeners != null) {
-            valueChangeListeners.remove(listener);
-        }
-    }
-
-    /**
-     * Registers a new value change listener for this ObjectProperty.
-     * 
-     * @param listener
-     *            the new Listener to be registered
-     */
-    public void addListener(Property.ValueChangeListener listener) {
-        if (valueChangeListeners == null) {
-            valueChangeListeners = new LinkedList<Property.ValueChangeListener>();
-        }
-        valueChangeListeners.add(listener);
-    }
-
-    /**
-     * Registers a new read-only status change listener for this Property.
-     * 
-     * @param listener
-     *            the new Listener to be registered
-     */
-    public void addListener(Property.ReadOnlyStatusChangeListener listener) {
-        if (readOnlyStatusChangeListeners == null) {
-            readOnlyStatusChangeListeners = new LinkedList<Property.ReadOnlyStatusChangeListener>();
-        }
-        readOnlyStatusChangeListeners.add(listener);
-    }
-
-    /**
-     * Removes a previously registered read-only status change listener.
-     * 
-     * @param listener
-     *            the listener to be removed.
-     */
-    public void removeListener(Property.ReadOnlyStatusChangeListener listener) {
-        if (readOnlyStatusChangeListeners != null) {
-            readOnlyStatusChangeListeners.remove(listener);
-        }
-    }
-
-    /**
-     * Sends a value change event to all registered listeners.
-     */
-    private void fireValueChange() {
-        if (valueChangeListeners != null) {
-            final Object[] l = valueChangeListeners.toArray();
-            final Property.ValueChangeEvent event = new ValueChangeEvent(this);
-            for (int i = 0; i < l.length; i++) {
-                ((Property.ValueChangeListener) l[i]).valueChange(event);
-            }
-        }
-    }
-
-    /**
-     * Sends a read only status change event to all registered listeners.
-     */
-    private void fireReadOnlyStatusChange() {
-        if (readOnlyStatusChangeListeners != null) {
-            final Object[] l = readOnlyStatusChangeListeners.toArray();
-            final Property.ReadOnlyStatusChangeEvent event = new ReadOnlyStatusChangeEvent(
-                    this);
-            for (int i = 0; i < l.length; i++) {
-                ((Property.ReadOnlyStatusChangeListener) l[i])
-                        .readOnlyStatusChange(event);
-            }
-        }
-    }
-
     /**
      * Listens for changes in the datasource.
      * 
index 8d991123382ee86a93370029b634bf9d753e3ea4..005372a04b2f24aa79267f89a3ca23388c734e82 100644 (file)
@@ -14,23 +14,23 @@ import java.io.IOException;
 import java.io.InputStreamReader;\r
 import java.io.OutputStreamWriter;\r
 import java.nio.charset.Charset;\r
-import java.util.LinkedList;\r
-\r
-import com.vaadin.data.Property;\r
 \r
 /**\r
  * Property implementation for wrapping a text file.\r
  * \r
- * Supports reading and writing of a File from/to String. ValueChangeNotifiers\r
- * are supported, but only fire when setValue(Object) is explicitly called.\r
+ * Supports reading and writing of a File from/to String.\r
+ * \r
+ * {@link ValueChangeListener}s are supported, but only fire when\r
+ * setValue(Object) is explicitly called. {@link ReadOnlyStatusChangeListener}s\r
+ * are supported but only fire when setReadOnly(boolean) is explicitly called.\r
+ * \r
  */\r
 @SuppressWarnings("serial")\r
-public class TextFileProperty implements Property, Property.ValueChangeNotifier {\r
+public class TextFileProperty extends AbstractProperty {\r
 \r
     private File file;\r
     private boolean readonly;\r
     private Charset charset = null;\r
-    private LinkedList<Property.ValueChangeListener> valueChangeListeners = null;\r
 \r
     /**\r
      * Wrap given file with property interface.\r
@@ -115,7 +115,11 @@ public class TextFileProperty implements Property, Property.ValueChangeNotifier
      * @see com.vaadin.data.Property#setReadOnly(boolean)\r
      */\r
     public void setReadOnly(boolean newStatus) {\r
+        boolean oldStatus = readonly;\r
         readonly = newStatus;\r
+        if (isReadOnly() != oldStatus) {\r
+            fireReadOnlyStatusChange();\r
+        }\r
     }\r
 \r
     /*\r
@@ -147,81 +151,4 @@ public class TextFileProperty implements Property, Property.ValueChangeNotifier
         }\r
     }\r
 \r
-    /*\r
-     * (non-Javadoc)\r
-     * \r
-     * @see java.lang.Object#toString()\r
-     */\r
-    @Override\r
-    public String toString() {\r
-        return (String) getValue();\r
-    }\r
-\r
-    /* Events */\r
-\r
-    /**\r
-     * An <code>Event</code> object specifying the TextFileProperty whose value\r
-     * has changed.\r
-     */\r
-    private class ValueChangeEvent extends java.util.EventObject implements\r
-            Property.ValueChangeEvent {\r
-\r
-        /**\r
-         * Constructs a new value change event for this object.\r
-         * \r
-         * @param source\r
-         *            the source object of the event.\r
-         */\r
-        protected ValueChangeEvent(TextFileProperty source) {\r
-            super(source);\r
-        }\r
-\r
-        /**\r
-         * Gets the Property whose read-only state has changed.\r
-         * \r
-         * @return source the Property of the event.\r
-         */\r
-        public Property getProperty() {\r
-            return (Property) getSource();\r
-        }\r
-    }\r
-\r
-    /**\r
-     * Removes a previously registered value change listener.\r
-     * \r
-     * @param listener\r
-     *            the listener to be removed.\r
-     */\r
-    public void removeListener(Property.ValueChangeListener listener) {\r
-        if (valueChangeListeners != null) {\r
-            valueChangeListeners.remove(listener);\r
-        }\r
-    }\r
-\r
-    /**\r
-     * Registers a new value change listener for this TextFileProperty.\r
-     * \r
-     * @param listener\r
-     *            the new Listener to be registered\r
-     */\r
-    public void addListener(Property.ValueChangeListener listener) {\r
-        if (valueChangeListeners == null) {\r
-            valueChangeListeners = new LinkedList<Property.ValueChangeListener>();\r
-        }\r
-        valueChangeListeners.add(listener);\r
-    }\r
-\r
-    /**\r
-     * Sends a value change event to all registered listeners.\r
-     */\r
-    private void fireValueChange() {\r
-        if (valueChangeListeners != null) {\r
-            final Object[] l = valueChangeListeners.toArray();\r
-            final ValueChangeEvent event = new ValueChangeEvent(this);\r
-            for (int i = 0; i < l.length; i++) {\r
-                ((ValueChangeListener) l[i]).valueChange(event);\r
-            }\r
-        }\r
-    }\r
-\r
 }\r