[weld-issues] [JBoss JIRA] (WELD-949) Move firing of ContainerInitialized event from StartMain to the end of Weld.initialize()

Ales Justin (Resolved) (JIRA) jira-events at lists.jboss.org
Thu Oct 27 11:02:45 EDT 2011


     [ https://issues.jboss.org/browse/WELD-949?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ales Justin resolved WELD-949.
------------------------------

    Resolution: Done

    
> Move firing of ContainerInitialized event from StartMain to the end of Weld.initialize()
> ----------------------------------------------------------------------------------------
>
>                 Key: WELD-949
>                 URL: https://issues.jboss.org/browse/WELD-949
>             Project: Weld
>          Issue Type: Feature Request
>          Components: Java SE Support
>    Affects Versions: 1.1.1.Final
>            Reporter: Ondrej Zizka
>            Assignee: Ales Justin
>            Priority: Optional
>             Fix For: 1.2.0.Beta1
>
>
> I suggest to move firing of ContainerInitialized event from StartMain to the end of Weld.initialize().
> Rationale:
>  * Having this done by Weld.initialize() is more expectable (IMO) than having it restricted to a concrete class.
>  * With the event fired from the end of initialize(), users who bootstrap Weld on their own would not need to introduce their own "startup", or worse, fire ContainerInitialized on their own.
>  * Backward compatible - currently only StartMain() fires the event. In case of this change, firing would be removed from StartMain(), so the current behavior would not change. The only case when it would do harm is if user fired ContainerInitialized in their own code. Which they should not - such usage smells :)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


More information about the weld-issues mailing list