how do you check that? i already did some fixes so that internal things do detach and also our AbstractWrapModel doesn't have an empty detach any more
On 11/14/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > so instead of detaching make it a check and throw an exception > > -igor > > On 11/13/07, Johan Compagner <[EMAIL PROTECTED]> wrote: > > yeah thats nice to say, but thats not really up to us. > > So we can make sure that it is really detached by calling it. > > As i said even all our own interal wrapmodels don't do that currently, so > it > > is that easy to forget. > > > > johan > > > > > > > > On Nov 14, 2007 12:16 AM, Eelco Hillenius <[EMAIL PROTECTED]> > wrote: > > > > > On Nov 13, 2007 3:03 PM, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > > > > i should always be able to simply call .detach() on any model and > > > > expect it to be detached no matter how it was wrapped, etc > > > > > > +1 > > > > > > Eelco > > > > > >
