On Thu, 27 Nov 2003, David Graham wrote:
> When changing Struts' messaging classes we also need to update
> commons-resources with identical changes. Otherwise, we'll end up getting
> frustrated users and duplicate bug reports when Struts starts using
> commons-resources.
Sure. In this particular case, I don't believe the issue arises in Commons
Resources, because it's structured differently. (Feel free to let me know
if that's not the case, though.) The problem here was that one particular
method was bypassing the locale defaulting mechanism, while all the rest
were picking it up correctly.
--
Martin Cooper
>
> David
>
>
> --- [EMAIL PROTECTED] wrote:
> > martinc 2003/11/27 14:14:51
> >
> > Modified: src/share/org/apache/struts/util MessageResources.java
> > Log:
> > Make sure getMessage(String) goes through the local implementation,
> > rather
> > than bypassing it and therefore skipping the default locale handling.
> >
> > PR: 22252
> > Submitted by: msc at ivu.de
> >
> > Revision Changes Path
> > 1.20 +5 -5
> > jakarta-struts/src/share/org/apache/struts/util/MessageResources.java
> >
> > Index: MessageResources.java
> > ===================================================================
> > RCS file:
> >
> /home/cvs/jakarta-struts/src/share/org/apache/struts/util/MessageResources.java,v
> > retrieving revision 1.19
> > retrieving revision 1.20
> > diff -u -r1.19 -r1.20
> > --- MessageResources.java 9 Sep 2003 04:33:12 -0000 1.19
> > +++ MessageResources.java 27 Nov 2003 22:14:51 -0000 1.20
> > @@ -204,7 +204,7 @@
> > */
> > public String getMessage(String key) {
> >
> > - return this.getMessage((Locale) null, key);
> > + return this.getMessage((Locale) null, key, null);
> >
> > }
> >
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
>
>
> __________________________________
> Do you Yahoo!?
> Free Pop-Up Blocker - Get it now
> http://companion.yahoo.com/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
