18

I'm trying to build a page for an inventory system that will allow a user to update a quantity of items received.

I want to show a table of all products and let the user enter the quantity received, which I'll post and iterate over to update the database.

Here is my view:

def new_shipment(request):
    list_of_active_products = Product.objects.filter(status=1)
    ShipmentFormSet = formset_factory(ShipmentForm, extra=0)
    formset = ShipmentFormSet(initial=list_of_active_products)
    return render_to_response('inventory/new_shipment.html', {'formset': formset})

Here's my model for the form:

class ShipmentForm(forms.Form):
    sku = forms.IntegerField()
    product_name = forms.CharField(max_length=100)
    quantity = forms.IntegerField()

And here is the form template:

<form method="post" action="">
    <table>
        {% for form in formset %}
    {{ form }}
    {% endfor %}
    </table>    
    <input type="submit" />
</form>

And here is the error I'm getting:

Caught AttributeError while rendering: 'Product' object has no attribute 'get'

Can anyone help me out with this?

Shawn Inman
  • 331
  • 1
  • 3
  • 8

2 Answers2

19

From the docs it looks like you have to pass in a list of dictionaries as the initial data, rather than a QuerySet:

Also note that we are passing in a list of dictionaries as the initial data.

You may want to change your initial query to:

list_of_active_products = Product.objects.filter(status=1).values()

which will return a list of dictionaries rather than model-instance objects.

Using initial data with a formset: https://docs.djangoproject.com/en/dev/topics/forms/formsets/#using-initial-data-with-a-formset

ValuesQuerySet: https://docs.djangoproject.com/en/dev/ref/models/querysets/#django.db.models.query.QuerySet.values

greggreg
  • 11,945
  • 6
  • 37
  • 52
rolling stone
  • 12,668
  • 9
  • 45
  • 63
  • 2
    That did it. I was unaware of the values() function, so thanks so much for the heads-up. Back to work... – Shawn Inman Jul 17 '11 at 02:21
  • 1
    Thanks a lot. You misspelled the function in the example though. It should say `list_of_active_products = Product.objects.filter(status=1).values()` – DMunoz Jul 24 '13 at 15:18
15

You can also use the queryset argument. This should work:

formset = ShipmentFormSet(queryset=list_of_active_products)

cf. https://docs.djangoproject.com/en/dev/topics/forms/modelforms/#changing-the-queryset

gellej
  • 369
  • 2
  • 6
  • 5
    The OP uses formset_factory, while queryset is only possible with modelformset_factory. At least, in some Django version, your code will not work – Edgar Navasardyan Jul 06 '16 at 10:12