svn commit: r1177923 - /ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java

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

svn commit: r1177923 - /ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java

jleroux@apache.org
Author: jleroux
Date: Sat Oct  1 05:42:30 2011
New Revision: 1177923

URL: http://svn.apache.org/viewvc?rev=1177923&view=rev
Log:
On David's remark on dev ml: back to simple solution: simply get the delegator from the request (and not the session as previously buggy when anonymous was used), don't try to find if it misses then throw an exception

Modified:
    ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java

Modified: ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java?rev=1177923&r1=1177922&r2=1177923&view=diff
==============================================================================
--- ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java (original)
+++ ofbiz/branches/release10.04/framework/webapp/src/org/ofbiz/webapp/stats/ServerHitBin.java Sat Oct  1 05:42:30 2011
@@ -112,9 +112,6 @@ public class ServerHitBin {
 
     protected static void countHit(String baseId, int type, HttpServletRequest request, long startTime, long runningTime, GenericValue userLogin, boolean isOriginal) {
         Delegator delegator = (Delegator)request.getAttribute("delegator");
-        if (delegator == null){
-            delegator = DelegatorFactory.getDelegator("default");
-        }
         if (delegator == null) {
             throw new IllegalArgumentException("In countHit could not find a delegator or delegatorName to work from");
         }