CODEWITHSUNDEEP
pythondjango
user298519
user298519

Reputation: 1190

Django: KeyError triggered in forms.py

Making a card app. User can make a deck and put cards in that deck. Decks and cards have an 'owner' field in their models to state who the user is.

forms.py

class CardForm(forms.ModelForm):
  def __init__(self, *args, **kwargs):
    # Pop() removes 'user' from the kwargs dictionary and populates the user variable
    user = kwargs.pop('owner') 
    super(CardForm, self).__init__(*args, **kwargs)
    self.fields['deck'] = forms.ModelChoiceField( # modify choices on 'deck' field
        queryset=Deck.objects.filter(owner=user)
  )

class Meta:
    model = Card 
    fields = ('term', 'definition', 'deck')

The part triggering the KeyError is

super(CardForm, self).__init__(*args, **kwargs)

views.py

def card_new(request, deck):
  if request.method == "POST":
      form = CardForm(request.POST)
      if form.is_valid():
          card = form.save(commit=False)
          card.save()
          return redirect('card:detail', deck)
  else:
      form = CardForm(initial={'deck': deck}, owner=request.user) # this initial field sets card's deck as current deck
  return render(request, 'card/card_edit.html', {'form': form})

models.py

class Card(models.Model):
  owner = models.ForeignKey(User, null=True, default=True, related_name='oc')
  term = models.CharField(max_length=100, default='N/A')
  definition = models.TextField(default='N/A')
  deck = models.ForeignKey(Deck, on_delete=models.CASCADE)

Upvotes: 1

Views: 274

Answers (1)

Alasdair
Alasdair

Reputation: 308779

You should pass owner for POST requests, as you already do for GET requests.

if request.method == "POST":
    form = CardForm(request.POST, owner=request.user)

Upvotes: 1

Related Questions