Author: schof
Date: Sun Sep  3 18:28:06 2006
New Revision: 439895

URL: http://svn.apache.org/viewvc?view=rev&rev=439895
Log:
Issue warning when no dialog config information is found in the default 
locations and none are found/specified by the contex param locations.  
(SHALE-272)

Modified:
    
shale/sandbox/shale-dialog2-legacy/src/main/java/org/apache/shale/dialog2/legacy/LegacyDialogManager.java

Modified: 
shale/sandbox/shale-dialog2-legacy/src/main/java/org/apache/shale/dialog2/legacy/LegacyDialogManager.java
URL: 
http://svn.apache.org/viewvc/shale/sandbox/shale-dialog2-legacy/src/main/java/org/apache/shale/dialog2/legacy/LegacyDialogManager.java?view=diff&rev=439895&r1=439894&r2=439895
==============================================================================
--- 
shale/sandbox/shale-dialog2-legacy/src/main/java/org/apache/shale/dialog2/legacy/LegacyDialogManager.java
 (original)
+++ 
shale/sandbox/shale-dialog2-legacy/src/main/java/org/apache/shale/dialog2/legacy/LegacyDialogManager.java
 Sun Sep  3 18:28:06 2006
@@ -35,8 +35,8 @@
 /**
  * <p>Implementation of [EMAIL PROTECTED] DialogContextManager} for integrating
  * legacy dialog support into the Shale Dialog Manager.</p>
- * 
- * 
+ *
+ *
  * @since 1.0.4
  */
 public final class LegacyDialogManager implements DialogContextManager, 
Serializable {
@@ -185,8 +185,6 @@
             resources = "";
             // Because we process the default resource even if it is not
             // explicitly listed, we don't want to warn here
-//            log().warn("No dialog configuration information present.  Comma 
separated files should be listed under the " +
-//                    "context parameter: " + Globals.CONFIGURATION);
         }
         int comma = 0;
         String path = null;
@@ -201,6 +199,11 @@
                     resources = resources.substring(comma + 1);
                 }
                 if (path.length() < 1) {
+                    if (log().isWarnEnabled()) {
+                        log.warn("No dialog configuration information present. 
 No default configuration found at: " +
+                                DEFAULT_CONFIGURATION_RESOURCE + ".  No 
embedded configuration found at: " +
+                                EMBEDDED_CONFIGURATION_RESOURCE);
+                    }
                     break;
                 }
                 URL resource = (URL) 
context.getExternalContext().getResource(path);


Reply via email to