On Jun 22, 2006, at 2:00 PM, Jay Parlar wrote: > I noticed you just checked the patch in, thanks very much. I do have > the question though: Was the patch actually doing the overall right > thing? My patch modified the admin itself, instead of any of your > generic stuff. > > Maybe because of the nature of the generic stuff, this was the way to > do it, but it somehow *felt* that the generic key code should change, > not the admin. I only patched the admin because it was easy, and > wanted to get something out there as a temporary fix in case anyone > else needed it.
Yeah, you actually had it right -- the admin delete code is somewhat more involved than the general case delete(). At least, I think that's right... I'll go back when I get a moment and add some delete () unit tests to make sure. Jacob --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django users" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-users -~----------~----~----~----~------~----~------~--~---
