[ 
https://jira.jboss.org/browse/SEAMINTL-5?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12546334#action_12546334
 ] 

John Ament commented on SEAMINTL-5:
-----------------------------------

So I can add an injection point on defaultTimeZoneId, in 
DefaultTimeZoneProducer.
e.g.:
   @Inject @Named
   private String defaultTimeZoneId;

This passes a basic UT like this:

   @Produces @Named("defaultTimeZoneId")
   private String utTestDefaultTimeZoneId = "Africa/Abidjan";

   @Test
   public void testDefaultTimeZoneProducerID()
    {
       Assert.assertEquals(timeZone.getID(), utTestDefaultTimeZoneId);
   }

However, the problem comes into whether defaultTimeZoneID should be a required 
configuration.  We would need to change the xml config to use producer style in 
this case.  In addition, it makes the field a required step, so you need to 
either provide a Java based or XML configuration for the point, you can't skip 
it.  Personally, I don't see an issue with it, but I don't think it's my call. 
;-)

> Abillity to initialise locale and timezone for Application through Java, in 
> addition to existing XML Config
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: SEAMINTL-5
>                 URL: https://jira.jboss.org/browse/SEAMINTL-5
>             Project: Seam i18n
>          Issue Type: Feature Request
>          Components: Locales, Time zones
>    Affects Versions: 3.0.0.Alpha1
>            Reporter: Ken Finnigan
>             Fix For: 3.0.0.Alpha2
>
>
> Add initialisation of Application Locale and TimeZone through Events.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        
_______________________________________________
seam-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/seam-issues

Reply via email to