[Amdatu-commits] SVN r775 - in trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher: handler service

[subvers...@amdatu.org]

Author: bdekruijff at gmail.com
Date: Thu Feb 10 15:47:24 2011
New Revision: 775

Log:
AMDATU-282 minor code changes from review

Modified:
   
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/handler/FilterHandlerRegistry.java
   
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/service/DispatcherServiceImpl.java

Modified: 
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/handler/FilterHandlerRegistry.java
==============================================================================
--- 
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/handler/FilterHandlerRegistry.java
      (original)
+++ 
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/handler/FilterHandlerRegistry.java
      Thu Feb 10 15:47:24 2011
@@ -85,7 +85,8 @@
                 Arrays.sort(m_extenderFilterHandlerArray);
             }
             catch (ServletException e) {
-                e.printStackTrace();
+                getLogService().log(LogService.LOG_ERROR,
+                    "ServletException while initializing extenderfilter with 
serviceid " + serviceId + "", e);
             }
         }
         finally {
@@ -146,7 +147,8 @@
                 m_tenantFilterHandlerArrays.clear();
             }
             catch (ServletException e) {
-                e.printStackTrace();
+                getLogService().log(LogService.LOG_ERROR,
+                    "ServletException while initializing filter with serviceid 
" + serviceId + "", e);
             }
         }
         finally {

Modified: 
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/service/DispatcherServiceImpl.java
==============================================================================
--- 
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/service/DispatcherServiceImpl.java
      (original)
+++ 
trunk/amdatu-web/dispatcher/src/main/java/org/amdatu/web/dispatcher/service/DispatcherServiceImpl.java
      Thu Feb 10 15:47:24 2011
@@ -95,11 +95,12 @@
     }
 
     public void stop() {
+        m_httpService.unregisterFilter(m_interceptorFilter);
+
         m_servletHandlerRegistry = null;
         m_filterHandlerRegistry = null;
         m_httpContextManager = null;
 
-        m_httpService.unregisterFilter(m_interceptorFilter);
         m_interceptorFilter.setDispatcherService(null);
         m_interceptorFilter.setLogService(null);
         m_interceptorFilter = null;