ImageField file immagine sovrascrittura con lo stesso nome

Question

ho modello UserProfile con il campo avatar = models.ImageField(upload_to=upload_avatar)

upload_avatar nomi di funzioni file di immagine in base user.id (12.png per esempio).

Ma quando l'utente aggiorna l'avatar, il nuovo nome dell'avatar coincide con il vecchio nome dell'avatar e Django aggiunge il suffisso al nome del file (12-1.png ad esempio).

C'è un modo per sovrascrivere il file anziché creare il nuovo file?

Answer 1

Sì, questo è venuto anche per me. Ecco cosa ho fatto.

Modello:

from app.storage import OverwriteStorage 

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

definito anche in models.py:

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

in un file separato, storage.py: i valori dei campioni

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 

Ovviamente, questi sono qui , ma nel complesso questo funziona bene per me e questo dovrebbe essere abbastanza semplice da modificare se necessario.

Answer 2

Si può provare a definire il proprio Filesystemstorage e sovrascrivere il metodo predefinito get_availbale_name.

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

class MyFileSystemStorage(FileSystemStorage): 
    def get_available_name(self, name): 
     if os.path.exists(self.path(name)): 
      os.remove(self.path(name)) 
     return name 

Per la vostra immagine che si potrebbe definire un fs in questo modo:

fs = MyFileSystemStorage(base_url='/your/url/', 
    location='/var/www/vhosts/domain/file/path/') 
avatar = models.ImageField(upload_to=upload_avatar, storage=fs) 

Spero che questo aiuti.

Answer 3

ahem ... potrebbe sembrare non ortodosso, ma la mia soluzione, al momento, è di controllare & rimuovere il file esistente all'interno del callback che già utilizzo per fornire il nome del file caricato. In 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 4

è possibile scrivere classe di archiviazione ancora meglio in questo modo:

class OverwriteStorage(FileSystemStorage): 

    def get_available_name(self, name): 
     self.delete(name) 
     return name 

Answer 5

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 6

Basta refference il vostro modello di campo immagine, eliminarlo e salvare di nuovo.

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

Answer 7

Ho provato le soluzioni di cui qui. Ma sembra non funzionare a Django 1.10. Sarebbe aumentare il seguente errore da qualche parte al modello del admin:

url() missing 1 required positional argument: 'name'

Così mi si avvicinò con la mia soluzione, che consiste sulla creazione di un segnale pre_save che cerca di ottenere l'istanza dal database prima che sia salvato e elimina del percorso del file:

from django.db.models.signals import pre_save 


@receiver(pre_save, sender=Attachment) 
def attachment_file_update(sender, **kwargs): 
    attachment = kwargs['instance'] 
    # As it was not yet saved, we get the instance from DB with 
    # the old file name to delete it. Which won't happen if it's a new instance 
    if attachment.id: 
     attachment = Attachment.objects.get(pk=attachment.id) 
     storage, path = attachment.its_file.storage, attachment.its_file.path 
     storage.delete(path) 

Answer 8

per Django 1.10 Ho trovato ho dovuto modificare la risposta superiore per includere l'argomento max_length nella funzione:

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

class OverwriteStorage(FileSystemStorage): 
def get_available_name(self, name, max_length=None): 
    if self.exists(name): 
     os.remove(os.path.join(settings.MEDIA_ROOT, name)) 
    return name