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/03/04 12:01:37 UTC

svn commit: r918931 - /tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

Author: slaws
Date: Thu Mar  4 11:01:36 2010
New Revision: 918931

URL: http://svn.apache.org/viewvc?rev=918931&view=rev
Log:
improve the operation not found error message

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

Modified: tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
URL: http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java?rev=918931&r1=918930&r2=918931&view=diff
==============================================================================
--- tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java (original)
+++ tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java Thu Mar  4 11:01:36 2010
@@ -259,7 +259,11 @@
             Operation targetOperation = map(target.getInterface(), operation);
             if (targetOperation == null) {
                 if (!silent) {
-                    throw new IncompatibleInterfaceContractException("Operation not found on target", source, target);
+                    throw new IncompatibleInterfaceContractException("Operation " + 
+                                                                     operation.getName() +
+                                                                     " not found on target", 
+                                                                     source, 
+                                                                     target);
                 } else {
                     return false;
                 }
@@ -268,8 +272,11 @@
                 // FIXME: for remotable operation, only compare name for now
                 if (!isCompatible(operation, targetOperation, Compatibility.SUBSET)) {
                     if (!silent) {
-                        throw new IncompatibleInterfaceContractException("Target operations are not compatible",
-                                                                         source, target);
+                        throw new IncompatibleInterfaceContractException("Target operations called " +
+                                                                         operation.getName() +
+                                                                         " are not compatible",
+                                                                         source, 
+                                                                         target);
                     } else {
                         return false;
                     }