dev@glassfish.java.net

Re: [NEW] Seeking Review: Small EL Impl changes

From: Ed Burns <ed.burns_at_sun.com>
Date: Thu, 11 Aug 2005 17:07:46 -0700

Ken Paulsen wrote:

KP> You may want to consider simplifying the API by having "ELException"
KP> utilize the ELUtil class. Then you could just pass in the key:
KP> "setPropertyFailed" and the object[].
KP>
KP> Also, I would think developers may want to customize these messages.
KP> Do you have a mechanism for doing this? Perhaps a ResourceBundleFactory?

>>>>> On Thu, 11 Aug 2005 18:24:24 -0500, Jacob Hookom <jacob_at_hookom.net> said:

JH> I would keep it simple, ELException is public so unless you make ELUtil
JH> public too, I wouldn't extend that dependency. And I don't recommend
JH> making ELUtil public.

Yes, ELUtil was intended to be an impl private class. I definately
don't want to make it public. I'd leave the message customizing to the
layer on top of EL, in this case JSP or JSF.

Ed

-- 
| ed.burns_at_sun.com  | {home: 407 869 9587, office: 408 884 9519 OR x31640}
| homepage:         | http://purl.oclc.org/NET/edburns/
| aim: edburns0sunw | iim: ed.burns_at_sun.com