svn commit: r1171391 - /ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

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

svn commit: r1171391 - /ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

sascharodekamp
Author: sascharodekamp
Date: Fri Sep 16 05:54:18 2011
New Revision: 1171391

URL: http://svn.apache.org/viewvc?rev=1171391&view=rev
Log:
FIX: encodeURI before construct a new href value for nav-buttons in layered lookup dialog. (https://issues.apache.org/jira/browse/OFBIZ-4423) A patch from Leon which fixes an encoding issue in the lookup URLs: In IE, if you use link's href to call javascript function and its parameter contains encoded multi-bytes characters, such as "%E2%82%AC" which is the result of encodeURI("€"), IE will incorrectly unescape it to "€" and then javascript call return unexpected result.

Modified:
    ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

Modified: ofbiz/trunk/framework/images/webapp/images/fieldlookup.js
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/fieldlookup.js?rev=1171391&r1=1171390&r2=1171391&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/fieldlookup.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/fieldlookup.js Fri Sep 16 05:54:18 2011
@@ -472,7 +472,7 @@ function modifySubmitButton (lookupDiv)
         //modify nav-pager
         var navPagers = jQuery("#" + lookupDiv + " .nav-pager a");
         jQuery.each(navPagers, function(navPager) {
-            jQuery(navPagers[navPager]).attr("href", "javascript:lookupPaginationAjaxRequest('" + jQuery(navPagers[navPager]).attr("href") + "','link')");
+            jQuery(navPagers[navPager]).attr("href", "javascript:lookupPaginationAjaxRequest('" + encodeURI(jQuery(navPagers[navPager]).attr("href")) + "','link')");
         });
 
         var navPagersSelect = jQuery("#" + lookupDiv + " .nav-pager select");