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/12/29 01:34:45 UTC

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

Author: dlouzan
Date: Thu Dec 28 16:34:44 2006
New Revision: 490878

URL: http://svn.apache.org/viewvc?view=rev&rev=490878
Log:
- Modified ProducerImpl according to modifications to its superclass in revision 490877
- Modified exception handling on GenericPersistentProducerRegistryImpl

Modified:
    portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/GenericPersistentProducerRegistryImpl.java
    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/GenericPersistentProducerRegistryImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/GenericPersistentProducerRegistryImpl.java?view=diff&rev=490878&r1=490877&r2=490878
==============================================================================
--- portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/GenericPersistentProducerRegistryImpl.java (original)
+++ portals/wsrp4j/trunk/commons-consumer/src/java/org/apache/wsrp4j/commons/consumer/driver/producer/GenericPersistentProducerRegistryImpl.java Thu Dec 28 16:34:44 2006
@@ -112,22 +112,11 @@
 
     public void stateChanged(StateChangedEvent event) {
 
+        Producer producer = (Producer) event.getSource();
         try {
-
-            Producer producer = (Producer) event.getSource();
             store(producer);
-
-        } catch (Throwable t) {
-
-            if (t instanceof ClassCastException) {
-
-                // TODO: throw exception
-            }
-
-            if (t instanceof WSRPException) {
-
-                // TODO: throw exception
-            }
+        } catch (WSRPException e) {
+            throw new Error(e);
         }
     }
 

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=490878&r1=490877&r2=490878
==============================================================================
--- 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 Thu Dec 28 16:34:44 2006
@@ -319,7 +319,7 @@
      * @param registrationRequired True if the producer requires in-band 
      * registration
      **/
-    public void setIsRegistrationRequired(boolean registrationRequired) {
+    public void setRegistrationRequired(boolean registrationRequired) {
         this.registrationRequired = registrationRequired;
         stateChanged();
     }
@@ -490,7 +490,7 @@
         if (serviceDescription != null) {
             this.serviceDescription = serviceDescription;
             updatePortletDescriptions(serviceDescription);
-            setIsRegistrationRequired(
+            setRegistrationRequired(
                     serviceDescription.isRequiresRegistration());
         }
     }