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/10/11 13:11:26 UTC

svn commit: r462766 - in /portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons: util/configuration/ConfigurationParametersManager.java ws/WSFactoryFinder.java

Author: dlouzan
Date: Wed Oct 11 04:11:25 2006
New Revision: 462766

URL: http://svn.apache.org/viewvc?view=rev&rev=462766
Log:
Misc log management modifications.

Modified:
    portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/configuration/ConfigurationParametersManager.java
    portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/ws/WSFactoryFinder.java

Modified: portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/configuration/ConfigurationParametersManager.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/configuration/ConfigurationParametersManager.java?view=diff&rev=462766&r1=462765&r2=462766
==============================================================================
--- portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/configuration/ConfigurationParametersManager.java (original)
+++ portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/util/configuration/ConfigurationParametersManager.java Wed Oct 11 04:11:25 2006
@@ -74,12 +74,18 @@
         
         if (value == null) {
             
-            if (log.isInfoEnabled()) {
-                log.info("Missing configuration parameter '" + name +
+            if (log.isErrorEnabled()) {
+                log.error("Missing configuration parameter '" + name +
                         "' from '" + CONFIGURATION_FILE +
                         "' configuration file");
             }
             throw new MissingConfigurationParameterException(name);
+        }
+        
+        if (log.isInfoEnabled()) {
+            log.info("Loading configuration parameter '" + name +
+                    "' with value '" + value + "' from '" +
+                    CONFIGURATION_FILE + "' configuration file");
         }
         
         return value;

Modified: portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/ws/WSFactoryFinder.java
URL: http://svn.apache.org/viewvc/portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/ws/WSFactoryFinder.java?view=diff&rev=462766&r1=462765&r2=462766
==============================================================================
--- portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/ws/WSFactoryFinder.java (original)
+++ portals/wsrp4j/trunk/commons/src/java/org/apache/wsrp4j/commons/ws/WSFactoryFinder.java Wed Oct 11 04:11:25 2006
@@ -54,7 +54,8 @@
             
         } catch (Exception e) {
             if (log.isErrorEnabled()) {
-                log.error("Error loading 'wsrp4j.ws.factory' class.", e);
+                log.error("Error loading " + FACTORY_CLASS_NAME_PARAMETER 
+                        + " class.", e);
             }
         }