myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthias Wessendorf" <mat...@apache.org>
Subject Re: TRINIDAD-1262
Date Tue, 21 Oct 2008 20:11:07 GMT
looks good so far.

I just wonder if this:
+      Locale locale = getLocale();
+      if (null == locale)
+      {
+        RequestContext reqContext = RequestContext.getCurrentInstance();
+        if (reqContext != null)
+        {
+          locale = reqContext.getFormattingLocale();
+        }
+        if (locale == null)
+        {
+           locale = context.getViewRoot().getLocale();
+        }
+      }

could be moved to a private function, since some other place is doing
the same, I think

-M

On Tue, Oct 21, 2008 at 9:46 PM, Anita Anandan <anita.anandan@oracle.com> wrote:
> Hi all -
>
> I was wondering if one of the Trinidad committees could take a look at
> trinidad-1262 (https://issues.apache.org/jira/browse/TRINIDAD-1262)? It
> needs to be reviewed and committed.
>
> Thanks
> Anita
>



-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

Mime
View raw message