You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ff...@apache.org on 2013/04/07 14:33:00 UTC

svn commit: r1465373 - /cxf/trunk/api/src/main/java/org/apache/cxf/interceptor/FaultOutInterceptor.java

Author: ffang
Date: Sun Apr  7 12:32:59 2013
New Revision: 1465373

URL: http://svn.apache.org/r1465373
Log:
[CXF-4940]CXF not (always) responding with the correct exception when working with an exception hierarchy 

Modified:
    cxf/trunk/api/src/main/java/org/apache/cxf/interceptor/FaultOutInterceptor.java

Modified: cxf/trunk/api/src/main/java/org/apache/cxf/interceptor/FaultOutInterceptor.java
URL: http://svn.apache.org/viewvc/cxf/trunk/api/src/main/java/org/apache/cxf/interceptor/FaultOutInterceptor.java?rev=1465373&r1=1465372&r2=1465373&view=diff
==============================================================================
--- cxf/trunk/api/src/main/java/org/apache/cxf/interceptor/FaultOutInterceptor.java (original)
+++ cxf/trunk/api/src/main/java/org/apache/cxf/interceptor/FaultOutInterceptor.java Sun Apr  7 12:32:59 2013
@@ -151,15 +151,19 @@ public class FaultOutInterceptor extends
      * @param class1
      */
     public FaultInfo getFaultForClass(BindingOperationInfo op, Class<?> class1) {
+        FaultInfo selectedFaultInfo = null;
+        Class<?> selectedFaultInfoClass = null;
         for (BindingFaultInfo bfi : op.getFaults()) {
 
             FaultInfo faultInfo = bfi.getFaultInfo();
             Class<?> c = (Class<?>)faultInfo.getProperty(Class.class.getName());
-            if (c != null && c.isAssignableFrom(class1)) {
-                return faultInfo;
+            if (c != null && c.isAssignableFrom(class1)
+                && (selectedFaultInfo == null || selectedFaultInfoClass.isAssignableFrom(c))) {
+                selectedFaultInfo = faultInfo;
+                selectedFaultInfoClass = c;
+
             }
         }
-
-        return null;
+        return selectedFaultInfo;
     }
 }