Could this be backported to 1.2 as well?

   Erik.

Martijn Dashorst schreef:
This change gives me a working unit test for 1.3 (ComponentTest).

Martijn

On 11/30/06, Martijn Dashorst <[EMAIL PROTECTED]> wrote:
I propose to remove the attached check in page.internalDetach (line
287) and to remove
- the internalDetach() call in component.renderComponent() (line 1658)
- the detachModels() call to behavior in
AjaxRequestTarget.respondHeaderContributions (lines 533 and 543)

Then we still detach everything, but only once (the behavior still
gets detached twice during the ajax request). Because the page call
still detaches everything we still perform correct detach behavior,
and we don't have scattered detach logic throughout the framework.

Martijn

--
Erik van Oosten
http://www.day-to-day-stuff.blogspot.com/

Reply via email to