views:

29

answers:

2

Using the following related models (one blog entry can have multiple revisions):

class BlogEntryRevision(models.Model):
    revisionNumber = models.IntegerField()
    title = models.CharField(max_length = 120)
    text = models.TextField()
    [...]

class BlogEntry(models.Model):
    revisions = models.ManyToManyField(BlogEntryRevision)
    [...]

How can I tell Django to delete all related BlogEntryRevisions when the corresponding BlogEntry is deleted? The default seems to be to keep objects in a many-to-many relation if an object of the "other" side is deleted. Any way to do this - preferably without overriding BlogEntry.delete?

+1  A: 

You can use a custom model manager, but the documentation seems to indicate that it does do something like this already and I can't recall exactly what this means:

The delete method, conveniently, is named delete(). This method immediately deletes the object and has no return value. Example:

e.delete()

You can also delete objects in bulk. Every QuerySet has a delete() method, which deletes all members of that QuerySet.

For example, this deletes all Entry objects with a pub_date year of 2005:

Entry.objects.filter(pub_date__year=2005).delete()

Keep in mind that this will, whenever possible, be executed purely in SQL, and so the delete() methods of individual object instances will not necessarily be called during the process. If you've provided a custom delete() method on a model class and want to ensure that it is called, you will need to "manually" delete instances of that model (e.g., by iterating over a QuerySet and calling delete() on each object individually) rather than using the bulk delete() method of a QuerySet.

When Django deletes an object, it emulates the behavior of the SQL constraint ON DELETE CASCADE -- in other words, any objects which had foreign keys pointing at the object to be deleted will be deleted along with it. For example:

b = Blog.objects.get(pk=1)
# This will delete the Blog and all of its Entry objects.
b.delete()
Daniel DiPaolo
Right, cascading deletes are the default for 1:N (`ForeignKey`) relations like `Blog 1:N Entry` in the quoted documentation. But I have a `ManyToManyRelation`, so the cascade only deletes the record which says "entry X and revision Y belong together" but not the `BlogEntryRevision` record itself. Maybe I should do it with `ForeignKey` in my use case... Anyway, consider the question as a generic question on many-to-many cascading deletions.
AndiDog
+1  A: 

I think you are misunderstanding the nature of a ManyToMany relationship. You talk about "the corresponding BlogEntry" being deleted. But the whole point of a ManyToMany is that each BlogEntryRevision has multiple BlogEntries related to it. (And, of course, each BlogEntry has multiple BlogEntryRevisions, but you know that already.)

From the names you have used, and the fact that you want this deletion cascade functionality, I think you would be better off with a standard ForeignKey from BlogEntryRevision to BlogEntry. As long as you don't set null=True on that ForeignKey, deletions will cascase - when the BlogEntry is deleted, all Revisions will be too.

Daniel Roseman
Yes, I admitted that mistake in the comment to the other answer. For the sake of completeness: Assuming a use case that makes more sense with `ManyToManyRelation`, what would be a good way to cascade deletes? Does the approach from the deleted answer of Gabi Purcaru work?
AndiDog