You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tuscany.apache.org by an...@apache.org on 2008/04/24 10:13:40 UTC

svn commit: r651191 - /incubator/tuscany/java/sca/modules/interface/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

Author: antelder
Date: Thu Apr 24 01:13:38 2008
New Revision: 651191

URL: http://svn.apache.org/viewvc?rev=651191&view=rev
Log:
Avoid NPE when the target op is null

Modified:
    incubator/tuscany/java/sca/modules/interface/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

Modified: incubator/tuscany/java/sca/modules/interface/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/interface/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java?rev=651191&r1=651190&r2=651191&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/interface/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java (original)
+++ incubator/tuscany/java/sca/modules/interface/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java Thu Apr 24 01:13:38 2008
@@ -280,7 +280,7 @@
      */
     public Operation map(Interface target, Operation source) {
         // TODO: How to handle the case that source operation is dynamic?
-        if (target.isDynamic()) {
+        if (target == null || target.isDynamic()) {
             return source;
         } else if (target.isRemotable()) {
             for (Operation op : target.getOperations()) {