You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by ba...@apache.org on 2007/02/16 20:39:37 UTC

svn commit: r508546 - /webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/ServiceDescriptionImpl.java

Author: barrettj
Date: Fri Feb 16 11:39:36 2007
New Revision: 508546

URL: http://svn.apache.org/viewvc?view=rev&rev=508546
Log:
Fix NPE in provider validation logic when binding type annotation not specified.

Modified:
    webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/ServiceDescriptionImpl.java

Modified: webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/ServiceDescriptionImpl.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/ServiceDescriptionImpl.java?view=diff&rev=508546&r1=508545&r2=508546
==============================================================================
--- webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/ServiceDescriptionImpl.java (original)
+++ webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/ServiceDescriptionImpl.java Fri Feb 16 11:39:36 2007
@@ -781,7 +781,14 @@
         } else {
             // We are in MESSAGE mode
             // Conformance: JAXWS Spec.- Sec. 4.3 (javax.activation.DataSource)
-            String bindingType = composite.getBindingTypeAnnot().value();
+            
+            // REVIEW: Should the provider interface validation be moved to post-construction validation, 
+            // since it seems that the logic to understand the default values for binding type 
+            // (see comment below) should be left to the creation of the Description objects.
+            String bindingType = null;
+            if (composite.getBindingTypeAnnot() != null ) {
+                bindingType = composite.getBindingTypeAnnot().value();
+            }
 
             Iterator<String> iter = composite.getInterfacesList().iterator();
 



---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org