[
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)
