[richfaces-issues] [JBoss JIRA] Updated: (RF-10192) rich:dropDownMenu: Enum in renderer vs. String in Component API - EL coercion problem
Nick Belaevski (JIRA)
jira-events at lists.jboss.org
Wed Jan 12 18:58:49 EST 2011
[ https://issues.jboss.org/browse/RF-10192?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Nick Belaevski updated RF-10192:
--------------------------------
Assignee: Nick Belaevski
Fix Version/s: 4.0.0.CR1
> rich:dropDownMenu: Enum in renderer vs. String in Component API - EL coercion problem
> -------------------------------------------------------------------------------------
>
> Key: RF-10192
> URL: https://issues.jboss.org/browse/RF-10192
> Project: RichFaces
> Issue Type: Bug
> Security Level: Public(Everyone can see)
> Components: component-menu
> Affects Versions: 4.0.0.Milestone5
> Environment: RichFaces 4 M5
> Reporter: Martin Kočí
> Assignee: Nick Belaevski
> Fix For: 4.0.0.CR1
>
>
> <r:dropDownMenu mode="#{beanNull.mode}" >
> java.lang.IllegalArgumentException: No enum const class org.richfaces.component.Mode.
> at java.lang.Enum.valueOf(Enum.java:196)
> at org.richfaces.component.Mode.valueOf(Mode.java:24)
> at org.richfaces.renderkit.html.MenuItemRendererBase.resolveSubmitMode(MenuItemRendererBase.java:120)
> Reason:
> MenuItemRendererBase calls Mode.valueOf(abstractDropDownMenu.getMode()); but AbstractDropDownMenu.getMode() returns String - then if #{} resolves to null, EL will coerce null to "" ([1]) and returns it from getMode() method.
> Solutions:
> 1) use quick fix like:
> String mode = parent.getMode();
> if (StringUtils.isEmpty(mode)) {
> return Mode.server;
> }
> 2) Disable this (stupid) coercion rule (workaround):
> JUEL: http://juel.sourceforge.net/guide/advanced/index.html
> TOMCAT: http://tomcat.apache.org/tomcat-6.0-doc/config/systemprops.html - COERCE_*
> 3) Use org.richfaces.component.Mode in AbstractDropDownMenu API:
> EL 2.2 spec 1.18.6 Coerce A to an Enum Type T: "If A is null, return null" - for Enum here is no coercion problem.
> I would prefer solution 3) if possibible, because we use Java API directly (menu.setMode(Mode.Server) is nicer and safer than a menu.setMode(Mode.server.name())
> [1]
> https://jsp-spec-public.dev.java.net/issues/show_bug.cgi?id=183
> https://uel.dev.java.net/issues/show_bug.cgi?id=12
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the richfaces-issues
mailing list