Author: slaws
Date: Fri Jan  6 12:40:14 2012
New Revision: 1228157

URL: http://svn.apache.org/viewvc?rev=1228157&view=rev
Log:
TUSCANY-4004 - add a comment highlighting some concern about import processing. 

Modified:
    
tuscany/sca-java-2.x/trunk/modules/interface-wsdl/src/main/java/org/apache/tuscany/sca/interfacedef/wsdl/xml/WSDLModelResolver.java

Modified: 
tuscany/sca-java-2.x/trunk/modules/interface-wsdl/src/main/java/org/apache/tuscany/sca/interfacedef/wsdl/xml/WSDLModelResolver.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/interface-wsdl/src/main/java/org/apache/tuscany/sca/interfacedef/wsdl/xml/WSDLModelResolver.java?rev=1228157&r1=1228156&r2=1228157&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/interface-wsdl/src/main/java/org/apache/tuscany/sca/interfacedef/wsdl/xml/WSDLModelResolver.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/interface-wsdl/src/main/java/org/apache/tuscany/sca/interfacedef/wsdl/xml/WSDLModelResolver.java
 Fri Jan  6 12:40:14 2012
@@ -511,7 +511,9 @@ public class WSDLModelResolver implement
             // set the correct WSDLDefinition import relationships.
             for (Map.Entry<String, List<javax.wsdl.Import>> entry :
                     ((Map<String, 
List<javax.wsdl.Import>>)definition.getImports()).entrySet()) {
-                if (!entry.getKey().equals(definition.getTargetNamespace())) { 
+                if (!entry.getKey().equals(definition.getTargetNamespace())) {
+                    // TUSCANY-4004 - This looks a bit dodgy as the 
wsdlDefinition object will be the same
+                    //                for multiple imports. 
                     WSDLDefinition wsdlDefinition = 
wsdlFactory.createWSDLDefinition();
                     wsdlDefinition.setUnresolved(true);
                     wsdlDefinition.setNamespace(entry.getKey());


Reply via email to