You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by mb...@apache.org on 2017/10/07 15:46:38 UTC
svn commit: r1811437 -
/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java
Author: mbrohl
Date: Sat Oct 7 15:46:38 2017
New Revision: 1811437
URL: http://svn.apache.org/viewvc?rev=1811437&view=rev
Log:
Fixed: the introduction of multicatch in Converters.java which came with
r1811434 breaks MiscTests#testLoadContainedConvertersIgnoresException
which throws a RuntimeException.
Not sure if we should change the test so the fix is to reintroduce the
catch of Exception.
Modified:
ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java
Modified: ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java?rev=1811437&r1=1811436&r2=1811437&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java Sat Oct 7 15:46:38 2017
@@ -152,7 +152,7 @@ OUTER:
loader.loadConverters();
}
}
- } catch (InstantiationException | IllegalAccessException | InvocationTargetException e) {
+ } catch (Exception e) {
Debug.logError(e, module);
}
}