On 1/18/06, hugo <[EMAIL PROTECTED]> wrote: > >Ok; in the absence of any objections, I've just committed the change, > >and closed off the ticket. Bulk delete is now available in > >magic-removal. > > Hum. That's a _tad_ bit fast committing for me - just because nobody > commented on it doesn't say there aren't any comments on it in the > queue :-)
Yeah, that's a tad bit fast for me, as well. A lack of comments doesn't give consent. > One problem I see is with related objects. When Django does a .delete() > on an object, it does delete related objects with it. And it deletes > especially used rows in relation tables (those that are behind the > ManyToMany machinery). Does your patch handle this? I don't think so - > so when using the bulk delete, there might be leftover related objects > that aren't deleted. The patch doesn't appear to handle this, which means we have inconsistent behavior between bulk delete and normal deletion. Adrian -- Adrian Holovaty holovaty.com | djangoproject.com | chicagocrime.org