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

[sascharodekamp]

Author: sascharodekamp
Date: Wed Feb 15 08:55:40 2012
New Revision: 1244403

URL: http://svn.apache.org/viewvc?rev=1244403&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/release11.04/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
    
ofbiz/branches/release11.04/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java

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

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