[cdi-dev] [JBoss JIRA] (CDI-331) Instance.iterator() shouldn't include disabled alternatives

[Pete Muir (JIRA)]

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

Pete Muir updated CDI-331:
--------------------------

          Summary: Instance.iterator() shouldn't include disabled alternatives  (was: Clarify Instance.iterator())
       Issue Type: Feature Request  (was: Clarification)
    Fix Version/s: TBD
                       (was: 1.1.FD)
      Description: 
The specification currently says that:

{quote}
The iterator() method must:
• Identify the set of beans that have the required type and required qualifiers and are eligible for injection into the class into
which the parent Instance was injected, according to the rules of typesafe resolution, as defined in Section 5.2.1.
{quote}

The ambiguity resolution algorithm as defined in 5.2.2 should be applied.

This is explicitly required for get() but is not required for iterator(). This causes the following inconsistency:

Assume two classes Foo and Bar which both implement common interface Common. Bar is an enabled alternative.

Assume further the following 
{noformat}
@Inject @Any Instance<Common> instance
{noformat}
injection point.

  was:
The specification currently says that:

{quote}
The iterator() method must:
• Identify the set of beans that have the required type and required qualifiers and are eligible for injection into the class into
which the parent Instance was injected, according to the rules of typesafe resolution, as defined in Section 5.2.1.
{quote}

The question that arises is whether the implementation should apply ambiguity resolution algorithm as defined in 5.2.2 or not.

This is explicitly required for get() but is not required for iterator(). This causes the following inconsistency:

Assume two classes Foo and Bar which both implement common interface Common. Bar is an enabled alternative.

Assume further the following 
{noformat}
@Inject @Any Instance<Common> instance
{noformat}
injection point.

It is clear that instance.get() returns Bar.

It is however not clear whether instance.iterator() iterates over:
a) Bar
b) Foo, Bar

and whether instance.isAmbiguous() returns:
a) true
b) false


    
> Instance.iterator() shouldn't include disabled alternatives
> -----------------------------------------------------------
>
>                 Key: CDI-331
>                 URL: https://issues.jboss.org/browse/CDI-331
>             Project: CDI Specification Issues
>          Issue Type: Feature Request
>          Components: Resolution
>    Affects Versions: 1.1.PRD
>            Reporter: Jozef Hartinger
>            Assignee: Pete Muir
>             Fix For: TBD
>
>
> The specification currently says that:
> {quote}
> The iterator() method must:
> • Identify the set of beans that have the required type and required qualifiers and are eligible for injection into the class into
> which the parent Instance was injected, according to the rules of typesafe resolution, as defined in Section 5.2.1.
> {quote}
> The ambiguity resolution algorithm as defined in 5.2.2 should be applied.
> This is explicitly required for get() but is not required for iterator(). This causes the following inconsistency:
> Assume two classes Foo and Bar which both implement common interface Common. Bar is an enabled alternative.
> Assume further the following 
> {noformat}
> @Inject @Any Instance<Common> instance
> {noformat}
> injection point.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira