[jboss-as7-dev] Module for EL

[Gunnar Morling]

Thanks to Scott's help, I think I understood the details a bit more.

HV uses javax.el.ExpressionFactory/FactoryFinder which in turn uses the
TCCL to find implementations and cache it in FactoryFinderFache. For WAR
deployments, the cache is pre-populated by ELExpressionFactoryProcessor
with the implementation from the web module mentioned by Shelly.

As this is not the case for other deployment types (e.g. RARs), tests using
HV based on such deployment types fail with the exception.

So I think a separate EL module (upon which HV could depend, independently
from the current deployment type) would indeed make sense.

--Gunnar



2013/4/9 Scott Marlow <smarlow at redhat.com>

> Here is some of it http://pastie.org/7391504 (shows the Caused by:
> "javax.el.ELException: Provider org.apache.el.ExpressionFactoryImpl not
> found").
>
> Should we create a static module that contains
> org.apache.el.ExpressionFactoryImpl?  Currently, jbossweb-7.2.0.Final
> contains this class.
>
> On 04/09/2013 09:33 AM, Jaikiran Pai wrote:
> > Hi Gunnar,
> >
> > Where are you seeing the failures? Can you point us to the stacktrace?
> >
> > -Jaikiran
> > On Tuesday 09 April 2013 06:59 PM, Gunnar Morling wrote:
> >> Hi all,
> >>
> >> I'm currently working on updating the AS module for Hibernate
> >> Validator to the latest release (HV 5.x, Bean Validation 1.1, see [1]).
> >>
> >> As of this release, Hibernate Validator has a new dependency to the
> >> Unified EL. So I have added this to the HV module descriptor:
> >>
> >>     <module name="javax.el.api"/>
> >>
> >> Based on test failures I'm seeing ("javax.el.ELException: Provider
> >> org.apache.el.ExpressionFactoryImpl not found"), I think I need to add
> >> a dependency for the actual EL implementation as well. I don't know
> >> though which module that would be.
> >>
> >> Does anyone know which is the right one or has an example of other
> >> modules depending on EL?
> >>
> >> Thanks,
> >>
> >> --Gunnar
> >>
> >> [1] https://issues.jboss.org/browse/AS7-6665
> >>
> >>
> >>
> >> _______________________________________________
> >> jboss-as7-dev mailing list
> >> jboss-as7-dev at lists.jboss.org
> >> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
> >
> >
> >
> > _______________________________________________
> > jboss-as7-dev mailing list
> > jboss-as7-dev at lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
> >
>
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-as7-dev/attachments/20130409/7a9fef32/attachment.html