Mailing-List: contact dev-help@myfaces.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "MyFaces Development" <dev@myfaces.apache.org>
Received-SPF: pass (athena.apache.org: domain of lu4242@gmail.com designates
 209.85.210.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CADt_2avqJ73aMAkw9sAMQphdipm3xgYmpjE7ajfgkZDmKeJJ6A@mail.gmail.com>
References: 
 <CADt_2asvy+WuZWp4p8nNnjtUaqhWp7Ub44LOgcf--+7YVJXWwQ@mail.gmail.com>
	<CABujnVcdm6hVuTDesB-Hbw7f08gSVbJUMUZE5oAMkuQ9s49bfA@mail.gmail.com>
	<CADt_2asqAj_CVj8gmAx4=XrBxM0zcJnbboSm34Zj3vnk76i5rg@mail.gmail.com>
	<CABujnVf5usxnob-DWk9d_pkEtOiv+qiD6r+MLPUQqKGAXstq8A@mail.gmail.com>
	<CADt_2avqJ73aMAkw9sAMQphdipm3xgYmpjE7ajfgkZDmKeJJ6A@mail.gmail.com>
Date: Fri, 21 Oct 2011 16:09:23 -0500
Message-ID: 
 <CABujnVf+a0qsmjKoU0+dAOqZ71nbRzL79PpWbA__YzXPSY_4kA@mail.gmail.com>
Subject: Re: Failed Tests in 2.0.x and 2.1.x and UIComponent.findComponent
 contract is not correctly implemented
From: Leonardo Uribe <lu4242@gmail.com>
To: Bernd Bohmann <bernd.bohmann@atanion.com>
Cc: MyFaces Development <dev@myfaces.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

Hi Bernd

It looks good.

regards,

Leonardo

2011/10/21 Bernd Bohmann <bernd.bohmann@atanion.com>:
> Hello Leonardo,
>
> can you review my commit, please.
>
> Thanks
>
> Bernd
>
> On Fri, Oct 21, 2011 at 9:36 PM, Leonardo Uribe <lu4242@gmail.com> wrote:
>> Hello Bernd
>>
>> I see. Could you commit the solution for 2.0.x/2.1.x branch?
>>
>> regards,
>>
>> Leonardo Uribe
>>
>> 2011/10/21 Bernd Bohmann <bernd.bohmann@atanion.com>:
>>> Hello Leonardo,
>>> I think we should fix it like my suggestion in 1.2.x. Your suggestion
>>> didn't work with the "data:1:command" expression. You can see it with
>>> my last commit on the trunk.
>>> Regards
>>> Bernd
>>>
>>> On Fri, Oct 21, 2011 at 6:35 PM, Leonardo Uribe <lu4242@gmail.com> wrote:
>>>> 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:
>>>>
>>>> :data:1:command
>>>>
>>>> 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:
>>>>
>>>> https://issues.apache.org/jira/secure/attachment/12500202/MYFACES-3268-3-fix-delegation.patch
>>>>
>>>> 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
>>>> attached.
>>>>
>>>> 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?
>>>>
>>>> regards,
>>>>
>>>> Leonardo Uribe
>>>>
>>>> 2011/10/21 Bernd Bohmann <bernd.bohmann@atanion.com>:
>>>>> 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
>>>>>
>>>>
>>>
>>
>