[cdi-dev] [JBoss JIRA] (CDI-513) Clarify whether passivating pseudo-scopes are valid

Mark Struberg (JIRA) issues at jboss.org
Tue Mar 3 10:40:49 EST 2015


    [ https://issues.jboss.org/browse/CDI-513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13045687#comment-13045687 ] 

Mark Struberg commented on CDI-513:
-----------------------------------

{quote}
Passivating scopes must be explicitly declared @NormalScope(passivating=true).
{quote}

This would effectively render the whole addScope Extension method useless...

How would you add a scope for the JSF @ViewScoped annotation in that case? (This annotation is neither @NormalScoped nor @Scope).

I suggest we remove this whole sentence as it contradicts other spec parts.


> Clarify whether passivating pseudo-scopes are valid
> ---------------------------------------------------
>
>                 Key: CDI-513
>                 URL: https://issues.jboss.org/browse/CDI-513
>             Project: CDI Specification Issues
>          Issue Type: Clarification
>    Affects Versions: 1.2.Final
>            Reporter: Mark Struberg
>
> On behalf of Jozef I copied this to a CDI ticket... See CDITCK-466
> I personally think it is clear as there is no single word which forbids this and there is a very vocal description about the single flags.
> -----
> AddingPassivatingScopeTest is illegal as addScope for passivating non-normalscopes is perfectly fine.
> There is nothing in the spec which says that a non-normalscope cannot be passivating.
> The practical use case for this is e.g. when bridging over to Spring. Those beans might need to get checked for Serializable BUT spring brings it's own proxies. So we do not need to wrap it into just another normalscoping proxy.
> Actually the test should come in 2 flavours:
> 1.) RomanEmpire being Serializable -> all fine must work
> 2.) RomainEmpire not Serializable -> DefinitionException



--
This message was sent by Atlassian JIRA
(v6.3.11#6341)


More information about the cdi-dev mailing list