“ user_id”列中的null值违反了非null约束Django

Question

I know there are a lot of solutions for this problem but they seem a bit different than mine. Here is my models.py:

from __future__ import unicode_literals

from django.db import models
from django.conf import settings
from django.contrib.auth.models import User
from django.contrib.postgres.fields import HStoreField
# Create your models here.

class Events(models.Model):
    user = models.ForeignKey(settings.AUTH_USER_MODEL)
    name = models.CharField(max_length=32)
    start = models.CharField(max_length=32)
    end = models.CharField(max_length=32)

Very simple table and I want the primary key in Auth_user to be the foreign key in my Events table. So of course this would mean that a User has to be logged in and authenticated for this to work. In my views.py I have:

def createEvent(request):
    if request.method == "POST":
        user = request.user
        print (user)        # Check for Authentication
        name = request.POST['name']
        start = request.POST['start']
        end = request.POST['end']
        Events.objects.create(
            name = name,
            start = start,
            end =end,
        )

The print statement will print out the current user logged in. I can confirm that this part does show that a user is logged in and that this user is in the auth_user table with a unique id . However, when I try to submit a form, I get a null value for the user column. Any ideas?

Answer 1

Just add the user instance because it's required by Events . I think you should use DateField or DateTimeField on start and end field. Not sure but I think you plan to put a date value there.

Events.objects.create(, name=name, start=start, end=end)

You should also add a @login_required for your function.

Answer 2

My solution is this:

    from django.contrib.auth.models import User
    user = request.user
    
    name = request.POST['name']
    start = request.POST['start']
    end = request.POST['end']
    Events.objects.create(
        
        name = name,
        start = start,
        end =end,
    )

Is that an alright solution? Could there be something better?

Answer 3

Making what you have right now work :

def createEvent(request):
    if request.method == "POST":
        user = request.user
        print (user)        # Check for Authentication
        name = request.POST['name']
        start = request.POST['start']
        end = request.POST['end']
        # The user has to be included before being saved
        Events.objects.create(name=name, start=start, end=end, )

Some better practices: Using a ModelForm (note there are class based generic views that make this easier too), and the login_required decorator

# forms.py
from django.forms import ModelForm
from .models import Event

class EventForm(ModelForm):
    class Meta:
        model = Event
        fields = ('user', 'start', 'end', 'name')

# views.py
from django.contrib.auth.decorators import login_required

@login_required
def create_event(request):
    if request.POST:
        form = EventForm(request.POST)
        if form.is_valid():
            form.save()
    else:
        form = EventForm()
        return render(..., {'form': form})