You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by sl...@apache.org on 2010/07/05 12:16:30 UTC

svn commit: r960521 - /tuscany/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/impl/CompositeActivatorImpl.java

Author: slaws
Date: Mon Jul  5 10:16:29 2010
New Revision: 960521

URL: http://svn.apache.org/viewvc?rev=960521&view=rev
Log:
Little bit of tidying. No function change.

Modified:
    tuscany/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/impl/CompositeActivatorImpl.java

Modified: tuscany/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/impl/CompositeActivatorImpl.java
URL: http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/impl/CompositeActivatorImpl.java?rev=960521&r1=960520&r2=960521&view=diff
==============================================================================
--- tuscany/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/impl/CompositeActivatorImpl.java (original)
+++ tuscany/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/impl/CompositeActivatorImpl.java Mon Jul  5 10:16:29 2010
@@ -279,7 +279,6 @@ public class CompositeActivatorImpl impl
             activate(compositeContext, (RuntimeEndpointReference) epr);
         }
 
-        // TODO reference wires are added at component start for some reason
     }
 
     public void deactivate(RuntimeComponent component, RuntimeComponentReference reference) {
@@ -290,6 +289,27 @@ public class CompositeActivatorImpl impl
             deactivate((RuntimeEndpointReference)endpointReference);
         }
     }
+    
+    public void activate(CompositeContext compositeContext, RuntimeEndpointReference epr) {
+        // create the wire
+        // null endpoint passed in here as the endpoint reference may
+        // not be resolved yet
+        epr.bind(compositeContext);
+
+        ComponentReference reference = epr.getReference(); 
+        InterfaceContract sourceContract = epr.getComponentTypeReferenceInterfaceContract();
+
+        // TODO - EPR - interface contract seems to be null in the implementation.web
+        //              case. Not introspecting the CT properly?
+        if (sourceContract == null){
+            // TODO - Can't do this with move of matching to wire
+            // take the contract from the service to which the reference is connected
+            sourceContract = ((RuntimeEndpoint) epr.getTargetEndpoint()).getComponentTypeServiceInterfaceContract();
+            reference.setInterfaceContract(sourceContract);
+        }
+
+        // endpointReference.setInterfaceContract(sourceContract.makeUnidirectional(false));
+    }    
 
     public void deactivate(RuntimeEndpointReference endpointReference) {
         endpointReference.unbind();
@@ -573,27 +593,4 @@ public class CompositeActivatorImpl impl
             }
         }
     }
-
-    public void activate(CompositeContext compositeContext, RuntimeEndpointReference epr) {
-        // create the wire
-        // null endpoint passed in here as the endpoint reference may
-        // not be resolved yet
-        epr.bind(compositeContext);
-
-        ComponentReference reference = epr.getReference(); 
-        InterfaceContract sourceContract = epr.getComponentTypeReferenceInterfaceContract();
-
-        // TODO - EPR - interface contract seems to be null in the implementation.web
-        //              case. Not introspecting the CT properly?
-        if (sourceContract == null){
-            // TODO - Can't do this with move of matching to wire
-            // take the contract from the service to which the reference is connected
-            sourceContract = ((RuntimeEndpoint) epr.getTargetEndpoint()).getComponentTypeServiceInterfaceContract();
-            reference.setInterfaceContract(sourceContract);
-        }
-
-        // endpointReference.setInterfaceContract(sourceContract.makeUnidirectional(false));
-    }
-
-
 }