Author: ffang
Date: Tue Jun 2 09:37:02 2009
New Revision: 780983
URL: http://svn.apache.org/viewvc?rev=780983&view=rev
Log:
[SMXCOMP-555]cxf bc provider should handle soap fault correctly with empty
detail
Modified:
servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java
Modified:
servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java
URL:
http://svn.apache.org/viewvc/servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java?rev=780983&r1=780982&r2=780983&view=diff
==============================================================================
---
servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java
(original)
+++
servicemix/components/bindings/servicemix-cxf-bc/trunk/src/main/java/org/apache/servicemix/cxfbc/interceptors/JbiInWsdl1Interceptor.java
Tue Jun 2 09:37:02 2009
@@ -232,7 +232,8 @@
nodeList = soapFault.getElementsByTagName("faultstring");
message.put("faultstring",
nodeList.item(0).getFirstChild().getTextContent());
nodeList = soapFault.getElementsByTagName("detail");
- if (nodeList != null && nodeList.getLength() > 0) {
+ if (nodeList != null && nodeList.getLength() > 0
+ && nodeList.item(0).getFirstChild() != null) {
jbiFaultDetail =
doc.importNode(nodeList.item(0).getFirstChild(), true);
} else {
message.put("hasdetail", false);
@@ -249,7 +250,8 @@
nodeList = soapFault.getElementsByTagName("soap:Reason");
message.put("faultstring",
nodeList.item(0).getFirstChild().getTextContent());
nodeList = soapFault.getElementsByTagName("soap:Detail");
- if (nodeList != null && nodeList.getLength() > 0) {
+ if (nodeList != null && nodeList.getLength() > 0
+ && nodeList.item(0).getFirstChild() != null) {
jbiFaultDetail =
doc.importNode(nodeList.item(0).getFirstChild(), true);
} else {
message.put("hasdetail", false);