Author: jkaputin
Date: Wed Jan  4 09:43:06 2006
New Revision: 365955

URL: http://svn.apache.org/viewcvs?rev=365955&view=rev
Log:
In the addXXX() methods, check for a null argument to
avoid adding nulls to the Lists.

Modified:
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/BindingOperationImpl.java
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/ConfigurableImpl.java
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DescriptionImpl.java
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DocumentableImpl.java
    incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceImpl.java
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceOperationImpl.java
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPHeaderBlockImpl.java
    
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPModuleImpl.java

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/BindingOperationImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/BindingOperationImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/BindingOperationImpl.java
 (original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/BindingOperationImpl.java
 Wed Jan  4 09:43:06 2006
@@ -109,8 +109,11 @@
     /* (non-Javadoc)
      * @see 
org.apache.woden.wsdl20.xml.BindingOperationElement#addMessageReferenceElement(org.apache.woden.wsdl20.xml.BindingMessageReferenceElement)
      */
-    public void addMessageReferenceElement(BindingMessageReferenceElement 
msgRef) {
-        fMessageRefs.add(msgRef);
+    public void addMessageReferenceElement(BindingMessageReferenceElement 
msgRef) 
+    {
+        if(msgRef != null) {
+            fMessageRefs.add(msgRef);
+        }
     }
 
     /* (non-Javadoc)
@@ -132,8 +135,11 @@
     /* (non-Javadoc)
      * @see 
org.apache.woden.wsdl20.xml.BindingOperationElement#addFaultReferenceElement(org.apache.woden.wsdl20.xml.FaultReferenceElement)
      */
-    public void addFaultReferenceElement(FaultReferenceElement faultRef) {
-        fFaultRefs.add(faultRef);
+    public void addFaultReferenceElement(FaultReferenceElement faultRef) 
+    {
+        if(faultRef != null) {
+            fFaultRefs.add(faultRef);
+        }
     }
 
     /* (non-Javadoc)

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/ConfigurableImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/ConfigurableImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/ConfigurableImpl.java 
(original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/ConfigurableImpl.java 
Wed Jan  4 09:43:06 2006
@@ -73,7 +73,9 @@
      */
     public void addFeatureElement(FeatureElement feature) 
     {
-        fFeatures.add(feature);
+        if(feature != null) {
+            fFeatures.add(feature);
+        }
     }
 
     /* (non-Javadoc)
@@ -91,7 +93,9 @@
      */
     public void addPropertyElement(PropertyElement property) 
     {
-        fProperties.add(property);
+        if(property != null) {
+            fProperties.add(property);
+        }
     }
 
     /* (non-Javadoc)

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DescriptionImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/DescriptionImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DescriptionImpl.java 
(original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DescriptionImpl.java 
Wed Jan  4 09:43:06 2006
@@ -265,7 +265,9 @@
     
     public void addImportElement(ImportElement importEl)
     {
-        fImportElements.add(importEl);
+        if(importEl != null) {
+            fImportElements.add(importEl);
+        }
     }
 
     public ImportElement[] getImportElements()
@@ -277,7 +279,9 @@
     
     public void addIncludeElement(IncludeElement include)
     {
-        fIncludeElements.add(include);
+        if(include != null) {
+            fIncludeElements.add(include);
+        }
     }
 
     public IncludeElement[] getIncludeElements()
@@ -299,7 +303,9 @@
 
     public void addInterfaceElement(InterfaceElement intface) 
     {
-        fInterfaceElements.add(intface);
+        if(intface != null) {
+            fInterfaceElements.add(intface);
+        }
     }
     
     public InterfaceElement[] getInterfaceElements() 
@@ -311,7 +317,9 @@
     
     public void addBindingElement(BindingElement binding)
     {
-        fBindingElements.add(binding);
+        if(binding != null) {
+            fBindingElements.add(binding);
+        }
     }
     
     public BindingElement[] getBindingElements() 
@@ -323,7 +331,9 @@
     
     public void addServiceElement(ServiceElement service)
     {
-        fServiceElements.add(service);
+        if(service != null) {
+            fServiceElements.add(service);
+        }
     }
     
     public ServiceElement[] getServiceElements() 
@@ -424,12 +434,18 @@
      *  Non-API implementation methods
      * ************************************************************/
 
-    public void addElementDeclaration(ElementDeclaration elDec) {
-        fElementDeclarations.add(elDec);
+    public void addElementDeclaration(ElementDeclaration elDec) 
+    {
+        if(elDec != null) {
+            fElementDeclarations.add(elDec);
+        }
     }
 
-    public void addTypeDefinition(TypeDefinition typeDef) {
-        fTypeDefinitions.add(typeDef);
+    public void addTypeDefinition(TypeDefinition typeDef) 
+    {
+        if(typeDef != null) {
+            fTypeDefinitions.add(typeDef);
+        }
     }
     
     private void initComponents() 

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DocumentableImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/DocumentableImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DocumentableImpl.java 
(original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/DocumentableImpl.java 
Wed Jan  4 09:43:06 2006
@@ -36,8 +36,11 @@
     /* (non-Javadoc)
      * @see 
org.apache.woden.wsdl20.xml.DocumentableElement#addDocumentationElement(org.apache.woden.wsdl20.xml.DocumentationElement)
      */
-    public void addDocumentationElement(DocumentationElement docEl) {
-        fDocumentationElements.add(docEl);
+    public void addDocumentationElement(DocumentationElement docEl) 
+    {
+        if(docEl != null) {
+            fDocumentationElements.add(docEl);
+        }
     }
 
     /* (non-Javadoc)

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceImpl.java 
(original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceImpl.java 
Wed Jan  4 09:43:06 2006
@@ -137,7 +137,9 @@
      */
     public void addStyleDefaultURI(URI uri)
     {
-        fStyleDefault.add(uri);
+        if(uri != null) {
+            fStyleDefault.add(uri);
+        }
     }
     
     /* 
@@ -155,7 +157,9 @@
      */
     public void addExtendsQName(QName qname)
     {
-        fExtendsQNames.add(qname);
+        if(qname != null) {
+            fExtendsQNames.add(qname);
+        }
     }
     
     /* 
@@ -173,7 +177,9 @@
      */
     public void addInterfaceFaultElement(InterfaceFaultElement fault)
     {
-        fInterfaceFaultElements.add(fault);
+        if(fault != null) {
+            fInterfaceFaultElements.add(fault);
+        }
     }
 
     /* 
@@ -191,7 +197,9 @@
      */
     public void addInterfaceOperationElement(InterfaceOperationElement 
operation)
     {
-        fInterfaceOperationElements.add(operation);
+        if(operation != null) {
+            fInterfaceOperationElements.add(operation);
+        }
     }
 
     /* 
@@ -208,8 +216,11 @@
      *  Non-API implementation methods
      * ************************************************************/
 
-    public void addExtendedInterface(Interface intface) {
-        fExtendedInterfaces.add(intface);
+    public void addExtendedInterface(Interface intface) 
+    {
+        if(intface != null) {
+            fExtendedInterfaces.add(intface);
+        }
     }
 
 }

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceOperationImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceOperationImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceOperationImpl.java
 (original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/InterfaceOperationImpl.java
 Wed Jan  4 09:43:06 2006
@@ -139,7 +139,9 @@
      */
     public void addStyleURI(URI uri)
     {
-        fStyle.add(uri);
+        if(uri != null) {
+            fStyle.add(uri);
+        }
     }
     
     /*
@@ -155,7 +157,9 @@
      */
     public void addMessageReferenceElement(InterfaceMessageReferenceElement 
msgRef)
     {
-        fMessageRefs.add(msgRef);
+        if(msgRef != null) {
+            fMessageRefs.add(msgRef);
+        }
     }
     
     /*
@@ -181,7 +185,9 @@
      */
     public void addFaultReferenceElement(FaultReferenceElement faultRef)
     {
-        fFaultRefs.add(faultRef);
+        if(faultRef != null) {
+            fFaultRefs.add(faultRef);
+        }
     }
     
     /*

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPHeaderBlockImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPHeaderBlockImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPHeaderBlockImpl.java
 (original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPHeaderBlockImpl.java
 Wed Jan  4 09:43:06 2006
@@ -157,8 +157,11 @@
     /* (non-Javadoc)
      * @see 
org.apache.woden.wsdl20.extensions.soap.SOAPHeaderBlockElement#addDocumentationElement(org.apache.woden.wsdl20.xml.DocumentationElement)
      */
-    public void addDocumentationElement(DocumentationElement docEl) {
-        fDocumentationElements.add(docEl);
+    public void addDocumentationElement(DocumentationElement docEl) 
+    {
+        if(docEl != null) {
+            fDocumentationElements.add(docEl);
+        }
     }
 
     /* (non-Javadoc)

Modified: 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPModuleImpl.java
URL: 
http://svn.apache.org/viewcvs/incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPModuleImpl.java?rev=365955&r1=365954&r2=365955&view=diff
==============================================================================
--- 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPModuleImpl.java
 (original)
+++ 
incubator/woden/java/src/org/apache/woden/internal/wsdl20/extensions/soap/SOAPModuleImpl.java
 Wed Jan  4 09:43:06 2006
@@ -111,8 +111,11 @@
     /* (non-Javadoc)
      * @see 
org.apache.woden.wsdl20.extensions.soap.SOAPModuleElement#addDocumentationElement(org.apache.woden.wsdl20.xml.DocumentationElement)
      */
-    public void addDocumentationElement(DocumentationElement docEl) {
-        fDocumentationElements.add(docEl);
+    public void addDocumentationElement(DocumentationElement docEl) 
+    {
+        if(docEl != null) {
+            fDocumentationElements.add(docEl);
+        }
     }
 
     /* (non-Javadoc)



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to