svn commit: r1801336 - /ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java

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

svn commit: r1801336 - /ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java

Deepak Dixit-5
Author: deepak
Date: Sat Jul  8 22:54:06 2017
New Revision: 1801336

URL: http://svn.apache.org/viewvc?rev=1801336&view=rev
Log:
Fixed: fix tag "disable-if-empty" in menus if field has no value (OFBIZ-9377)
Thanks Gaudin Pierre and Humera Khan for your contribution.

Modified:
    ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java

Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java?rev=1801336&r1=1801335&r2=1801336&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroMenuRenderer.java Sat Jul  8 22:54:06 2017
@@ -147,7 +147,7 @@ public class MacroMenuRenderer implement
             List<String> keys = StringUtil.split(disableIfEmpty, "|");
             for (String key : keys) {
                 Object obj = context.get(key);
-                if (obj == null) {
+                if (UtilValidate.isEmpty(obj)) {
                     disabled = true;
                     break;
                 }