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:54:28 UTC

svn commit: r490896 - in /portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer: configportlet/services/ proxyportlet/impl/

Author: dlouzan
Date: Thu Dec 28 16:54:27 2006
New Revision: 490896

URL: http://svn.apache.org/viewvc?view=rev&rev=490896
Log:
- Modified exception handling
- Misc formatting

Modified:
    portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/configportlet/services/ProducerRegistryService.java
    portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/ProducerRegistryImpl.java
    portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/URLGeneratorImpl.java
    portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/WSRPRequestImpl.java

Modified: portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/configportlet/services/ProducerRegistryService.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/configportlet/services/ProducerRegistryService.java?view=diff&rev=490896&r1=490895&r2=490896
==============================================================================
--- portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/configportlet/services/ProducerRegistryService.java (original)
+++ portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/configportlet/services/ProducerRegistryService.java Thu Dec 28 16:54:27 2006
@@ -40,8 +40,6 @@
 import org.apache.wsrp4j.consumer.configportlet.model.to.ProducerTO;
 import org.apache.wsrp4j.consumer.configportlet.util.Constants;
 
-import org.apache.wsrp4j.consumer.proxyportlet.impl.ProducerRegistryImpl;
-
 
 /**
  * @version $Id$

Modified: portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/ProducerRegistryImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/ProducerRegistryImpl.java?view=diff&rev=490896&r1=490895&r2=490896
==============================================================================
--- portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/ProducerRegistryImpl.java (original)
+++ portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/ProducerRegistryImpl.java Thu Dec 28 16:54:27 2006
@@ -15,6 +15,9 @@
  */
 package org.apache.wsrp4j.consumer.proxyportlet.impl;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 import org.apache.wsrp4j.commons.util.configuration.ConfigurationParametersManager;
 
 import org.apache.wsrp4j.commons.consumer.interfaces.producer.ProducerRegistry;
@@ -30,6 +33,8 @@
  */
 public class ProducerRegistryImpl extends GenericPersistentProducerRegistryImpl {
 
+    private static final Log log = LogFactory.getLog(ProducerRegistryImpl.class);
+    
     private static ProducerRegistry instance;
     
     private static final String XML_PERSISTENCE_DIR_PARAMETER =
@@ -52,11 +57,11 @@
 
             restore();
 
-        } catch (Throwable t) {
+        } catch (Exception e) {
 
-            t.printStackTrace();
+            log.error("Error while instantiating producer registry", e);
 
-            if (t instanceof WSRPException) {
+            if (e instanceof WSRPException) {
 
                 // TODO: throw exception
             } else {

Modified: portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/URLGeneratorImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/URLGeneratorImpl.java?view=diff&rev=490896&r1=490895&r2=490896
==============================================================================
--- portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/URLGeneratorImpl.java (original)
+++ portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/URLGeneratorImpl.java Thu Dec 28 16:54:27 2006
@@ -26,7 +26,7 @@
 import javax.portlet.WindowState;
 
 import org.apache.wsrp4j.commons.consumer.interfaces.urlgenerator.URLGenerator;
-import org.apache.wsrp4j.commons.consumer.util.PortletUtils;
+import org.apache.wsrp4j.commons.consumer.util.portlet.PortletUtils;
 
 import org.apache.wsrp4j.commons.util.Constants;
 import org.apache.wsrp4j.commons.util.Modes;

Modified: portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/WSRPRequestImpl.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/WSRPRequestImpl.java?view=diff&rev=490896&r1=490895&r2=490896
==============================================================================
--- portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/WSRPRequestImpl.java (original)
+++ portals/wsrp4j/trunk/consumer-proxyportlet/src/java/org/apache/wsrp4j/consumer/proxyportlet/impl/WSRPRequestImpl.java Thu Dec 28 16:54:27 2006
@@ -44,7 +44,7 @@
 import org.apache.wsrp4j.commons.util.Constants;
 import org.apache.wsrp4j.commons.util.LocaleHelper;
 import org.apache.wsrp4j.commons.util.Modes;
-import org.apache.wsrp4j.commons.consumer.util.PortletUtils;
+import org.apache.wsrp4j.commons.consumer.util.portlet.PortletUtils;
 import org.apache.wsrp4j.commons.util.WindowStates;
 import org.apache.wsrp4j.commons.util.Utility;