You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@click.apache.org by sa...@apache.org on 2010/04/25 17:01:52 UTC

svn commit: r937808 - in /click/trunk/click/framework/src/org/apache/click: ClickServlet.java service/XmlConfigService.java

Author: sabob
Date: Sun Apr 25 15:01:52 2010
New Revision: 937808

URL: http://svn.apache.org/viewvc?rev=937808&view=rev
Log:
improved log messages

Modified:
    click/trunk/click/framework/src/org/apache/click/ClickServlet.java
    click/trunk/click/framework/src/org/apache/click/service/XmlConfigService.java

Modified: click/trunk/click/framework/src/org/apache/click/ClickServlet.java
URL: http://svn.apache.org/viewvc/click/trunk/click/framework/src/org/apache/click/ClickServlet.java?rev=937808&r1=937807&r2=937808&view=diff
==============================================================================
--- click/trunk/click/framework/src/org/apache/click/ClickServlet.java (original)
+++ click/trunk/click/framework/src/org/apache/click/ClickServlet.java Sun Apr 25 15:01:52 2010
@@ -187,6 +187,7 @@ public class ClickServlet extends HttpSe
      * @throws ServletException if the application configuration service could
      * not be initialized
      */
+    @Override
     public void init() throws ServletException {
 
         try {
@@ -197,8 +198,9 @@ public class ClickServlet extends HttpSe
             logger = configService.getLogService();
 
             if (logger.isInfoEnabled()) {
-                logger.info("initialized in " + configService.getApplicationMode()
-                            + " mode");
+                logger.info("Click " + ClickUtils.getClickVersion()
+                    + " initialized in " + configService.getApplicationMode()
+                    + " mode");
             }
 
             resourceService = configService.getResourceService();
@@ -224,6 +226,7 @@ public class ClickServlet extends HttpSe
     /**
      * @see javax.servlet.GenericServlet#destroy()
      */
+    @Override
     public void destroy() {
 
         try {
@@ -266,6 +269,7 @@ public class ClickServlet extends HttpSe
      * @throws ServletException if click app has not been initialized
      * @throws IOException if an I/O error occurs
      */
+    @Override
     protected void doGet(HttpServletRequest request,
             HttpServletResponse response) throws ServletException, IOException {
 
@@ -283,6 +287,7 @@ public class ClickServlet extends HttpSe
      * @throws ServletException if click app has not been initialized
      * @throws IOException if an I/O error occurs
      */
+    @Override
     protected void doPost(HttpServletRequest request,
             HttpServletResponse response) throws ServletException, IOException {
 

Modified: click/trunk/click/framework/src/org/apache/click/service/XmlConfigService.java
URL: http://svn.apache.org/viewvc/click/trunk/click/framework/src/org/apache/click/service/XmlConfigService.java?rev=937808&r1=937807&r2=937808&view=diff
==============================================================================
--- click/trunk/click/framework/src/org/apache/click/service/XmlConfigService.java (original)
+++ click/trunk/click/framework/src/org/apache/click/service/XmlConfigService.java Sun Apr 25 15:01:52 2010
@@ -1407,7 +1407,8 @@ public class XmlConfigService implements
         } else if (modeValue.equalsIgnoreCase("trace")) {
             mode = TRACE;
         } else {
-            logService.error("invalid application mode: " + modeValue);
+            logService.error("invalid application mode: '" + modeValue
+                + "' - defaulted to '" + MODE_VALUES[DEBUG] + "'");
             mode = DEBUG;
         }
 
@@ -1545,11 +1546,12 @@ public class XmlConfigService implements
 
         logService.onInit(getServletContext());
 
-        if (getLogService().isDebugEnabled()) {
-            String msg = "initialized LogService: "
-                + logService.getClass().getName();
-            getLogService().debug(msg);
-        }
+        logService.info("***  Initializing Click " + ClickUtils.getClickVersion()
+            + "  ***");
+
+        String msg = "initialized LogService: "
+            + logService.getClass().getName();
+        getLogService().info(msg);
     }
 
     private void loadPageInterceptors(Element rootElm) throws Exception {