开发者

Why won't my file save to instance (it saves to disk...)?

开发者 https://www.devze.com 2023-03-15 04:43 出处:网络
I can get my file to save to disk where I tell it to, but can\'t get it to save to the instance and I haven\'t the slightest idea why!

I can get my file to save to disk where I tell it to, but can't get it to save to the instance and I haven't the slightest idea why!

models.py

    class Song(models.Model):
        name = models.CharField(max_length=50)
        audio_file = models.FileField(upload_to='uploaded/music/', blank=True)

views.py

    def create_song(request, band_id):
        if request.method == 'POST':
    开发者_如何学Python        band = Band.objects.get(id=band_id)
            form = SongForm(request.POST, request.FILES)
            if form.is_valid():
                handle_uploaded_file(request.FILES['audio_file'])
                form.save()
                return HttpResponseRedirect(band.get_absolute_url)
        else:
            form = SongForm(initial={'band': band_id})
        return render_to_response('shows/song_upload.html', {'form': form}, context_instance=RequestContext(request))

handle_uploaded_file

    def handle_uploaded_file(f):
        ext = os.path.splitext(f.name)[1]
        destination = open('media/uploaded/music/name%s' %(ext), 'wb+')
        for chunk in f.chunks():
            destination.write(chunk)
        destination.close()

song_upload.html (relevant part)

    {% block main %}
    {{band.name}}
        <form enctype="multipart/form-data" method="post" action="">{% csrf_token %}
           {{ form.as_p}}
           <input type="submit" value="Add song" />
        </form>
    {% endblock %}

forms.py

    class SongForm(forms.ModelForm):
        band = forms.ModelChoiceField(queryset=Band.objects.all(), widget=forms.HiddenInput) 
        def clean_audio_file(self):
            file = self.cleaned_data.get('audio_file',False)
            if file:
                if file._size > 10*1024*1024:
                    raise forms.ValidationError("Audio file too large ( > 10mb)")
                if not file.content_type in ["audio/mp3", "audio/mp4"]:
                    raise forms.ValidationError("Content type is not mp3/mp4")
                if not os.path.splitext(file.name)[1] in [".mp3", ".mp4"]:
                    raise forms.ValidationErorr("Doesn't have proper extension")
            else:
                raise forms.ValidationError("Couldn't read uploaded file")
        class Meta:
            model = Song

The file is right there in media/uploaded/music, but in admin audio_file is blank, and if i set blank=False (which is what I want to do) for audio_file, I'm told this field is required. What gives??

Thanks in advance! Been at this one for a while now, docs seem light to me (newb).


clean_audio_file should return the cleaned data for this specific field, so you will need to add a return file to it!

From django's documentation:

Just like the general field clean() method, above, this method should return the cleaned data, regardless of whether it changed anything or not.


As far as I know, you don't need to handle_uploaded_file(). You're using ModelForm and models.FileField. It supports upload_to argument, and saves the file to the destination automatically.

The basic file uploading example from Django docs is using Form and forms.FileField, and because of that it involves handle_uploaded_file() function.

So, try to simply remove this line:

handle_uploaded_file(request.FILES['audio_file'])

from your view and please tell what happens then.

EDIT: Also, @lazerscience is right, you need to add

return self.cleaned_data

at the end of your clean_audio_file(). (But not return file, since you always should return the whole cleaned data dictionary even if you're cleaning just one field.)

0

精彩评论

暂无评论...
验证码 换一张
取 消