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 2009/05/15 09:41:20 UTC
svn commit: r775051 - in /cxf/trunk/rt:
bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
core/src/main/java/org/apache/cxf/binding/AbstractBindingFactory.java
Author: ffang
Date: Fri May 15 07:41:19 2009
New Revision: 775051
URL: http://svn.apache.org/viewvc?rev=775051&view=rev
Log:
[CXF-2217]revert last commit
Modified:
cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
cxf/trunk/rt/core/src/main/java/org/apache/cxf/binding/AbstractBindingFactory.java
Modified: cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java?rev=775051&r1=775050&r2=775051&view=diff
==============================================================================
--- cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java (original)
+++ cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java Fri May 15 07:41:19 2009
@@ -354,48 +354,45 @@
sb.getOutFaultInterceptors().add(new StaxOutInterceptor());
sb.getOutFaultInterceptors().add(new SoapOutInterceptor(getBus()));
-
- if (!Boolean.TRUE.equals(binding.getProperty(SMX_DATABINDING_DISABLED))) {
- sb.getInInterceptors().add(new AttachmentInInterceptor());
- sb.getInInterceptors().add(new StaxInInterceptor());
- sb.getInInterceptors().add(new SoapActionInInterceptor());
+ sb.getInInterceptors().add(new AttachmentInInterceptor());
+ sb.getInInterceptors().add(new StaxInInterceptor());
+ sb.getInInterceptors().add(new SoapActionInInterceptor());
- sb.getOutInterceptors().add(new AttachmentOutInterceptor());
- sb.getOutInterceptors().add(new StaxOutInterceptor());
- sb.getOutInterceptors().add(new SoapHeaderOutFilterInterceptor());
-
- if (SoapBindingConstants.BINDING_STYLE_RPC.equalsIgnoreCase(bindingStyle)) {
- sb.getInInterceptors().add(new RPCInInterceptor());
- sb.getOutInterceptors().add(new RPCOutInterceptor());
- } else if (SoapBindingConstants.BINDING_STYLE_DOC.equalsIgnoreCase(bindingStyle)
+ sb.getOutInterceptors().add(new AttachmentOutInterceptor());
+ sb.getOutInterceptors().add(new StaxOutInterceptor());
+ sb.getOutInterceptors().add(new SoapHeaderOutFilterInterceptor());
+
+ if (SoapBindingConstants.BINDING_STYLE_RPC.equalsIgnoreCase(bindingStyle)) {
+ sb.getInInterceptors().add(new RPCInInterceptor());
+ sb.getOutInterceptors().add(new RPCOutInterceptor());
+ } else if (SoapBindingConstants.BINDING_STYLE_DOC.equalsIgnoreCase(bindingStyle)
&& SoapBindingConstants.PARAMETER_STYLE_BARE.equalsIgnoreCase(parameterStyle)) {
- //sb.getInInterceptors().add(new BareInInterceptor());
- sb.getInInterceptors().add(new DocLiteralInInterceptor());
- if (hasWrapped) {
- sb.getOutInterceptors().add(new WrappedOutInterceptor());
- }
- sb.getOutInterceptors().add(new BareOutInterceptor());
- } else {
- //sb.getInInterceptors().add(new WrappedInInterceptor());
- sb.getInInterceptors().add(new DocLiteralInInterceptor());
- sb.getOutInterceptors().add(new WrappedOutInterceptor());
- sb.getOutInterceptors().add(new BareOutInterceptor());
+ //sb.getInInterceptors().add(new BareInInterceptor());
+ sb.getInInterceptors().add(new DocLiteralInInterceptor());
+ if (hasWrapped) {
+ sb.getOutInterceptors().add(new WrappedOutInterceptor());
}
- sb.getInInterceptors().add(new SoapHeaderInterceptor());
-
- sb.getInInterceptors().add(new ReadHeadersInterceptor(getBus()));
- sb.getInInterceptors().add(new CheckFaultInterceptor());
- sb.getInInterceptors().add(new MustUnderstandInterceptor());
- sb.getOutInterceptors().add(new SoapPreProtocolOutInterceptor());
- sb.getOutInterceptors().add(new SoapOutInterceptor(getBus()));
- sb.getOutFaultInterceptors().add(new SoapOutInterceptor(getBus()));
-
- // REVISIT: The phase interceptor chain seems to freak out if this added
- //first. Not sure what the deal is at the moment, I suspect the
- //ordering algorithm needs to be improved
- sb.getInInterceptors().add(new URIMappingInterceptor());
- }
+ sb.getOutInterceptors().add(new BareOutInterceptor());
+ } else {
+ //sb.getInInterceptors().add(new WrappedInInterceptor());
+ sb.getInInterceptors().add(new DocLiteralInInterceptor());
+ sb.getOutInterceptors().add(new WrappedOutInterceptor());
+ sb.getOutInterceptors().add(new BareOutInterceptor());
+ }
+ sb.getInInterceptors().add(new SoapHeaderInterceptor());
+
+ sb.getInInterceptors().add(new ReadHeadersInterceptor(getBus()));
+ sb.getInInterceptors().add(new CheckFaultInterceptor());
+ sb.getInInterceptors().add(new MustUnderstandInterceptor());
+ sb.getOutInterceptors().add(new SoapPreProtocolOutInterceptor());
+ sb.getOutInterceptors().add(new SoapOutInterceptor(getBus()));
+ sb.getOutFaultInterceptors().add(new SoapOutInterceptor(getBus()));
+
+ // REVISIT: The phase interceptor chain seems to freak out if this added
+ // first. Not sure what the deal is at the moment, I suspect the
+ // ordering algorithm needs to be improved
+ sb.getInInterceptors().add(new URIMappingInterceptor());
if (version.getVersion() == 1.1) {
sb.getInFaultInterceptors().add(new Soap11FaultInInterceptor());
Modified: cxf/trunk/rt/core/src/main/java/org/apache/cxf/binding/AbstractBindingFactory.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/core/src/main/java/org/apache/cxf/binding/AbstractBindingFactory.java?rev=775051&r1=775050&r2=775051&view=diff
==============================================================================
--- cxf/trunk/rt/core/src/main/java/org/apache/cxf/binding/AbstractBindingFactory.java (original)
+++ cxf/trunk/rt/core/src/main/java/org/apache/cxf/binding/AbstractBindingFactory.java Fri May 15 07:41:19 2009
@@ -47,8 +47,6 @@
public abstract class AbstractBindingFactory implements BindingFactory, WSDLBindingFactory {
- public static final String SMX_DATABINDING_DISABLED = "smx.databinding.disabled";
-
protected Collection<String> activationNamespaces;
protected Bus bus;