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 2009/10/06 12:36:12 UTC

svn commit: r822207 - /tuscany/java/sca/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ComponentBuilderImpl.java

Author: slaws
Date: Tue Oct  6 10:36:12 2009
New Revision: 822207

URL: http://svn.apache.org/viewvc?rev=822207&view=rev
Log:
TUSCANY-3219 - switch the order of the interface contracts when tested for compatibility so that the top level contracts is tested as being a superset of the bottom level contract

Modified:
    tuscany/java/sca/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ComponentBuilderImpl.java

Modified: tuscany/java/sca/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ComponentBuilderImpl.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ComponentBuilderImpl.java?rev=822207&r1=822206&r2=822207&view=diff
==============================================================================
--- tuscany/java/sca/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ComponentBuilderImpl.java (original)
+++ tuscany/java/sca/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ComponentBuilderImpl.java Tue Oct  6 10:36:12 2009
@@ -1036,7 +1036,7 @@
             topContract.setInterfaceContract(bottomInterfaceContract);
         } else if (bottomInterfaceContract != null) {
             // Check that the top and bottom interface contracts are compatible
-            boolean isCompatible = interfaceContractMapper.isCompatible(topInterfaceContract, bottomInterfaceContract);
+            boolean isCompatible = interfaceContractMapper.isCompatible(bottomInterfaceContract, topInterfaceContract);
             if (!isCompatible) {
                 if (topContract instanceof Reference) {
                     Monitor.error(monitor,