[ 
https://issues.apache.org/jira/browse/MYFACES-2621?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12848884#action_12848884
 ] 

Jakob Korherr commented on MYFACES-2621:
----------------------------------------

Totally right!

Of course, we could check if unified el 2.2 is available before the creation of 
the TagValueExpression and use an el-2.2 aware implementation of 
TagValueExpression instead. But this would lead to lots of duplicate code, 
because TagValueExpression is not the only ValueExpression wrapper. There are 
some more.

Furthermore I don't think that this is that slow when using reflection. Also a 
big advantage here is for future versions of MyFaces that have el-2.2 as a 
requirement that we just have to change this one method to not use reflection 
in every wrapper.

I am prefering the reflection way, but if you think that this is just too slow, 
we can also have the duplicate wrappers ;)

> BeanValidation does not work with Unified EL 2.2
> ------------------------------------------------
>
>                 Key: MYFACES-2621
>                 URL: https://issues.apache.org/jira/browse/MYFACES-2621
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-314
>    Affects Versions: 2.0.0-beta-3
>         Environment: bean validation 1.0.0.GA, unified el 2.2
>            Reporter: Jakob Korherr
>            Assignee: Jakob Korherr
>
> When using the new Unified EL 2.2, BeanValidation does not work, because 
> _BeanValidatorUELUtils.getUELValueReferenceWrapper() always returns null.
> See also the related thread on the mailing list: 
> http://www.mail-archive.com/[email protected]/msg55250.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to