modelformset - django formmodel

Django formset: ¿cómo actualizar un objeto?

Question

¿Cómo puedo actualizar un objeto de un formset usando request.POST ?

Aquí está mi código y mi problema es que esto siempre crea un nuevo objeto PhoneNumber . Pero quiero actualizar el antiguo objeto PhoneNumber .

def contact_detail(request, contact_id):
    contact = get_object_or_404(Contact, pk=contact_id)
    phone_number_list = PhoneNumber.objects.filter(contact=contact_id)

    if request.method == ''POST'':
        cform = ContactForm(request.POST, instance=contact)
        #the next line is probably wrong!
        phonenumberformset = PhoneNumberFormSet(request.POST, queryset=phone_number_list)

        if cform.is_valid() and phonenumberformset.is_valid():
            phonenumber_instances = phonenumberformset.save(commit=False)
            for phonenumber in phonenumber_instances:
                phonenumber.contact = contact
                phonenumber.save()

            request.user.message_set.create(message=''The contact "%s" was chanced successfully.'' % contact.__str__())
            return HttpResponseRedirect("/crm/contacts/?oby=1")
    else:
        cform = ContactForm(instance=contact)
        phonenumberformset = PhoneNumberFormSet(queryset=phone_number_list)

    return render_to_response(
        ''crm/contact_detail.html'',
        {''cform'': cform, ''phonenumberformset'': phonenumberformset,},
        context_instance = RequestContext(request),
    )

Editar: creo tres PhoneNumberForms:

PhoneNumberFormSet = modelformset_factory(PhoneNumber, max_num=3, extra=3, exclude=(''contact'',))

Editar: la solución que usa inlineformset_factory:

@login_required
def contact_detail(request, contact_id):
    contact = get_object_or_404(Contact, pk=contact_id)
    PhoneNumberInlineFormSet = inlineformset_factory(Contact, PhoneNumber, max_num=3)

    if request.method == ''POST'':
        cform = ContactForm(request.POST, instance=contact)
        classificationformset = ClassificationInlineFormSet(request.POST, request.FILES, instance=contact)
        addressformset = AddressInlineFormSet(request.POST, request.FILES, instance=contact)
        phonenumberformset = PhoneNumberInlineFormSet(request.POST, request.FILES, instance=contact)
        if cform.is_valid() and phonenumberformset.is_valid():
            contact = cform.save()
            phonenumberformset.save()

            request.user.message_set.create(message=''The contact "%s" was chanced successfully.'' % contact.__str__())
            return HttpResponseRedirect("/crm/contacts/?oby=1")
    else:
        cform = ContactForm(instance=contact)
        phonenumberformset = PhoneNumberInlineFormSet(instance=contact)

return render_to_response(
        ''crm/contact_detail.html'',
        {''cform'': cform, ''phonenumberformset'': phonenumberformset,},
        context_instance = RequestContext(request),)

Este enfoque incluso agrega una casilla de verificación Eliminar a cada formulario en línea. Fácil y genial

Answer 1

En lugar de usar modelformset_factory , use inlineformset_factory ; consulte la documentación aquí . Lo siento, debería haberle señalado inicialmente.

A continuación, puede soltar las cosas del conjunto de consultas, ya que inlineformset_factory se encarga de eso, y simplemente pasa el argumento de la instance (que aquí se refiere al modelo principal , es decir, el objeto de Contact ). Tampoco necesitará iterar explícitamente para establecer el atributo de contacto en el guardado, ya que una vez más eso se solucionó.