svn commit: r1244402 - in /ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget: form/MacroFormRenderer.java html/HtmlFormRenderer.java

[sascharodekamp]

Author: sascharodekamp
Date: Wed Feb 15 08:54:51 2012
New Revision: 1244402

URL: http://svn.apache.org/viewvc?rev=1244402&view=rev
Log:
BUGFIX: return value of String.replace(CharSequence, CharSequence) ignored in 
org.ofbiz.widget.form.[Macro/Html]FormRenderer.appendExternalLoginKey(String) 
(https://issues.apache.org/jira/browse/OFBIZ-4694): A patch from Michael Brohl 
Same error in both Classes MacroFormRenderer and HtmlFormRenderer.
Will provide a patch.

Modified:
    
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
    
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java

Modified: 
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java?rev=1244402&r1=1244401&r2=1244402&view=diff
==============================================================================
--- 
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
 (original)
+++ 
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
 Wed Feb 15 08:54:51 2012
@@ -2100,7 +2100,7 @@ public class MacroFormRenderer implement
         if (questionIndex == -1) {
             result += sessionId;
         } else {
-            result.replace("?", sessionId + "?");
+            result = result.replace("?", sessionId + "?");
         }
         return result;
     }

Modified: 
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
URL: 
http://svn.apache.org/viewvc/ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java?rev=1244402&r1=1244401&r2=1244402&view=diff
==============================================================================
--- 
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
 (original)
+++ 
ofbiz/branches/release10.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
 Wed Feb 15 08:54:51 2012
@@ -2285,7 +2285,7 @@ public class HtmlFormRenderer extends Ht
         if (questionIndex == -1) {
             result += sessionId;
         } else {
-            result.replace("?", sessionId + "?");
+            result = result.replace("?", sessionId + "?");
         }
         return result;
     }