Замена изображения Django не удаляет оригинал

Лучшая стратегия, которую я нашел, - это создать собственный метод сохранения в модели:

class Photo(models.Model):

    image = ImageField(...) # works with FileField also

    def save(self, *args, **kwargs):
        # delete old file when replacing by updating the file
        try:
            this = Photo.objects.get(id=self.id)
            if this.image != self.image:
                this.image.delete(save=False)
        except: pass # when new photo then we do nothing, normal case          
        super(Photo, self).save(*args, **kwargs)

И будьте осторожны, так как при обновлении, которое не удаляет серверный файл, удаление модели экземпляра (здесь фото) не удаляет серверный файл, во всяком случае, не в Django 1.3, вы для этого нужно добавить больше пользовательского кода (или регулярно выполнять какую-то грязную работу cron).

Наконец, проверьте все ваши случаи обновления/удаления с вашими отношениями ForeignKey, ManytoMany и другими, чтобы проверить, правильно ли удалены внутренние файлы. Верьте только тому, что проверяете.

Разве замена образа не должна также удалять ненужный файл с диска?

Раньше FileField стремился очистить потерянные файлы. Но это изменилось в Django 1.2:

В более ранних версиях Django при удалении экземпляра модели, содержащего FileField, FileField брал на себя также удаление файла из внутреннего хранилища. Это открыло двери для нескольких потенциально серьезных сценариев потери данных, включая откаты транзакций и поля в разных моделях, ссылающиеся на один и тот же файл. В Django 1.2.5 FileField никогда не будет удалять файлы из внутреннего хранилища.

Код в следующем рабочем примере после загрузки изображения в поле ImageField определит, существует ли файл с таким же именем, и в этом случае удалит этот файл перед сохранением нового.

Его можно легко изменить так, чтобы он удалял старый файл независимо от имени файла. Но это не то, что я хотел в своем проекте.

Добавьте следующий класс:

from django.core.files.storage import FileSystemStorage
class OverwriteStorage(FileSystemStorage):
    def _save(self, name, content):
        if self.exists(name):
            self.delete(name)
        return super(OverwriteStorage, self)._save(name, content)

    def get_available_name(self, name):
        return name

И используйте его с ImageField следующим образом:

class MyModel(models.Model):
    myfield = models.ImageField(
        'description of purpose',
        upload_to='folder_name',
        storage=OverwriteStorage(),  ### using OverwriteStorage here
        max_length=500,
        null=True,
        blank=True,
        height_field='height',
        width_field='width'
    )
    height = models.IntegerField(blank=True, null=True)
    width = models.IntegerField(blank=True, null=True)

Если вы не используете транзакции или не боитесь потерять файлы при откате транзакции, вы можете использовать django- очистка

Было несколько обращений по этой проблеме, хотя, скорее всего, она не попадет в ядро. Наиболее полным является http://code.djangoproject.com/ticket/11663. Патчи и комментарии к заявкам могут дать вам некоторое направление, если вы ищете решение.

Вы также можете рассмотреть возможность использования другого StorageBackend, такого как система хранения файлов с перезаписью, указанная во фрагменте 976 Django. .org/snippets/976/. Вы можете изменить хранилище по умолчанию на этот бэкэнд или переопределить его в каждом объявлении FileField/ImageField.

Вот код, который может работать с upload_to=... или blank=True или без них, а также когда отправленный файл имеет то же имя, что и старый.

(синтаксис py3, протестирован на Django 1.7)

class Attachment(models.Model):

    document = models.FileField(...)  # or ImageField

    def delete(self, *args, **kwargs):
        self.document.delete(save=False)
        super().delete(*args, **kwargs)

    def save(self, *args, **kwargs):
        if self.pk:
            old = self.__class__._default_manager.get(pk=self.pk)
            if old.document.name and (not self.document._committed or not self.document.name):
                old.document.delete(save=False)
        super().save(*args, **kwargs)

Помните, что такое решение применимо только в том случае, если вы находитесь в нетранзакционном контексте (без отката, потому что файл окончательно потерян)

Я использовал простой метод с popen, поэтому, когда я сохраняю свою модель Info, я удаляю прежний файл перед связыванием с новым:

import os

try:
    os.popen("rm %s" % str(info.photo.path))
except:
    #deal with error
    pass
info.photo = nd['photo']

Сохраняю исходный файл и если он изменился - удаляю.

class Document(models.Model):
    document = FileField()

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self._document = self.document

    def save(self, *args, **kwargs):
        if self.document != self._document:
            self._document.delete()
            super().save(*args, **kwargs)