Lost electricity
On Oct 1, 2013 4:33 PM, "Stuart Douglas" <notifications@github.com> wrote:

> In
> ee/src/main/java/org/jboss/as/ee/component/deployers/ResourceInjectionAnnotationParsingProcessor.java:
>
> > @@ -235,7 +225,7 @@ protected void process(final DeploymentPhaseContext phaseContext, final EEModule
> >          }
> >          InjectionSource valueSource = null;
> >          final boolean isEnvEntryType = this.isEnvEntryType(injectionType, module);
> > -        final boolean isResourceRefType = RESOURCE_REF_ENTRIES.contains(injectionType);
> > +        final boolean isResourceRefType = annotation.value("name") != null && RESOURCE_REF_ENTRIES.contains(injectionType);
>
> Name defaults according to a certain set of rules if it is not specified,
> its presence (or lack thereof) should not be used to determine how to treat
> the lookup.
>
> I'm not sure what you mean about the DataSource and Conn factory problem?
> Basically if nothing is specified it should default to the defaults,
> however if it is specified in a deployment descriptor the DD should
> override. This should be taken care of automatically by the existing
> infrastructure.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/wildfly/wildfly/pull/5176/files#r6690037>
> .
>