atom feed26 messages in org.apache.myfaces.devRe: [VOTE] fix MYFACES-2552 for 2.0.x...
FromSent OnAttachments
Leonardo UribeSep 21, 2011 5:19 am 
Leonardo UribeSep 21, 2011 5:20 am 
Michael KurzSep 21, 2011 5:34 am 
Grant SmithSep 21, 2011 8:07 am 
Rudy De BusscherSep 21, 2011 8:14 am 
Gerhard PetracekSep 21, 2011 9:18 am 
Matt BensonSep 21, 2011 9:28 am 
Mark StrubergSep 21, 2011 10:47 am 
Mark StrubergSep 21, 2011 10:49 am 
Leonardo UribeSep 21, 2011 11:11 am 
Leonardo UribeSep 21, 2011 11:31 am 
Jakob KorherrSep 21, 2011 1:20 pm 
Grant SmithSep 21, 2011 1:42 pm 
Matt BensonSep 21, 2011 2:01 pm 
Mark StrubergSep 21, 2011 2:50 pm 
Bernd BohmannSep 22, 2011 12:52 am 
Gerhard PetracekSep 22, 2011 12:58 am 
Martin MarinschekSep 22, 2011 1:17 am 
Jakob KorherrSep 22, 2011 3:15 am 
Michael KurzSep 22, 2011 4:14 am 
Mark StrubergSep 22, 2011 4:17 am 
Michael KurzSep 22, 2011 4:29 am 
Bernd BohmannSep 22, 2011 5:15 am 
Michael KurzSep 22, 2011 5:22 am 
Leonardo UribeSep 22, 2011 7:10 am 
Grant SmithSep 22, 2011 7:59 am 
Subject:Re: [VOTE] fix MYFACES-2552 for 2.0.x and 2.1.x branches
From:Michael Kurz (mich@gmx.at)
Date:Sep 21, 2011 5:34:54 am
List:org.apache.myfaces.dev

+1

Am 21.09.2011 um 14:20 schrieb Leonardo Uribe:

+1

Hi

More than a year ago, it was found that EL expressions like #{cc.attrs.test} does not resolve its type correctly, because the composite component EL resolver is not able to find the right type. Instead, MapELResolver always return Object.class as type, breaking composite components that use h:selectOneXXX into its internals. See

https://issues.apache.org/jira/browse/MYFACES-2552

The problem with this issue is we need to change the way how org.apache.myfaces.el.unified.resolver.CompositeComponentELResolver works. JSF 2.0 spec clearly says in its section 5.6.2.2 that getType() for that EL resolver should return null.

The issue was reported to the EG and a fix was included in JSF 2.2. spec, see:

http://java.net/jira/browse/JAVASERVERFACES_SPEC_PUBLIC-745

but we still receive reports about the same issue (MYFACES-3311 and others (last comment on MYFACES-1890) ).

So, the current behavior even if is described by the spec is too inconvenient. Note we already have some places in our implementation that does not follow strictly the spec, to keep things working as users expect. To follow the protocol in these cases, we need an official community decision about include it in 2.0.x and 2.1.x branches. Please vote:

+1 if you want this fix included in 2.0.x and 2.1.x. +0 -1 and the reason why if you see this could cause any problem.

regards,

[1] http://www.apache.org/foundation/voting.html#ReleaseVotes