svn commit: r1052195 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java

[lektran]

Author: lektran
Date: Thu Dec 23 08:05:33 2010
New Revision: 1052195

URL: http://svn.apache.org/viewvc?rev=1052195&view=rev
Log:
OFBIZ-4071 - menu-item's disable-if-empty attribute was not being handled, 
fixed by using the existing unused method to check if the item should be 
disabled and if true then applying the disabled style and not rendering the 
hyperlink portion of the child link (if present).  Based on a slightly modified 
patch and report from Deepak Dixit, thanks Deepak.

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java

Modified: 
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java?rev=1052195&r1=1052194&r2=1052195&view=diff
==============================================================================
--- 
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java 
(original)
+++ 
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/html/HtmlMenuRenderer.java 
Thu Dec 23 08:05:33 2010
@@ -158,7 +158,7 @@ public class HtmlMenuRenderer extends Ht
             }
         }
 
-        if (menuItem.getDisabled()) {
+        if (menuItem.getDisabled() || this.isDisableIfEmpty(menuItem, 
context)) {
             style = menuItem.getDisabledTitleStyle();
         }
 
@@ -389,7 +389,7 @@ public class HtmlMenuRenderer extends Ht
     public void renderLink(Appendable writer, Map<String, Object> context, 
ModelMenuItem.Link link) throws IOException {
         String target = link.getTarget(context);
         ModelMenuItem menuItem = link.getLinkMenuItem();
-        if (menuItem.getDisabled()) {
+        if (menuItem.getDisabled() || isDisableIfEmpty(menuItem, context)) {
             target = null;
         }