ImageField écrase le fichier image du même nom

Question

J'ai le modèle UserProfile avec le champ avatar = models.ImageField(upload_to=upload_avatar)

upload_avatar nomme le fichier image selon user.id (12.png par exemple).

Mais lorsque l'utilisateur met à jour l'avatar, le nouveau nom d'avatar coïncide avec l'ancien nom d'avatar et Django ajoute le suffixe au nom de fichier (12-1.png par exemple).

Il y a moyen d' écraser le fichier au lieu de créer un nouveau fichier ?

Answer 1

Ouais, ça m'est arrivé aussi. Voici ce que j'ai fait.

Modèle:

 from app.storage import OverwriteStorage

class Thing(models.Model):
    image = models.ImageField(max_length=SOME_CONST, storage=OverwriteStorage(), upload_to=image_path)

Également défini dans models.py :

 def image_path(instance, filename):
    return os.path.join('some_dir', str(instance.some_identifier), 'filename.ext')

Dans un fichier séparé, storage.py :

 from django.core.files.storage import FileSystemStorage
from django.conf import settings
import os

class OverwriteStorage(FileSystemStorage):

    def get_available_name(self, name):
        """Returns a filename that's free on the target storage system, and
        available for new content to be written to.

        Found at http://djangosnippets.org/snippets/976/

        This file storage solves overwrite on upload problem. Another
        proposed solution was to override the save method on the model
        like so (from https://code.djangoproject.com/ticket/11663):

        def save(self, *args, **kwargs):
            try:
                this = MyModelName.objects.get(id=self.id)
                if this.MyImageFieldName != self.MyImageFieldName:
                    this.MyImageFieldName.delete()
            except: pass
            super(MyModelName, self).save(*args, **kwargs)
        """
        # If the filename already exists, remove it as if it was a true file system
        if self.exists(name):
            os.remove(os.path.join(settings.MEDIA_ROOT, name))
        return name

Évidemment, ce sont des exemples de valeurs ici, mais dans l'ensemble, cela fonctionne bien pour moi et cela devrait être assez simple à modifier si nécessaire.

Answer 2

class OverwriteStorage(get_storage_class()):

    def _save(self, name, content):
        self.delete(name)
        return super(OverwriteStorage, self)._save(name, content)

    def get_available_name(self, name):
        return name

Answer 3

Vous pouvez encore mieux écrire la classe de stockage de cette façon :

 class OverwriteStorage(FileSystemStorage):

    def get_available_name(self, name, max_length=None):
        self.delete(name)
        return name

Fondamentalement, cela écrasera la fonction get_available_name pour supprimer le fichier s'il existe déjà et retourner le nom du fichier déjà stocké

Answer 4

ahem... cela peut sembler peu orthodoxe, mais ma solution, à l'heure actuelle, consiste à vérifier et à supprimer le fichier existant dans le rappel que j'utilise déjà pour fournir le nom du fichier téléchargé. Dans models.py :

 import os
from django.conf import settings

def avatar_file_name(instance, filename):
    imgname = 'whatever.xyz'
    fullname = os.path.join(settings.MEDIA_ROOT, imgname)
    if os.path.exists(fullname):
        os.remove(fullname)
    return imgname
class UserProfile(models.Model):
    avatar = models.ImageField(upload_to=avatar_file_name,
                                default=IMGNOPIC, verbose_name='avatar')

Answer 5

Il suffit de référencer le champ d'image de votre modèle, de le supprimer et de l'enregistrer à nouveau.

 model.image.delete()
model.image.save()