initial value for ModelForm field not working

Question

I'm tring to set the user field as the logged-in user, but it's returning None. Here's my model:

class Post(models.Model): user = models.ForeignKey(User, blank=True, null=True) title = models.TextField(max_length=76) date = models.DateTimeField(auto_now=True) content = models.TextField(null=False, default='') image = models.FileField(null=True, blank=True) category = models.CharField(max_length=20, choices=CATEGORY_CHOICES, default='1')

my form:

class PostForm(forms.ModelForm): content = forms.CharField(widget=PagedownWidget) title = forms.TextInput(attrs={'placeholder': 'title'}) class Meta: model = Post fields = [ 'title', 'content', 'category', 'image', 'id', 'user' ]

and my view where users make a post using a form:

def post(request): allauth_login = LoginForm(request.POST or None) allauth_signup = SignupForm(request.POST or None) if request.user.is_authenticated(): data = {'user': request.user} form_post = PostForm(request.POST, request.FILES, initial=data) if form_post.is_valid(): category = form_post.cleaned_data['category'] for a, b in CATEGORY_CHOICES: if a == category: category = b form_post.save() return HttpResponseRedirect('/%s' % category) else: form_post = PostForm() context = { 'allauth_login': allauth_login, 'allauth_signup': allauth_signup, 'form_post': form_post } return render(request, 'post.html', context) else: return HttpResponseRedirect("/accounts/signup/")

When I actually render the form in my template, {{ obj.user }}returns None. Any idea why?

{% for obj in Post.objects.all() %} {{ obj.title }} #works {{ obj.content }} #works ... {{ obj.user }} #does not work {% endfor %}

I don't see where you're defining obj. Did you mean to put user in context? — Jessie
– Jessie, Commented Feb 6, 2017 at 5:20
No that's just the template {% for obj in Post.objects.all() %}. all the other fields work, e.g. {{ obj.title }}, only user doesn't work (renders as None) — Zorgan
– Zorgan, Commented Feb 6, 2017 at 5:24
Are you sure that your posts are actually associated with a user? — Jessie
– Jessie, Commented Feb 6, 2017 at 5:31
So you mean the obj is from DB querying, not the post form? Or you supposed saving post form first and then query it from DB with updated user field? Anyway, providing template code would be helpful. — YKL
– YKL, Commented Feb 6, 2017 at 5:32
@user2896976 isn't that the point of my ForeignKey(User) field? — Zorgan
– Zorgan, Commented Feb 6, 2017 at 5:33

neverwalkaloner · Accepted Answer · 2017-02-06 06:11:27Z

You are initializing form incorrectly. If request method is not POST form will contain errors.

Try this:

form_post = PostForm(request.POST or None, request.FILES or None, initial=data)

UPDATE

You can also try to do it this way. Remove user field from form and in view do this:

if form_post.is_valid(): instance = form_post.save(commit=False) instance.user = request.user instance.save()

@Zorgan add print(request.user) to the view to check current user name.
I just printed it under if request.user.is_authenticated(): and it prints the correct logged-in username

YKL · Accepted Answer · 2017-02-06 05:47:07Z

I think you mixed up with initial and bound form with data. In your case, I'm guessing you're trying to save post with valid user first, and then redirect to the page to show the new post with user.

So it should be:

def post(request): allauth_login = LoginForm(request.POST or None) allauth_signup = SignupForm(request.POST or None) if request.user.is_authenticated(): data = request.POST.copy() data['user'] = request.user form_post = PostForm(data, request.FILES) if form_post.is_valid(): category = form_post.cleaned_data['category'] for a, b in CATEGORY_CHOICES: if a == category: category = b form_post.save() return HttpResponseRedirect('/%s' % category) else: form_post = PostForm() context = { 'allauth_login': allauth_login, 'allauth_signup': allauth_signup, 'form_post': form_post } return render(request, 'post.html', context) else: return HttpResponseRedirect("/accounts/signup/")

For initial -- These values are only displayed for unbound forms: https://docs.djangoproject.com/en/1.10/ref/forms/api/#django.forms.Form.initial

Yeah i'm trying to save post with valid user first, and then redirect to the page to show the new post with user. However your code for some reason doesn't work, the form doesn't submit properly (stays on the same page)

Collectives™ on Stack Overflow

initial value for ModelForm field not working

2 Answers 2

2 Comments

1 Comment

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Related