[ofbiz-framework] branch trunk updated: Fixed: Error while decoding url parameters with percent character (OFBIZ-12014)

[jleroux]

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 748e6c3  Fixed: Error while decoding url parameters with percent 
character (OFBIZ-12014)
748e6c3 is described below

commit 748e6c3452bdd840f563b66268ca3b513b536267
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Mon Sep 14 11:18:19 2020 +0200

    Fixed: Error while decoding url parameters with percent character 
(OFBIZ-12014)
    
    This has been already fixed (an clearly explained) in OFBIZ-10275 and broken
    again in OFBIZ-11822
    
    Thanks: Pradeep Choudhary
---
 framework/base/src/main/java/org/apache/ofbiz/base/util/UtilCodec.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilCodec.java 
b/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilCodec.java
index 5faae93..50c358a 100644
--- a/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilCodec.java
+++ b/framework/base/src/main/java/org/apache/ofbiz/base/util/UtilCodec.java
@@ -259,7 +259,7 @@ public class UtilCodec {
         @Override
         public String decode(String original) {
             try {
-                original = canonicalize(original);
+                canonicalize(original);
                 return URLDecoder.decode(original, "UTF-8");
             } catch (UnsupportedEncodingException ee) {
                 Debug.logError(ee, MODULE);