Re: [S1] Validator does not respect locale when validating double value?

2013-01-20 Thread Lukasz Lenart
2013/1/19 Christopher Schultz ch...@christopherschultz.net: So, is it worth filing a bug and attaching a patch, or should I just write my own validator and use that for my own uses? Fill a bug and attach patch and write your own custom validator as well :-) Regards -- Łukasz + 48 606 323 122

Re: [S1] Validator does not respect locale when validating double value?

2013-01-19 Thread Christopher Schultz
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Łukasz, On 1/18/13 4:38 PM, Lukasz Lenart wrote: 2013/1/18 Christopher Schultz ch...@christopherschultz.net: Honestly, I'm shocked that struts 1, which is at least 10 years old, still has a glaring internationalization bug like this. It

[S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Christopher Schultz
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 All, I'm running Struts 1.3.10 (with commons-validator 1.3.1), and I'm trying to validate (and subsequently parse) a floating-point value as a double. My validator configuration looks like this (I apologize for it's potential unreadability):

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Christopher Schultz
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 All, On 1/18/13 1:02 PM, Christopher Schultz wrote: Before I go digging-through the code Struts/commons-validator to find out exactly what might be wrong, can anyone give me any suggestions at to what I might be missing? Actually, it didn't

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Lukasz Lenart
2013/1/18 Christopher Schultz ch...@christopherschultz.net: The method validateDouble() totally ignores the user's Locale and calls commons-validator's formatDouble(String) method instead of the formatDouble(String,Locale) method. The same seems to be true for all the validate[NumberType]

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Christopher Schultz
Łucaz, On Jan 18, 2013, at 16:00, Lukasz Lenart lukaszlen...@apache.org wrote: 2013/1/18 Christopher Schultz ch...@christopherschultz.net: The method validateDouble() totally ignores the user's Locale and calls commons-validator's formatDouble(String) method instead of the

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Lukasz Lenart
2013/1/18 Christopher Schultz ch...@christopherschultz.net: Honestly, I'm shocked that struts 1, which is at least 10 years old, still has a glaring internationalization bug like this. It isn't actively developed any more, that's the problem :-) Regards -- Łukasz + 48 606 323 122

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Christopher Schultz
-chris -- Christopher Schultz Chief Technology Officer, Total Child Health Inc. Technical Director, Center for Promotion of Child Development through Primary Care tel: +1.410.807.4500 x20 tel: +1.888.4CHADIS (+1.888.424.2347) email: cschu...@chadis.com On Jan 18, 2013, at 16:38, Lukasz

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Christopher Schultz
-chris -- Christopher Schultz Chief Technology Officer, Total Child Health Inc. Technical Director, Center for Promotion of Child Development through Primary Care tel: +1.410.807.4500 x20 tel: +1.888.4CHADIS (+1.888.424.2347) email: cschu...@chadis.com On Jan 18, 2013, at 16:38, Lukasz

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Christopher Schultz
Łucaz, On Jan 18, 2013, at 16:38, Lukasz Lenart lukaszlen...@apache.org wrote: 2013/1/18 Christopher Schultz ch...@christopherschultz.net: Honestly, I'm shocked that struts 1, which is at least 10 years old, still has a glaring internationalization bug like this. It isn't actively

Re: [S1] Validator does not respect locale when validating double value?

2013-01-18 Thread Lukasz Lenart
2013/1/18 Christopher Schultz ch...@christopherschultz.net: I've got an old web app that has used S1 for years, so that's why I'm using it. Don't worry, I'm not trying to launch a brand-new project using struts one. :D Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/