re: svn commit 574709

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

re: svn commit 574709

cjhowe
track-stats (renamed later track-serverhit) isn't an initiated
parameter by default so therefore only works when the individual
webapps web.xml contains the variable track-serverhit.  Shouldn't there
be a config setting somewhere to set this globally and then overridden
by individual webapp preferences?  ServerHit db logging should be the
"show all tools" of a default checkout.

---
ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/ControlServlet.java
2007/09/11 22:00:36 574708
+++
ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/ControlServlet.java
2007/09/11 22:09:40 574709
@@ -281,7 +281,11 @@
             try {
                 UtilHttp.setInitialRequestInfo(request);
                 VisitHandler.getVisitor(request, response);
-                ServerHitBin.countRequest(webappName + "." + rname,
request, requestStartTime, System.currentTimeMillis() -
requestStartTime, userLogin, delegator);
+                if (requestHandler.trackStats(request)) {
+                    ServerHitBin.countRequest(webappName + "." +
rname, request, requestStartTime, System.currentTimeMillis() -
requestStartTime, userLogin, delegator);
+                } else {
+                    Debug.log("Track status is disabled for this
request");
+                }
             } catch (Throwable t) {
                 Debug.logError(t, "Error in ControlServlet saving
ServerHit/Bin information; the output was successful, but can't save
this tracking information. The error was: " + t.toString(), module);
             }