svn commit: r1678998 - /sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/Result.java

[sseifert]

Author: sseifert
Date: Tue May 12 16:18:31 2015
New Revision: 1678998

URL: http://svn.apache.org/r1678998
Log:
SLING-4630 Avoid logging an failure that is thrown as exception in 
ModelFactory.createModel (patch supplied by Georg Henzler)

Modified:
    
sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/Result.java

Modified: 
sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/Result.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/Result.java?rev=1678998&r1=1678997&r2=1678998&view=diff
==============================================================================
--- 
sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/Result.java
 (original)
+++ 
sling/trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/Result.java
 Tue May 12 16:18:31 2015
@@ -203,9 +203,13 @@ public class Result<ModelType> {
     }
 
     public void throwException(Logger log) {
-        for (int i = 0; i < failures.size() - 1; i++) {
-            failures.get(i).log(log);
+        // log all failures save the last if multiple failures exist
+        if (failures.size() > 1) {
+            for (int i = 0; i < failures.size() - 2; i++) {
+                failures.get(i).log(log);
+            }
         }
+        // throw the last failure as exception
         if (failures.size() >= 1) {
             failures.get(failures.size() - 1).throwException();
         }