Re: Seeking Review: 21-MethodBinding - Partial

From: Adam Winer <>
Date: Thu, 11 Nov 2004 08:46:54 -0800

Ed Burns wrote:
>>>>>>On Thu, 11 Nov 2004 10:04:37 -0500, Roger Kitain <Roger.Kitain_at_Sun.COM> said:
> RK> Please review the change bundle at:
> RK>
> RK> Note: This is only a partial checkin for ConverterTag/ValidatorTag in api.
> RK> The rest of the checkin will be done for some core tags (RI) and TLV
> RK> modifications.
> 1. New Messages
> First, I would have preferred to have done the new messages, some of
> which appear to have nothing to do with 21-MethodBindings, in a separate
> checkin. You can leave it as is now, but in the future, I like to keep
> the checkins as tightly focused and cohesive as possible.
> Second, all of these must be added to section in the spec PDF
> document.
> + public static final String INVALID_EXPRESSION_MESSAGE_ID =
> "javax.faces.webapp.INVALID_EXPRESSION";
> Third, this should be javax.el.INVALID_EXPRESSION. We can do it this
> way now since it's only a message key, and a new one at that.
> + public static final String COMPONENT_FROM_TAG_ERROR_MESSAGE_ID =
> "javax.faces.webapp.COMPONENT_FROM_TAG_ERROR";
> + public static final String NOT_NESTED_IN_TYPE_TAG_ERROR_MESSAGE_ID =
> "javax.faces.webapp.NOT_NESTED_IN_TYPE_TAG_ERROR";
> + public static final String NOT_NESTED_IN_FACES_TAG_ERROR_MESSAGE_ID =
> "javax.faces.webapp.NOT_NESTED_IN_FACES_TAG_ERROR";
> + public static final String CANT_CREATE_CLASS_ID =
> "javax.faces.webapp.CANT_CREATE_CLASS";

Why are we adding "javax.faces" messages here? So far,
"javax.faces" messages have been reserved for user-presented messages
(e.g., validation errors), and "com.sun.faces" messages for internal

I see little reason to change this; we can make all of these
above messages "com.sun.faces" and keep them out of the spec.
We certainly can't add "javax.faces" messages without any EG

-- Adam

To unsubscribe, e-mail:
For additional commands, e-mail: