[ 
https://issues.apache.org/jira/browse/TAPESTRY-1997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12588873#action_12588873
 ] 

Geoff Callender commented on TAPESTRY-1997:
-------------------------------------------

I've just confirmed that country codes should not be lower case.  From the 
javadoc for Locale (http://java.sun.com/j2se/1.5.0/docs/api/index.html) 
regarding the constructors for Locale:

"The country argument is a valid ISO Country Code. These codes are the 
upper-case, two-letter codes as defined by ISO-3166. You can find a full list 
of these codes at a number of sites, such as: 
http://www.iso.ch/iso/en/prods-services/iso3166ma/02iso-3166-code-lists/list-en1.html";



> PersistentLocale is lower-casing locales
> ----------------------------------------
>
>                 Key: TAPESTRY-1997
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-1997
>             Project: Tapestry
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.0.6
>            Reporter: Geoff Callender
>
> An issue affecting localization: PersistentLocale is converting locales from 
> mixed case to all lower case, which is useless for formatting.  For example, 
> if page 1 sets the locale like this:
>       @Inject
>       private PersistentLocale _persistentLocaleService;
>       Locale locale = Locale.UK;
>       _persistentLocaleService.set(locale);
>       System.out.println("locale is " + locale + " - " + 
> locale.getDisplayName());
>  
> then this is what prints:
>       locale is en_GB - English (United Kingdom)
> But when I'm in page 2 I get the locale and find it has mutated...
>       Locale locale = _persistentLocaleService.get();
>       System.out.println("locale is " + locale + " - " + 
> locale.getDisplayName());
> ...this is what prints:
>       locale is en_gb - en_gb
> This mutated locale in page 2 is useless for formatting.  Code like the 
> following produces default-styling instead of the styling for en_GB:
>       _myDateFormat = DateFormat.getDateInstance(DateFormat.LONG, locale);
>       System.out.println(_myDateFormat.format(new Date()));
> It seems this has also adversely affected how supported-locales are declared 
> (maybe in previous releases only).  See 
> http://thread.gmane.org/gmane.comp.java.tapestry.user/56526/focus=56527

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to