[OPEN-ILS-DEV] Re: [open-ils-commits] r7663 - trunk/Open-ILS/xul/staff_client/server/locale/en-US

Dan Scott denials at gmail.com
Wed Aug 15 08:11:09 EDT 2007


On 14/08/07, Mike Rylander <mrylander at gmail.com> wrote:
> On 8/14/07, Jason Etheridge <phasefx at gmail.com> wrote:
> > > http://developer.mozilla.org/en/docs/XUL_Tutorial:Property_Files
> > > clearly shows upper-case $S being used, and that's what I've used in
> > > previous commits (and testing with the built-in chrome
> > > stringbundleset/stringbundle interface showed that it worked). Are you
> > > sure you want to use lower-case here?
> >
> > Hrm, you're right.  I just did that because the sprintf implementation
> > we were using didn't work with %S, thinking it was the same standard.
> > Maybe we should make it case-insensitive in this case?  Should we
> > disallow extra sprintf-isms and just support the Mozilla syntax?
> >
>
> We're going to be using messagecatalog for this, right?  If so, let's
> please use sprintf syntax (this, again, is why I didn't want the
> string "stringbundle" involved in our messagecatalog stuff)...

Yes, we're using messagecatalog for this. I thought messagecatalog was
going to maintain compatibility with the Mozilla syntax, but as long
as we're willing to document the messagecatalog way of doing things
and accept whatever slight pain we incur from doing things slightly
differently from the XUL standard, we can do whatever works (and in
the end, working code rules!). And I'm willing to document that --
just give me some time :)

Sorry for sounding the alarm, Jason, but it sounds like we clarified a
slightly fuzzy i18n area so hopefully it was worth it. When I get some
more time (in a day or two) I'll fix up the property files I had
submitted previously.

-- 
Dan Scott
Laurentian University


More information about the Open-ils-dev mailing list