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;