MarkupContainer.removeAll() does not detach models recursively

2011-07-01 Thread Nazaret K 
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-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org

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

2011-07-01 Thread Igor Vaynberg 
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 nazare...@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-unsubscr...@wicket.apache.org
 For additional commands, e-mail: users-h...@wicket.apache.org



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