svn commit: r1811437 - /ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1811437 - /ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/conversion/Converters.java

mbrohl
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);
             }
         }