Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

django two ModelForms with same field name on one template

I have two models and both have field 'status' which has different meaning for them.

class Order(models.Model):
    ...
    status = models.PositiveIntegerField(default=0, choices=ORDER_STATUSES)
    ...

class ACS(models.Model):
    status = models.IntegerField(default=-1, choices=STATUSES)
    order = models.ForeignKey(Order, blank=True, null=True)
    ...

Their forms looks like:

class ACSForm(forms.ModelForm):
    status = forms.ChoiceField(
        choices=STATUSES,
        widget=forms.Select(attrs={'class': 'form-control'})
    )
    ...

    class Meta:
       model = ACS
       fields = ('status',)

class OrderACSEditForm(forms.ModelForm):
    status = forms.ChoiceField(
        choices=ORDER_STATUSES,
        widget=forms.Select(attrs={'class': 'form-control'})
    )

    class Meta:
        model = Order
        fields = ('status',)

I want to edit both this fields on the same page. My view.py looks like

def edit(request, item_id=""):
    data = ACS.objects.get(pk=item_id)
    form = ACSForm(instance=data)
    order = Order.objects.get(id=data.order.id)
    form_edit = OrderACSEditForm(instance=order)
    if request.POST:
        form = ACSForm(request.POST, instance=data)
        form_edit = OrderACSEditForm(request.POST)
        if form.is_valid() and form_edit.is_valid():
            form_edit.save()
            obj = form.save()
            messages.add_message(request, messages.SUCCESS, 'Your data successfully saved.')

        if request.POST['action'] == "save_stay":
            return redirect("/panel/packages/acs/edit/" + str(obj.id))
        else:
            return redirect("/panel/packages/acs/")
    return render(request, 'ui/packages/acs/edit.html', dict(data=data, form=form, form_edit=form_edit, item_id=item_id))

And template:

<div class="form-group {% if form.status.errors %}has-error{% endif %}">
    <label>{% trans "Status" %}</label>
        {% if form.status.errors %}
            {% for error in form.status.errors %}
                <label class="control-label">{{ error }}</label>
            {% endfor %}
        {% endif %}
        {{ form.status }}
</div>

<div class="form-group {% if form_edit.status.errors %}has-error{% endif %}">
    <label>{% trans "Order status" %}</label>
        {% if form_edit.status.errors %}
            {% for error in form_edit.status.errors %}
                <label class="control-label">{{ error }}</label>
            {% endfor %}
        {% endif %}
        {{ form_edit.status }}
</div>

But in result form.status gets values from form_edit.status which is not correct. I need to solve this problem without changing names of model fields but don't know how.

like image 607
Petya Kostyukevich Avatar asked Oct 05 '15 10:10

Petya Kostyukevich


1 Answers

Use the prefix argument for your forms, to namespace the field names.

form = ACSForm(prefix='acs', instance=data)
form_edit = OrderACSEditForm(prefix='edit', instance=order)

Remember to use the same prefix when you instantiate the form with POST data as well.

like image 96
Alasdair Avatar answered Oct 11 '22 23:10

Alasdair