Changeset 5233 for trunk/src/com/itmill/toolkit/data/util/BeanItem.java

Timestamp:

08/21/08 10:18:44 (5 months ago)

Author:

marc.englund@…

Message:

Unified constructors + getter required, setter not; fixes #1998 and really-fixes #1076

Files:

• trunk/src/com/itmill/toolkit/data/util/BeanItem.java (modified) (4 diffs)

trunk/src/com/itmill/toolkit/data/util/BeanItem.java

@@ -9 +9 @@
 import java.beans.PropertyDescriptor;
 import java.lang.reflect.Method;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.Iterator;
@@ -63 +64 @@
                 final String name = pd[i].getName();
 
-                if ((getMethod != null) && (setMethod != null)) {
+                if ((getMethod != null)) {
                     final Property p = new MethodProperty(type, bean,
                             getMethod, setMethod);
@@ -110 +111 @@
                         final Method setMethod = pd[i].getWriteMethod();
                         final Class type = pd[i].getPropertyType();
-
-                        final Property p = new MethodProperty(type, bean,
-                                getMethod, setMethod);
-                        addItemProperty(name, p);
+                        if ((getMethod != null)) {
+                            final Property p = new MethodProperty(type, bean,
+                                    getMethod, setMethod);
+                            addItemProperty(name, p);
+                        }
                     }
                 }
@@ -142 +144 @@
      */
     public BeanItem(Object bean, String[] propertyIds) {
-        this.bean = bean;
-
-        // Try to introspect, if it fails, we just have an empty Item
-        try {
-            // Create bean information
-            final BeanInfo info = Introspector.getBeanInfo(bean.getClass());
-            final PropertyDescriptor[] pd = info.getPropertyDescriptors();
-
-            // Add all the bean properties as MethodProperties to this Item
-            for (int j = 0; j < propertyIds.length; j++) {
-                final Object id = propertyIds[j];
-                for (int i = 0; i < pd.length; i++) {
-                    final String name = pd[i].getName();
-                    if (name.equals(id)) {
-                        final Method getMethod = pd[i].getReadMethod();
-                        final Method setMethod = pd[i].getWriteMethod();
-                        final Class type = pd[i].getPropertyType();
-
-                        final Property p = new MethodProperty(type, bean,
-                                getMethod, setMethod);
-                        addItemProperty(name, p);
-                    }
-                }
-            }
-
-        } catch (final java.beans.IntrospectionException ignored) {
-        }
+        this(bean, Arrays.asList(propertyIds));
     }