You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ad...@apache.org on 2013/08/27 18:56:14 UTC

svn commit: r1517880 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/MacroMenuRenderer.java

Author: adrianc
Date: Tue Aug 27 16:56:14 2013
New Revision: 1517880

URL: http://svn.apache.org/r1517880
Log:
Fixed a bug in MacroMenuRenderer.java - reported by Hans on the dev mailing list.

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/MacroMenuRenderer.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/MacroMenuRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/MacroMenuRenderer.java?rev=1517880&r1=1517879&r2=1517880&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/MacroMenuRenderer.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/menu/MacroMenuRenderer.java Tue Aug 27 16:56:14 2013
@@ -227,6 +227,9 @@ public class MacroMenuRenderer implement
             targetParameters.append("]");
 
         }
+        if (targetParameters.length() == 0) {
+            targetParameters.append("\"\"");
+        }
         if (UtilValidate.isNotEmpty(target)) {
             if (!"hidden-form".equals(linkType)) {
                 StringBuilder sb = new StringBuilder();
@@ -236,7 +239,7 @@ public class MacroMenuRenderer implement
         }
         parameters.put("linkUrl", linkUrl);
         parameters.put("actionUrl", actionUrl);
-        parameters.put("parameterList", targetParameters.toString());
+        parameters.put("parameterList", targetParameters);
         String imgStr = "";
         Image img = link.getImage();
         if (img != null) {