Deleting File/Image while uploading a new one

Hi,

I am new in django and I am having problems with the removal of a file when a user uploads a new one (the old file is no longer useful for the app).

The thing is that when I user wants to update his/her profile picture, my application should remove the old one. I tried this by overriding the save function in my form UserProfileForm, related to the UserProfile model.

I have written the same lines of the save function in the shell console and it seems to work. I also tried to override the save function in the model but I got the same negative result.

I would really appreciate if anyone could give me some advice about what I am doing wrong and how should I fix it! ;)

Thank you in advance, Mario

PS: I am using the django-profiles app. It calls the save function of the form_class, which is actually my UserProfileForm. =)

One portion of my model

class UserProfile(models.Model):
  user = models.ForeignKey(User, unique=True)
  name = models.CharField(max_length=30, null=True)
  image = models.ImageField(upload_to='profile_images', default='default-user.png')

My form for the model

from django import forms
from django.db import models
from django.forms import ModelForm
from myportal.portal.models import UserProfile
from myportal.portal.models import GENDER_CHOICES
from myportal.portal.models import LEVEL_CHOICES
from django.contrib.auth.models import User

from myportal.portal.files import handle_uploaded_file

from os.path import join as isfile
from django.conf import settings
import os

class ProfileForm(ModelForm):
    def __init__(self, *args, **kwargs):
        super(ProfileForm, self).__init__(*args, **kwargs)
        try:
            self.fields['email'].initial = self.instance.user.email
            # self.fields['first_name'].initial = self.instance.user.first_name
            # self.fields['last_name'].initial = self.instance.user.last_name
        except User.DoesNotExist:
            pass

    name = forms.CharField(max_length=30, label='Nombre')
    gender = forms.ChoiceField(choices=GENDER_CHOICES, label='Sexo')
    city = forms.CharField(max_length=30, label='Ciudad')
    level = forms.ChoiceField(choices=LEVEL_CHOICES, label='Nivel de juego')
    email = forms.EmailField(label='Correo')
    image = forms.ImageField(label='Imagen', required=False)

    class Meta:
      model = UserProfile
      exclude = ('user',)

    def save(self, *args, **kwargs):
        """
        Update the primary email address on the related User object as well.
        """
        u = self.instance.user
        u.email = self.cleaned_data['email']
        #u.save()
        profile = u.get_profile()
        image_path = settings.MEDIA_ROOT + profile.image.name
        if os.path.isfile(image_path):
            os.remove(image_path)
        profile = super(ProfileForm, self).save(*args,**kwargs)
        handle_uploaded_file(self.cleaned_data['image'])
        return profile

Finally the function handle_uploaded_file, which actually works fine! ;) from PIL import Image as PImage from os.path import join as pjoin from django.conf import settings

def handle_uploaded_file(file):
        #print type(file), "file.name=",file.name
        #print dir(file)
        #destination = open(MEDIA_ROOT + '/images/test.jpg', 'wb+')
        #for chunk in file.chunks():
        #   destination.write(chunk)
        imfn = pjoin(settings.MEDIA_ROOT + 'profile_images/', file.name)
        im = PImage.open(imfn)
        im.thumbnail((150,150), PImage.ANTIALIAS)
        im.save(imfn, "JPEG")