Re: [tools] Adding localization method to DateTool

2009-11-15 Thread Christopher Schultz
Nathan, Resurrecting this because I'm ready to commit. On 10/29/2009 3:04 PM, Nathan Bubna wrote: On Thu, Oct 29, 2009 at 8:06 AM, Christopher Schultz ch...@christopherschultz.net wrote: Nathan, On 10/29/2009 11:02 AM, Christopher Schultz wrote: I like re-coding into SimpleDataFormat

Re: [tools] Adding localization method to DateTool

2009-11-15 Thread Nathan Bubna
On Sun, Nov 15, 2009 at 7:26 AM, Christopher Schultz ch...@christopherschultz.net wrote: Nathan, Resurrecting this because I'm ready to commit. On 10/29/2009 3:04 PM, Nathan Bubna wrote: On Thu, Oct 29, 2009 at 8:06 AM, Christopher Schultz ch...@christopherschultz.net wrote: Nathan, On

Re: [tools] Adding localization method to DateTool

2009-10-29 Thread Antonio Petrelli
2009/10/28 Nathan Bubna nbu...@gmail.com: One question, would it make sense to also add a: public String toLocalizedPattern(String format) {     return toLocalizedPattern(format, getLocale() } for completeness?  Or is that redundant? IMHO this method would be useful too, as long as

Re: [tools] Adding localization method to DateTool

2009-10-29 Thread Tomas Wredendal
2009/10/29 Antonio Petrelli antonio.petre...@gmail.com: 2009/10/29 Claude Brisson cla...@renegat.net: On jeu., 2009-10-29 at 09:22 +0100, Antonio Petrelli wrote: 2009/10/28 Nathan Bubna nbu...@gmail.com: One question, would it make sense to also add a: public String

Re: [tools] Adding localization method to DateTool

2009-10-29 Thread Antonio Petrelli
2009/10/29 Claude Brisson cla...@renegat.net: On jeu., 2009-10-29 at 11:02 +0100, Antonio Petrelli wrote: 2009/10/29 Claude Brisson cla...@renegat.net: On jeu., 2009-10-29 at 09:22 +0100, Antonio Petrelli wrote: 2009/10/28 Nathan Bubna nbu...@gmail.com: One question, would it make sense

Re: [tools] Adding localization method to DateTool

2009-10-29 Thread Christopher Schultz
Nathan, On 10/28/2009 6:32 PM, Nathan Bubna wrote: You've not been forgotten. :) You're correct that this is not currently supported. This only adds a simple method to a single tool's VTL API and thus would not impede any forthcoming long-wished for 2.0 release (at least in my book). Go

Re: [tools] Adding localization method to DateTool

2009-10-29 Thread Christopher Schultz
All, On 10/28/2009 5:10 PM, Christopher Schultz wrote: public String toLocalizedPattern(String dateFormat, Locale targetLocale) { new SimpleDateFormat(formatPattern, targetLocale).toLocalizedPattern(); } After looking at DateTool, I'm thinking

Re: [tools] Adding localization method to DateTool

2009-10-29 Thread Christopher Schultz
Nathan, On 10/29/2009 10:36 AM, Nathan Bubna wrote: return df.toLocalizedPattern(); I believe you'll have to cast to (SimpleDateFormat) to get the toLocalizedPattern() method. Yup, I noticed that as I started to write the code. For now, I just have it allowing a ClassCastException. I

[tools] Adding localization method to DateTool

2009-10-28 Thread Christopher Schultz
All, I'm a big fan of DateTool and use it all the time in my Velocity templates. I recently had a revelation that, when providing hints to my users like please format your date like this: MM/dd/, the date format itself is being read from the user's preferences and always in the