svn commit: r698859 - /activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java

[ha...@apache.org]

Author: hadrian
Date: Thu Sep 25 01:24:05 2008
New Revision: 698859

URL: http://svn.apache.org/viewvc?rev=698859&view=rev
Log:
CAMEL-84.  Minor changes.

Modified:
    activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java

Modified: activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java?rev=698859&r1=698858&r2=698859&view=diff
==============================================================================
--- activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java (original)
+++ activemq/camel/trunk/camel-core/src/main/java/org/apache/camel/util/IntrospectionSupport.java Thu Sep 25 01:24:05 2008
@@ -198,17 +198,17 @@
                 } else {
                     // We need to convert it
                     try {
+                        // ignore exceptions as there could be another setter method where we could type convert successfully
                         Object convertedValue = convert(typeConverter, setter.getParameterTypes()[0], value);
                         setter.invoke(target, convertedValue);
                         return true;
                     } catch (NoTypeConversionAvailableException e) {
-                        // ignore we could not find a suitable type converter for this method
+                        typeConvertionFailed = e;
                     } catch (IllegalArgumentException e) {
                         typeConvertionFailed = e;
-                        // ignore as there could be another setter method where we could type convert with success
-                        LOG.trace("Setter " + setter + " with parameter type " + setter.getParameterTypes()[0]
-                                + " could not be used for type conertions of " + value);
                     }
+                    LOG.trace("Setter \"" + setter + "\" with parameter type \"" + 
+                            setter.getParameterTypes()[0] + "\" could not be used for type conertions of " + value);
                 }
             }
             // we did not find a setter method to use, and if we did try to use a type converter then throw