CODEWITHSUNDEEP
pythondjangoimageforms
Ebtessam Zoheir
Ebtessam Zoheir

Reputation: 169

form.is_valid(): always returns false

Form.is_valid() always returns false. Here's my code.

#urls.py
url(r'^create/', "app.views.createpost", name="createpost"),

My models.py

class Post(models.Model):
    """docstring for Post"""
    post_image = AjaxImageField(upload_to='posts', max_width=200, max_height=200, crop=True, null= False, default='site_media/media/BobMarley/bob1.jpg')
    poster = models.ForeignKey(User, null= False, default=User.objects.get(username="admin")

Here's my forms.py

#forms.py
class AjaxImageUploadForm(forms.Form):
    image = forms.URLField(widget=AjaxImageWidget(upload_to='posts'))

view.py

#views.py
def createpost(request):

    if request.method == 'POST':
        form = AjaxImageUploadForm(request.POST, request.FILES)
        if form.is_valid():
            newpost = Post(post_image = request.FILES['image'])
            newpost.poster = request.user
            newpost.save()

            return HttpResponseRedirect('/create/')


    else:
        form = AjaxImageUploadForm() # An empty, unbound form


    posts = Post.objects.all()

    return render_to_response('create.html',{'posts': posts, 'form': form},context_instance=RequestContext(request))

The Template

#create.html
{% block extra_head %}    
    {{ form.media }}
{% endblock %}
{% block body %}
    <form method="POST" enctype="multipart/form-data" action="{% url "createpost" %}">      
        {% csrf_token %} 
        {{ form.errors}}
        {{ form.as_p }}
        <button type="submit" name='image'>Upload</button>
    </form>   
{% endblock %}

The form is never valid and the error printed is "This field is required." And this is the form widget begin created 

<tr><th><label for="id_image">Image:</label></th><td><ul class="errorlist"><li>This field is required.</li></ul>
<div class="ajaximage"><a class="file-link" target="_blank" href=""> <img class="file-img" src=""></a> <a class="file-remove" href="#remove">Remove</a>
<input class="file-path" type="hidden" value="" id="id_image" name="image" /> <input type="file" class="file-input" name="image"/> <input class="file-dest" type="hidden" name="image" value="/ajaximage/upload/posts/0/0/0"> <div class="progress progress-striped active"> <div class="bar"></div></div>    
</div>
</td></tr>

I am using the django package called django-ajaximage and their custom widget (AjaxImageWidget) is probably a custom text widget

class AjaxImageWidget(widgets.TextInput):

Thank you for you help

Upvotes: 0

Views: 763

Answers (1)

&#193;rni St. Steinunnarson
&#193;rni St. Steinunnarson

Reputation: 1610

You do not need the init on your model class. Also, null=False is the default. I'm not sure the text 'admin' will default to the user with username admin. You would need to do:

default=User.objects.get(username="admin")

You can also include the poster assignment in the creation of the object like so:

newpost = Post(post_image=request.FILES['image'], poster=request.user)

To make the form validate you need to put blank=True in the field like so:

poster = models.ForeignKey(User, blank=True, default=User.objects.get(username="admin")

I would also make the form be a ModelForm:

class AjaxImageUploadForm(forms.ModelForm):
    class Meta:
        model=Post
        exclude=['poster',]

Upvotes: 1

Related Questions