CODEWITHSUNDEEP
pythondjango
Ann
Ann

Reputation: 11

ValueError when creating Django comment form

I have a problem when I was creating the comment form with django. After I wrote my view.py, models.py and html, I got an ValueError that said: 

Cannot assign "<class 'blog.models.post'>": "Comment.post" must be a "post" instance".

Below are my codes.

HTML

{% block content %}
<h1>Add New Comment:</h1>
<form method='POST' action=''>
    {% csrf_token %}
    {{ form.as_p }}
<button type='submit'>Submit</button>
</form>
{% endblock %}

views.py

def add_comment(request, slug):
    po = get_object_or_404(post, slug=slug)
    if request.method == 'POST':
        form = CommentForm(request.POST or None)
        if form.is_valid():
            comment = form.save(commit=False)
            comment.post = post
            comment.save()
            return redirect('blog:post', slug=post.slug)
    else:
        form = CommentForm()
    return render(request, 'blog/post/add_comment.html', {'form': form})

models.py

class Comment(models.Model):
    post = models.ForeignKey(post, related_name='comments', on_delete=models.CASCADE)
    user = models.CharField(max_length=250)
    email = models.EmailField() 
    body = models.TextField() 
    created = models.DateTimeField(auto_now_add=True) 
    approved = models.BooleanField(default=False) 

    def approved(self):
        self.approved = True
        self.save()

    def __str__(self):
        return self.user

Upvotes: 1

Views: 191

Answers (2)

Alasdair
Alasdair

Reputation: 308809

The post you fetched from the database is po:

po = get_object_or_404(post, slug=slug)

Therefore you should set form.post = po:

def add_comment(request, slug):
    po = get_object_or_404(post, slug=slug)
    if request.method == 'POST':
        form = CommentForm(request.POST or None)
        if form.is_valid():
            comment = form.save(commit=False)
            comment.post = po
            comment.save()

Note that normally in Django you would use Post for your model and post for the instance you fetch from the database.

Upvotes: 1

user10898133
user10898133

Reputation:

Try it:

class Comment(models.Model):
    post = models.ForeignKey(Post, related_name='comments', on_delete=models.CASCADE)

Upvotes: 0

Related Questions