You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2012/06/20 11:19:09 UTC

svn commit: r1352005 - in /ofbiz/branches/release12.04: ./ framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java

Author: jacopoc
Date: Wed Jun 20 09:19:08 2012
New Revision: 1352005

URL: http://svn.apache.org/viewvc?rev=1352005&view=rev
Log:
Applied fix from trunk for revision: 1351778 
===

Setting a default value of DeltaManager was causing a side effect: the method setWebContextObjects was always invoked with the persistSerialized argument set to false; this was causing a series of issues (including one reported recently happening when two users login/logout to the same application from the same browser and visit another application).
 
 The commit when this bug was introduced is:
 
 Author: jleroux
 Date: Mon Jun  6 20:26:35 2011
 New Revision: 1132749
 
 and unfortunately the revision was backported to the release branches; the commit may have introduced other issues but I can't review it now.

Modified:
    ofbiz/branches/release12.04/   (props changed)
    ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java

Propchange: ofbiz/branches/release12.04/
------------------------------------------------------------------------------
  Merged /ofbiz/trunk:r1351778

Modified: ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java?rev=1352005&r1=1352004&r2=1352005&view=diff
==============================================================================
--- ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java (original)
+++ ofbiz/branches/release12.04/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java Wed Jun 20 09:19:08 2012
@@ -659,7 +659,7 @@ public class LoginWorker {
             String mgrClassName = null;
             try {
                 cc = ContainerConfig.getContainer("catalina-container", configFile);
-                mgrClassName = ContainerConfig.getPropertyValue(cc, "manager-class", "org.apache.catalina.ha.session.DeltaManager");
+                mgrClassName = ContainerConfig.getPropertyValue(cc, "manager-class", "");
             } catch (ContainerException e) {
                 Debug.logError(e, "No catalina-container configuration found in container config!");
             }