Correct the algorithm for selecting callback bindings to match what the old builders were doing. However this is not correct. In particular we should probably be always selecting the forward binding if not callback binding is specified.

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

1 files changed, 3 insertions, 1 deletions

diff --git a/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/CallbackServiceReferenceImpl.java b/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/CallbackServiceReferenceImpl.java
index 47dc99aac9..31db699063 100644
--- a/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/CallbackServiceReferenceImpl.java
+++ b/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/CallbackServiceReferenceImpl.java
@@ -142,7 +142,9 @@ public class CallbackServiceReferenceImpl<B> extends ServiceReferenceImpl<B> {
         callbackEndpointReference.setReference(ref);
         callbackEndpointReference.setTargetEndpoint(callbackEndpoint);
         callbackEndpointReference.setUnresolved(true);
-       
+               
+        // TODO - should really use incoming callback info but awaiting
+        //        decision from OASIS on what will happen with callbacks
         // The callback endpoint will be resolved with the registry 
         // when the wire chains are created
         ref.getEndpointReferences().add(callbackEndpointReference);