Mark Collette wrote:
> In the method
> javax.faces.component.MessageFactory.getLabel(FacesContext,UIComponent),
> at the line where o = component.getValueExpression("label"), should
> getLabel(-) be returning the evaluation of the ValueExpression, and
> not the ValueExpression itself?
>
>
> |*
> <http://java.sun.com/javaee/5/docs/api/javax/el/ValueExpression.html#getValue%28javax.el.ELContext%29>*|
> static Object getLabel(FacesContext context,
> UIComponent component) {
> Object o =
> component.getAttributes().get("label");
> if (o == null || (o instanceof String && ((String) o).length() ==
> 0)) {
> o = component.getValueExpression("label");
> <<<<<<<<<<<<<<<<
> }
> // Use the "clientId" if there was no label specified.
> if (o == null) {
> o = component.getClientId(context);
> }
> return o;
> }
>
Offhand, that branch shouldn't be needed.
component.getAttributes().get("label"); should result
in the VE being evaluated if present.
Please feel free to log an issue and we'll get it cleaned up.
>
> - Mark Collette
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_javaserverfaces.dev.java.net
> For additional commands, e-mail: dev-help_at_javaserverfaces.dev.java.net
>