You are viewing a plain text version of this content. The canonical link for it is here.
Posted to wsrp4j-dev@portals.apache.org by dl...@apache.org on 2006/11/28 13:30:26 UTC

svn commit: r480019 - /portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/ProducerImpl.java

Author: dlouzan
Date: Tue Nov 28 04:30:25 2006
New Revision: 480019

URL: http://svn.apache.org/viewvc?view=rev&rev=480019
Log:
Improved error handling for getting a new service description: now if we try to register with null registrationData, an exception is thrown

Modified:
    portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/ProducerImpl.java

Modified: portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/ProducerImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/ProducerImpl.java?view=diff&rev=480019&r1=480018&r2=480019
==============================================================================
--- portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/ProducerImpl.java (original)
+++ portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/ProducerImpl.java Tue Nov 28 04:30:25 2006
@@ -557,8 +557,14 @@
             
         } else {
             
-            // TODO: Either no registration data or producer does not support 
-            // in-band registration
+            if (registrationData == null) {
+                /* Either no registration data or producer does not support 
+                   in-band registration */
+                WSRPXHelper.throwX(log, ErrorCodes.INVALID_REGISTRATION_DATA);
+            } else if (registrationInterface == null) {
+                WSRPXHelper.throwX(log, ErrorCodes.INVALID_URL_OF_REGISTRATION_PORT);
+            }
+            
         }
         
         return this.registrationContext;