django: calling delete on object doesn't delete the file

Question

So I have the following model.

class StoreSegments01(models.Model):
    segment = models.FileField(upload_to=content_file_name)
    segmentID = models.TextField(max_length=100, default=11)

class StoreSegments01Form(forms.ModelForm):
    class Meta:
    model = StoreSegments01
    fields = ['segment', 'segmentID']

def content_file_name(instance, filename):
    return '{0}'.format(instance.segmentID)

I want to provide a way for the users to delete their files. In manage.py shell I try the following:

obj = StoreSegments01.objects.get(segmentID='239fd363-562a-41b3-a915-b7a84cc4a172')
>>> obj.delete()

It deletes the record related to the specified segmentID, but the file associated with the ID is still there. I tried the same with queryset, still the file is not deleted.

What am I missing here?

Possible duplicate of How do I get Django Admin to delete files when I remove an object from the database/model? — Variant
– Variant, Commented Jan 30, 2016 at 16:34
Or just install pip install django-cleanup as one of the answers mention — Ramast
– Ramast, Commented Jan 30, 2016 at 16:51

Community · Accepted Answer · 2017-05-23 12:32:40Z

4

Here's an alternative to using signals: override the delete method of your model.

Example:

class StoreSegments01(models.Model):
    segment = models.FileField(upload_to=content_file_name)
    segmentID = models.TextField(max_length=100, default=11)

    def delete(self, *args, **kwargs):
        # first, delete the file
        self.segment.delete(save=False)

        # now, delete the object
        super(StoreSegments01, self).delete(*args, **kwargs)

For Python 3, call super like this: super().delete(*args, **kwargs)

Heads up! : This works if you delete individual objects. But if you delete objects in bulk, the associated files aren't deleted. Because deleting a single object calls model.delete() method but deleting in bulk calls queryset.delete() method. Thanks to Håken Lid who pointed this out in a comment below.

edited May 23, 2017 at 12:32

CommunityBot

11 silver badge

answered Jan 30, 2016 at 18:34

xyres

22k5 gold badges64 silver badges89 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Håken Lid Over a year ago

You can use django.db.models.signals.pre_delete instead of overriding the delete() method. Signals will listen to both model.delete() and queryset.delete(). Docs: docs.djangoproject.com/en/1.9/topics/signals

xyres Over a year ago

@HåkenLid Hey, thanks. Now I understand why files weren't deleting using bulk delete.

Tarek Kalaji Over a year ago

You made my day, thany you a lot, it was very simple :D self.segment.delete(save=False) and also super(self.__class__, self).delete(*args, **kwargs)

Collectives™ on Stack Overflow

django: calling delete on object doesn't delete the file

1 Answer 1

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related