myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Leonardo Uribe <>
Subject Re: Failed Tests in 2.0.x and 2.1.x and UIComponent.findComponent contract is not correctly implemented
Date Fri, 21 Oct 2011 16:35:16 GMT
Hi Bernd

I have checked the problem against the algorithm implemented in 2.0.x
and I see the problem. The spec does not define if this method can be
overriden, so that details was ignored on MYFACES-3268, and all known
tests doesn't check that part. It seems we need to sync the algorithm
in 1.2.x with the code in 2.0.x, which was enhanced.

What I see is the case that fails is:


The base when a findComponent expression starts with ':' is
UIViewRoot, the algorithm found it, then it found data, but the code
does not delegate to data, instead start looking from data.

This is the patch agains 2.0.x branch:

Note the overriden method on the test case has a problem, because the
call for findComponent does not assume the id of the map should be

I have seen other changes on 1.2.x, but I think we should copy the
code on 2.0.x/2.1.x to 1.2.x (changing the separator stuff). Do you
agree with this fix?


Leonardo Uribe

2011/10/21 Bernd Bohmann <>:
> Hello,
> just commited a Test that shows the wrong behavior of
> UIComponent.findComponent since 2.0.8 and 2.1.2.
> I would like to discuss this with leonardo.
> Sorry for any inconvenience.
> Regards
> Bernd

View raw message