TUSCANY-3894. Refactor WSDL InterfaceContract generation so it's only done if the BindingSCATransformer calls for it.

git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@1151710 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 7 insertions, 2 deletions

diff --git a/sca-java-2.x/trunk/modules/binding-sca-runtime/src/main/java/org/apache/tuscany/sca/binding/local/XMLWFLocalSCAReferenceBindingProvider.java b/sca-java-2.x/trunk/modules/binding-sca-runtime/src/main/java/org/apache/tuscany/sca/binding/local/XMLWFLocalSCAReferenceBindingProvider.java
index 2c5b4f73e1..170908c82d 100644
--- a/sca-java-2.x/trunk/modules/binding-sca-runtime/src/main/java/org/apache/tuscany/sca/binding/local/XMLWFLocalSCAReferenceBindingProvider.java
+++ b/sca-java-2.x/trunk/modules/binding-sca-runtime/src/main/java/org/apache/tuscany/sca/binding/local/XMLWFLocalSCAReferenceBindingProvider.java
@@ -23,6 +23,7 @@ import org.apache.tuscany.sca.binding.sca.transform.BindingSCATransformer;
 import org.apache.tuscany.sca.binding.sca.transform.DefaultBindingSCATransformer;

 import org.apache.tuscany.sca.binding.sca.transform.XMLWFBindingSCATransformer;

 import org.apache.tuscany.sca.core.ExtensionPointRegistry;

+import org.apache.tuscany.sca.interfacedef.InterfaceContract;

 import org.apache.tuscany.sca.interfacedef.Operation;

 import org.apache.tuscany.sca.invocation.InvocationChain;

 import org.apache.tuscany.sca.provider.SCABindingMapper;

@@ -36,8 +37,12 @@ public class XMLWFLocalSCAReferenceBindingProvider extends DefaultLocalSCARefere
         super(extensionPoints, endpointReference, mapper);

     }

     

-    protected BindingSCATransformer getBindingTransformer(Operation operation, InvocationChain chain) {

-        Operation wsdlBindingOperation = interfaceContractMapper.map(wsdlBindingInterfaceContract.getInterface(), operation);                        

+    protected BindingSCATransformer getBindingTransformer(Operation operation, InvocationChain chain) { 

+        InterfaceContract bindingInterfaceContract = getWSDLBindingInterfaceContract();   

+        if (!bindingInterfaceContract.getInterface().isRemotable()) {

+            throw new IllegalStateException("This method should only have been called for a remotable interface.");

+        }

+        Operation wsdlBindingOperation = interfaceContractMapper.map(bindingInterfaceContract.getInterface(), operation);                        

         return new XMLWFBindingSCATransformer(mediator, operation, wsdlBindingOperation, chain);                

     }

 }