[ https://issues.apache.org/jira/browse/MYFACES-3786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13794207#comment-13794207 ]
Gerhard Petracek commented on MYFACES-3786: ------------------------------------------- @leo: the change itself works. you would be right if the key would be e.g. the class (because 2+ request scoped instances of the same class would lead to the same key in an application scoped map). however, the key is the identity-hash-code which is different. the only issue is: >if< we will have to support other scopes in the future, identity-hash-code might not be strong enough + we >could< get a potential leak (if the cleanup code wouldn't work properly). however, i thought about it already and i had the same (spi-)change in mind. i didn't do it, because i don't like that we would expose a detail of one implementation. -> i planned to discuss such a change after the upcoming beta release. > Web Container injection support should be provided for additional lifecycle > artifacts (not just managed beans) > -------------------------------------------------------------------------------------------------------------- > > Key: MYFACES-3786 > URL: https://issues.apache.org/jira/browse/MYFACES-3786 > Project: MyFaces Core > Issue Type: Task > Components: JSR-344 > Reporter: Leonardo Uribe > Assignee: Leonardo Uribe > Fix For: 2.2.0 > > Attachments: cdiELresolverWeb.zip, cdiELResolver.zip, > cdiPartialViewContext.war, cdiPartialViewContext.zip, cdi.patch, > cdiphaselistener1.patch, cdiphaselistener2.patch, cdirevised.patch, > cdiValidatorSource.zip, cdiValidator.war, MYFACES-3786-1.patch, > MYFACES-3786-2.patch > > > This issue is all about how to inject beans into jsf artifacts. > See JSF 2.2 section 5.4.1 > The problem here is in some point we need to give the control to the > underlying environment to inject beans into the artifacts, but we don't know > much about how to properly do it, so we need to try with examples. -- This message was sent by Atlassian JIRA (v6.1#6144)