MarkupContainer.removeAll() does not detach models recursively

[Nazaret K <na...@gmail.com>]

Hi,

There are cases where I am explicitly removing all children components
from a ListView by calling removeAll();
For some reason, the removeAll() on MarkupContainer does not call
detach() on its child components, instead it calls detachModel();
This has the side effect that the models of the components that are
deeper in the component tree are never detached.
Is this a bug?

I am using wicket 1.4.17

Thanks

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org

Re: MarkupContainer.removeAll() does not detach models recursively

[Igor Vaynberg <ig...@gmail.com>]

i believe that is fixed in 1.4.x and will make it into upcoming 1.4.18

-igor

On Fri, Jul 1, 2011 at 4:47 AM, Nazaret K <na...@gmail.com> wrote:
> Hi,
>
> There are cases where I am explicitly removing all children components
> from a ListView by calling removeAll();
> For some reason, the removeAll() on MarkupContainer does not call
> detach() on its child components, instead it calls detachModel();
> This has the side effect that the models of the components that are
> deeper in the component tree are never detached.
> Is this a bug?
>
> I am using wicket 1.4.17
>
> Thanks
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
> For additional commands, e-mail: users-help@wicket.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@wicket.apache.org
For additional commands, e-mail: users-help@wicket.apache.org