You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2010/02/14 23:38:26 UTC

svn commit: r910104 - /ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/Converters.java

Author: doogie
Date: Sun Feb 14 22:38:26 2010
New Revision: 910104

URL: http://svn.apache.org/viewvc?rev=910104&view=rev
Log:
In loadConverters, if the contained class implements ConverterLoader,
call it's loadConverters method.

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/Converters.java

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/Converters.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/Converters.java?rev=910104&r1=910103&r2=910104&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/Converters.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/conversion/Converters.java Sun Feb 14 22:38:26 2010
@@ -124,11 +124,17 @@
             try {
                 // non-abstract, which means no interfaces or abstract classes
                 if ((clz.getModifiers() & Modifier.ABSTRACT) == 0) {
+                    Object value;
                     try {
-                        clz.getConstructor().newInstance();
+                        value = clz.getConstructor().newInstance();
                     } catch (NoSuchMethodException e) {
                         // ignore this, as this class might be some other helper class,
                         // with a non-pubilc constructor
+                        continue;
+                    }
+                    if (value instanceof ConverterLoader) {
+                        ConverterLoader loader = (ConverterLoader) value;
+                        loader.loadConverters();
                     }
                 }
             } catch (Exception e) {