[ https://issues.apache.org/jira/browse/MYFACES-3413?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mike Kienenberger resolved MYFACES-3413. ---------------------------------------- Resolution: Fixed Fix Version/s: 2.1.5-SNAPSHOT Leave value as empty string if no ExpressionString. > Default MyFaces Error handling throws NullPointerException during component > tree when javax.el.Expression.getExpressionString() is null > --------------------------------------------------------------------------------------------------------------------------------------- > > Key: MYFACES-3413 > URL: https://issues.apache.org/jira/browse/MYFACES-3413 > Project: MyFaces Core > Issue Type: Bug > Components: JSR-314 > Affects Versions: 2.1.3 > Reporter: Mike Kienenberger > Assignee: Mike Kienenberger > Priority: Minor > Fix For: 2.1.5-SNAPSHOT > > > The default MyFaces Error handling throws a NullPointerException during the > writing out of the component tree when an attribute's > ValueExpression.getExpressionString() returns null. > As an example, this can happen with Richfaces 4.1.x skinning as the > org.richfaces.application.GlobalResourcesViewHandler$SkinningResourceRenderedExpression > used for "rendered" on the __rf_skinning_resource UIOutput has a null value > for getExpressionString(). > From what I can tell, it's legal for Expression.getExpressionString() to > return null. > http://docs.oracle.com/javaee/5/api/javax/el/Expression.html#getExpressionString%28%29 > In any case, since this is an error handling routine, it should > probably be more flexible in handling unexpected values. > What do we want to do in this case? > We could skip outputting the attribute. > We could output the attribute with empty string as the value expression. > In this particular case, we could check for isReadOnly() and if true, output > getValue(). Not sure if we'd want to do that in the case of !isReadOnly(). > In fact, I'm not entirely certain we want to be evaluating getValue(). > We could output the class of the value expression. > As a first pass, I am going to output the value expression as the empty > string. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira