13

In Django, I have the following model:

from django.db import models
from django.core.files.base import File
import os, os.path

class Project(models.Model):
    video = models.FileField(upload_to="media")

    def replace_video(self):
        """Convert video to WebM format."""
        # This is where the conversion takes place,
        # returning a path to the new converted video
        # that I wish to override the old one.
        video_path = convert_video()

        # Replace old video with new one,
        # and remove original unconverted video and original copy of new video.
        self.video.delete(save=True)
        self.video.save(os.path.basename(video_path), File(open(video_path ,"wb")), save=True)
        os.remove(video_path)

I want to be able to replace the file in the FileField video on a model object/instance. The above method I've written does not work. Once I delete the original file, I get the following error:

ValueError: The 'video' attribute has no file associated with it.

How can I replace the file with an updated one, and remove the original one (no more necessary)?

Side-Note: I have found a related issue, but with no satisfying answer.

Community
  • 1
  • 1
Ory Band
  • 14,716
  • 14
  • 59
  • 66

3 Answers3

12

You have two choices.

I'll assume your Project model is only a snippet of code.

Option 1 is to break your model down so that a Project does not have a single file, but rather a Project model is associated with a ProjectFile model. Perhaps one-to-many. One Project as many ProjectFiles. That is, ProjectFile has a ForeigKey to Project.

Then you can add new ProjectFile based on an old ProjectFile. You can delete them, and fool around all you want. Indeed, you can keep both ProjectFile's with an indicator of which is "current".

Option 2 is to self.video.open("w") to open the file for writing. Rewrite the contents "in place". Instead of deleting and replacing the file, rewrite the old file with the new content.

with open(video_path ,"rb") as source:
    self.video.open("wb")
    bytes= source.read(4096)
    if bytes: 
        self.video.write( bytes )
        bytes= source.read(4096)

That will probably do what you want.

Yes, it seems inefficient. It's really not that bad. The conversion takes for ever. The copy takes moments.

S.Lott
  • 384,516
  • 81
  • 508
  • 779
5

I came across this problem recently myself, and solved it something like this:

from django.db import models
from django.core.files.base import File
import os, os.path

class Project(models.Model):
    video = models.FileField(upload_to="media")

    def replace_video(self):
        """Convert video to WebM format."""
        # This is where the conversion takes place,
        # returning a path to the new converted video
        # that I wish to override the old one.
        video_path = convert_video()

        # Replace old video with new one,
        # and remove original unconverted video and original copy of new video.
        old_path = self.video.path
        self.video.save(os.path.basename(video_path), File(open(video_path ,"wb")), save=True)
        os.remove(video_path)
        os.remove(old_path)
dwurf
  • 12,393
  • 6
  • 30
  • 42
  • Could show where in the docs this functionality pertains – There Aug 16 '21 at 00:44
  • This answer is 8 years old now. Unless you're using Django v1 I don't think the docs, or this answer, will help you much. Just in case, [here they are](https://docs.djangoproject.com/en/1.8/ref/forms/fields/#django.forms.FileField). – dwurf Aug 16 '21 at 01:34
0

Another solution is to delete the original model instance and create a new one.

For example:

def upload_metadata(request):
  if request.method == 'POST':
    try: # removes existing file
      uf = UserFile.objects.get(
        library_id = request.POST['library_id'],
        file = 'uploads/' + request.FILES['file']._get_name())
      uf.delete()
    except Exception as e:
      pass
    # ... continue with form handling as before ...
There
  • 498
  • 6
  • 18