You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2010/04/15 09:18:07 UTC
svn commit: r934308 -
/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java
Author: davsclaus
Date: Thu Apr 15 07:18:07 2010
New Revision: 934308
URL: http://svn.apache.org/viewvc?rev=934308&view=rev
Log:
CAMEL-2642: Added javadoc and TODO
Modified:
camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java
Modified: camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java
URL: http://svn.apache.org/viewvc/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java?rev=934308&r1=934307&r2=934308&view=diff
==============================================================================
--- camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java (original)
+++ camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java Thu Apr 15 07:18:07 2010
@@ -97,6 +97,17 @@ public final class IntrospectionSupport
return params.length == 1 && type.equals(Void.TYPE);
}
+ /**
+ * Will inspect the target for properties.
+ * <p/>
+ * <b>Notice:</b> only properties which is convertable to String will be selected,
+ * other properties will be skipped.
+ *
+ * @param target the target bean
+ * @param properties the map to fill in found properties
+ * @param optionPrefix an optional prefix to append the property key
+ * @return <tt>true</tt> if any properties was found, <tt>false</tt> otherwise.
+ */
@SuppressWarnings("unchecked")
public static boolean getProperties(Object target, Map properties, String optionPrefix) {
ObjectHelper.notNull(target, "target");
@@ -119,6 +130,7 @@ public final class IntrospectionSupport
continue;
}
+ // TODO: Why must the found property be convertable to String?
String strValue = convertToString(value, type);
if (strValue == null) {
continue;
@@ -372,6 +384,7 @@ public final class IntrospectionSupport
}
private static boolean isSettableType(Class<?> clazz) {
+ // TODO: Why limit to what the JDK property editor can set?
if (PropertyEditorManager.findEditor(clazz) != null) {
return true;
}