clickstream/src/com/opensymphony/clickstream ClickstreamFilter.java,1.3,1.4 ClickstreamListener.java,1.2,1.3

Update of /cvsroot/opensymphony/clickstream/src/com/opensymphony/clickstream
In directory sc8-pr-cvs1:/tmp/cvs-serv18926/clickstream/src/com/opensymphony/clickstream

Modified Files:
	ClickstreamFilter.java ClickstreamListener.java 
Log Message:
Fixed CLK-9, switched over to commons logging and did some cosmetics..

Index: ClickstreamFilter.java
===================================================================
RCS file: /cvsroot/opensymphony/clickstream/src/com/opensymphony/clickstream/ClickstreamFilter.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -d -r1.3 -r1.4
--- ClickstreamFilter.java	8 Jun 2002 19:56:51 -0000	1.3
+++ ClickstreamFilter.java	2 Sep 2003 09:40:47 -0000	1.4
@@ -1,8 +1,14 @@
 package com.opensymphony.clickstream;
 
-import com.opensymphony.util.Logger;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
-import javax.servlet.*;
+import javax.servlet.Filter;
+import javax.servlet.FilterChain;
+import javax.servlet.FilterConfig;
+import javax.servlet.ServletException;
+import javax.servlet.ServletRequest;
+import javax.servlet.ServletResponse;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpSession;
 import java.io.IOException;
@@ -13,7 +19,7 @@
  * @author <a href="[email protected]">Patrick Lightbody</a>
  */
 public class ClickstreamFilter implements Filter {
-    private static Logger log = new Logger(ClickstreamFilter.class);
+    private static Log log = LogFactory.getLog(ClickstreamFilter.class);
 
     protected FilterConfig filterConfig;
     private final static String FILTER_APPLIED = "_clickstream_filter_applied";

Index: ClickstreamListener.java
===================================================================
RCS file: /cvsroot/opensymphony/clickstream/src/com/opensymphony/clickstream/ClickstreamListener.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -d -r1.2 -r1.3
--- ClickstreamListener.java	2 Jul 2002 14:16:28 -0000	1.2
+++ ClickstreamListener.java	2 Sep 2003 09:40:47 -0000	1.3
@@ -1,7 +1,8 @@
 package com.opensymphony.clickstream;
 
-import com.opensymphony.util.Logger;
 import com.opensymphony.clickstream.logger.ClickstreamLoggerFactory;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 import javax.servlet.ServletContextEvent;
 import javax.servlet.ServletContextListener;
@@ -19,7 +20,7 @@
  * @author <a href="[email protected]">Patrick Lightbody</a>
  */
 public class ClickstreamListener implements ServletContextListener, HttpSessionListener {
-    private static Logger log = new Logger(ClickstreamListener.class);
+    private static final Log log = LogFactory.getLog(ClickstreamListener.class);
 
     private Map clickstreams = new HashMap();
 
@@ -49,7 +50,15 @@
         log.debug("Session Destroyed");
         HttpSession session = hse.getSession();
         Clickstream stream = (Clickstream) clickstreams.get(session.getId());
-        ClickstreamLoggerFactory.getLogger().log(stream);
-        clickstreams.remove(session.getId());
+
+        try {
+            ClickstreamLoggerFactory.getLogger().log(stream);
+        }
+        catch (Exception e) {
+            log.error(e);
+        }
+        finally {
+            clickstreams.remove(session.getId());
+        }
     }
-}
+}
\ No newline at end of file




-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf