Styjun

I am making a delivery note transaction form, I have created a formset for which I want Django to ignore item transactions where the item is not selected and is empty.

forms.py

class Delivery_note_transiction_form(forms.Form):

 item = forms.CharField(widget=Select2Widget(attrs="class" : "item"),label=False,required=False)
 description = forms.CharField(widget=forms.TextInput(attrs= 'placeholder' : 'optionall','class' : 'description'),label=False,required=False)
 quantity = forms.IntegerField(widget=forms.NumberInput(attrs='class' : 'quantity'),label=False,min_value=1)
 id = forms.CharField(widget=forms.HiddenInput,required=False)

Delivery_note_transiction_form_formset = forms.formset_factory(Delivery_note_transiction_form,extra=1)

views.py

def feedback(request):
 if request.method == "POST" and request.is_ajax():
 form = Deliver_Note_Main_Modelform(request.POST)
 formset = Delivery_note_transiction_form_formset(request.POST,request.FILES)
 if form.is_valid() and formset.is_valid():

 ins = form.save(commit=False)
 ins.author = request.user

 result = Customer_data.objects.get(pk=form.cleaned_data['customer'])

 ins.customer_list = result
 ins.save()


 max_invoice = Invoice_max.objects.get(invoice_name='delivery')
 max_invoice.invoice_no = max_invoice.invoice_no + 1
 max_invoice.save()

 print(formset)
 for instant in formset:
 if instant.cleaned_data['item']:

 item = Product.objects.get(pk=instant.cleaned_data['item'])
 description = instant.cleaned_data['description']
 quantity = instant.cleaned_data['quantity']
 Delivery_Note_Transiction.objects.create(
 item=item,
 description=description,
 quantity=quantity,
 delivery_invoice_no=ins
 )

 return JsonResponse('success':True, 'next' : reverse_lazy('delivery note:delivery note home page'))

 else:
 return render(request,"delivery_note/ajax/delivery note error message.html","error" : form, "transiction_error": formset)
return HttpResponse("Hello from feedback!")

template.html

% for delivery in delivery_transiction %
 <tr class=" delivery_form ">
 <td class="col-sm-4"> delivery.item</td>
 <td class="col-sm-4">as_crispy_field </td>
 <td class="col-sm-4">as_crispy_field </td>
 </tr>
% endfor %

The post data is sent by Ajax and the selected option is created on the template. When it is loaded, a new row is added by Ajax. The problem is I want to it ignore transaction entry if the item is not selected or is empty, but when I run it, it gives this error:

"KeyError: 'item'"

It should ignore empty or not selected items. This only happens when the item is not selected in the transaction. I want to fix this error so that it will simply ignore rows in which the item is not selected.

You have to use a try except when searching a dictionary for a key.

for instant in formset:
 try:
 item = Product.objects.get(pk=instant.cleaned_data['item'])
 Except KeyError:
 # What to do if no 'item'.

You will have to figure out where to put the rest of your code, but this will get you past the KeyError.