svn commit: r1071362 - /ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java

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

svn commit: r1071362 - /ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java

sascharodekamp
Author: sascharodekamp
Date: Wed Feb 16 19:06:10 2011
New Revision: 1071362

URL: http://svn.apache.org/viewvc?rev=1071362&view=rev
Log:
Feature - "LoginWorker should use expandedEntites von the checkLogin-query" (https://issues.apache.org/jira/browse/OFBIZ-3453) - The parameters in the query String are separated by an &. The Code is more clean (and XHTML strict) when the separator is encoded. Thanks Dimitri for the patch.

Orig. Msg.:
I guess that the makeLoginUrl-method in the LoginWorker should return a string with expandedEntites.

Modified:
    ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java

Modified: ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java?rev=1071362&r1=1071361&r2=1071362&view=diff
==============================================================================
--- ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java (original)
+++ ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/control/LoginWorker.java Wed Feb 16 19:06:10 2011
@@ -98,7 +98,7 @@ public class LoginWorker {
     }
     public static StringWrapper makeLoginUrl(HttpServletRequest request, String requestName) {
         Map<String, Object> urlParams = UtilHttp.getUrlOnlyParameterMap(request);
-        String queryString = UtilHttp.urlEncodeArgs(urlParams, false);
+        String queryString = UtilHttp.urlEncodeArgs(urlParams);
         String currentView = UtilFormatOut.checkNull((String) request.getAttribute("_CURRENT_VIEW_"));
 
         String loginUrl = "/" + requestName;