django saving model with formset

Apostolos picture Apostolos · Feb 16, 2015 · Viewed 9.5k times · Source

I have the following view code

account = PatientAccount.objects.get(pk=account_id)
payments = Payments.objects.filter(account=account)
total_cost = 0
for hasjob in account.hasjobs.all().select_related('job'):
    total_cost += hasjob.quantity * hasjob.job.cost*(1.0 - hasjob.discount / 100.0)

PaymentFormset = modelformset_factory(Payments, form=PaymentModelForm, can_delete=False)
formset = PaymentFormset(request.POST or None, queryset=payments)

if request.method == 'POST':
    print formset.is_valid()
    if formset.is_valid():
        for form in formset:
            if form.is_valid():
                try:
                    if form.cleaned_data.get('DELETE') and form.instance.pk:
                        form.instance.delete()
                    else:
                        instance = form.save(commit=False)
                        print instance.pk
                        instance.account = account
                        print instance.account
                        instance.save()
                        messages.success(request, "Payments saved successfully")
                except DatabaseError:
                    messages.error(request, "Database error. Please try again")

                return redirect('job_management-edit', account_id=account.pk)

But It won't save the models. Everything works as it should formset is valid form is valid.

Answer

Ankit Popli picture Ankit Popli · Feb 16, 2015

The problem seems to be that you are returning the response from inside the for loop.

Try this instead:

account = PatientAccount.objects.get(pk=account_id)
payments = Payments.objects.filter(account=account)
total_cost = 0
for hasjob in account.hasjobs.all().select_related('job'):
    total_cost += hasjob.quantity * hasjob.job.cost*(1.0 - hasjob.discount / 100.0)

PaymentFormset = modelformset_factory(Payments, form=PaymentModelForm, can_delete=False)
formset = PaymentFormset(request.POST or None, queryset=payments)

if request.method == 'POST':
    print formset.is_valid()
    if formset.is_valid():
        for form in formset:
            if form.is_valid():
                try:
                    if form.cleaned_data.get('DELETE') and form.instance.pk:
                        form.instance.delete()
                    else:
                        instance = form.save(commit=False)
                        print instance.pk
                        instance.account = account
                        print instance.account
                        instance.save()
                        messages.success(request, "Payments saved successfully")
                except DatabaseError:
                    messages.error(request, "Database error. Please try again")

        return redirect('job_management-edit', account_id=account.pk)