DefaultMenuItem: EL is not calculated in setStyleClass method

1) Environment

• PrimeFaces version: 7.0.RC3
• Does it work on the newest released PrimeFaces version? Version? No (7.0.RC3)
• Does it work on the newest sources in GitHub? (Build by source -> https://github.com/primefaces/primefaces/wiki/Building-From-Source) not tested
• Application server + version: Wildfly 7.0.0.Final
• Affected browsers: Google Chrome

2) Expected behavior

EL expression assigned to the item by method DefaultMenuItem.setStyleClass() is correctly interpreted and produce the required behavior of dynamic menu component.

3) Actual behavior

EL expression assigned to the item by method DefaultMenuItem.setStyleClass() treated as a simple string and assigned to final HTML item class as it was passed into the method. Example of how it’s looks a tabindex="-1" class="ui-menuitem-link ui-corner-all #{"1".equals(categories.category) ? "ui-state-active" : ""}">

4) Steps to reproduce

Create a managed bean and XHTML pattern as explained below and run

5) Sample XHTML

<p:panelMenu id="id-categories" model="#{categories.model}" expanded="true"/>

6) Sample bean

@ManagedBean(name = "categories")
@SessionScoped
public class CategoriesBean {

    private static String CATEGORY_ID ="categoryId";

    private DynamicMenuModel model;

    @Inject
    private CategoryRepository categoryRepository;

    private Category category;

    public Category getCategory() {
        return category;
    }

    public void setCategory(Category category) {
        this.category = category;
    }

    public Collection<Category> getAll() {
        return categoryRepository.getAll();
    }

    @PostConstruct
    private void init() {
        model = new DynamicMenuModel();

        DefaultSubMenu categoriesSubmenu = new DefaultSubMenu("Categories");
        categoriesSubmenu.setExpanded(true);

        for (Category category : categoryRepository.getAll()) {
            categoriesSubmenu.addElement(createMenuItem(String.valueOf(category.getId()), category.getName()));
        }
        model.addElement(categoriesSubmenu);

        model.generateUniqueIds();
    }

    public MenuModel getModel() {
        return model;
    }

    public String actionSelectCategory(ActionEvent event) {
        MenuItem menuItem = ((MenuActionEvent) event).getMenuItem();
        String id = menuItem.getParams().get(CATEGORY_ID).get(0);

        this.category = categoryRepository.getById(Long.valueOf(id));
        return "/index.xhtml?faces-redirect=true";
    }

    private MenuItem createMenuItem(String id, String name) {
        DefaultMenuItem item = new DefaultMenuItem(name);
        item.setId(id);
        item.setCommand("#{categories.actionSelectCategory}");
        item.setParam(CATEGORY_ID, id);
        item.setStyleClass("#{\"" + id + "\".equals(categories.category) ? \"ui-state-active\" : \"\"}");
        return item;
    }
}

[tandraschko]

As you are the first one since years, i suggest you to use the following workaround:

class MyDefaultMenuItem extends DefaultMenuItem {    
    @Override    
    public String getStyleClass() {    
        FacesContext facesContext = FacesContext.getCurrentInstance();    
        return facesContext.getApplication().evaluateExpressionGet(facesContext, super.getStyleClass(), String.class);    
    }    
}    

[ansurakin]

Thanks! My be need create property like: boolean evaluateExpression