On Tue, 2005-11-29 at 01:59 +0000, Niall Pemberton wrote:
> On 11/28/05, Christian Meder <[EMAIL PROTECTED]> wrote:
> > On Mon, 2005-11-28 at 21:43 +0100, Christian Meder wrote:
> > > On Mon, 2005-11-28 at 03:00 +0000, Niall Pemberton wrote:
> > > > On 11/28/05, Craig McClanahan <[EMAIL PROTECTED]> wrote:
> > > > > On 11/27/05, Niall Pemberton <[EMAIL PROTECTED]> wrote:
> > > > > >
> > > > > > A couple of questions I had on the resources API:
> > > > > >
> > > > > > 1) I was wondering why TimeZone is include in the Resources's
> > > > > > content
> > > > > > read methods? Is there a use case, for example, for giving west
> > > > > > coast
> > > > > > Americans a different message to east coast?
> > > > >
> > > > >
> > > > > Some of the format options for displaying times take a time zone
> > > > > parameter
> > > > > (so that it can be included in the emitted string). Simply pass null
> > > > > if it
> > > > > doesn't matter to you.
> > > >
> > > > I don't see any valid formats specified in java.text.MessageFormat
> > > > that take a TimeZone. Thats slightly irrelevant though since AFAIK
> > > > Commons Resources doesn't deal anywhere with message formats. My
> > > > understanding that happends after the resource has been retrieved,
> > > > something like....
> > > >
> > > > String msgResource = myResources.getString("foo.key", locale, null);
> > > > MessageFormat msgFormat = new MessageFormat(msgResource);
> > > > msgFormat.setLocale(locale);
> > > > String message = msgFormat.format(args);
> > > >
> > > > Have I got the wrong end of the stick on this? Struts MessageResources
> > > > does do the MessageFormat bit, but Commons Resources doesn't.
> > >
> > > Actually I just nuked my reply (still stuck on my laptop) which
> > > reiterated my concern from my mail a couple of days that
> > > MessageResources and the MessageFormat stuff is missing like you did.
> > > But I guess I found it: it's in Messages.
> > >
> > > So I guess it should be more along the line:
> > >
> > > Messages myMsgResources = new Messages(myResources);
> > > String message = myMsgResources.getMessage(locale, "foo.key", args);
> > >
> > > But there are some things which should get improved wrt Messages:
> > >
> > > * include all parametric replacement options like in Struts
> > > MessageResources (not only arg0 and [] args)
> >
> > Following up to myself:
> >
> > I just found the following in the log for Messages:
> >
> >
> > r142735 | dgraham | 2003-10-02 02:24:13 +0200 (Thu, 02 Oct 2003) | 8
> > lines
> > Changed paths:
> >
> > M
> > /jakarta/commons/proper/resources/trunk/src/java/org/apache/commons/resources/Messages.java
> >
> > M
> > /jakarta/commons/proper/resources/trunk/src/test/org/apache/commons/resources/MessagesTestCase.java
> >
> > Removed Messages methods that provided 2-4 replacement
> > arguments that were carried over from Struts. This makes
> > the interface easier to understand and the implementation
> > simpler. The methods that are left provide for 0, 1, or an
> > array of replacement arguments.
> >
> > I happen to disagree with the reasoning but I wanted to put this up for
> > discussion.
>
> Sorry I didn't respond to your earlier post, don't seem to have enough
> hours in the day.
You surely seem to have more hours in the day than me ;-)
> My view is the simpler the better, especially as Messages can be
> easily extended to include these additional methods.
Ok. So this part of the API will stay the same.
> > > * the order of locale and key is reversed compared to Resources, it's
> > > (locale, key); Resources is (key, locale)
>
> Good point, I agree it should be consistent.
Ok. Patch submitted. If you include the patch I will correct the user
guide in the wiki as well.
> > > * MessageFormat isn't cached as it is in Struts MessageResources
>
> The problem with formats is that they are not thread safe, so caching
> them means you need to synchronize the cache. I don't know whether
> caching provides any benefit because of that. Also if we set the
> Locale as you suggest below, then we would have to cache formats by
> locale.
I've got no strong opinions on the caching. Struts includes it and
there's a TODO in Messages. Did anybody produce performance numbers wrt
the Struts formats cache ?
>
> > > * setLocale isn't called on the used MessageFormat instances
>
> I agree we should do this.
Ok. Patch submitted.
>
> > > * the TimeZone parameter isn't used like in the rest of
> > > commons-resources
>
> I'm don't know how it could be - and was one of my original questions.
Looking at MessageFormat I don't see an interesting TimeZone use case
either. Perhaps Craig can explain which use cases he had specifically in
mind when added the TimeZone parameter ?
>
> > > If nobody objects I'll whip up patches for the above points.
>
> Dam, I had just got resources down to zero open bugs ;-)
Always glad to help ;-)
Christian
--
Christian Meder, email: [EMAIL PROTECTED]
The Way-Seeking Mind of a tenzo is actualized
by rolling up your sleeves.
(Eihei Dogen Zenji)
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
