Форма обновления создает новый объект в Django - PullRequest
Новая
       2

Форма обновления создает новый объект в Django

1 голос
/ 08 марта 2020

У меня есть модель Trip и я создал форму, чтобы она обновляла существующие поездки.

models.py: 

class Trip(models.Model):
    trip_id = models.CharField(max_length=20, verbose_name="Ref. Viagem")
    destination = models.CharField(max_length=200, null=True, verbose_name='Destino')
    client = models.ForeignKey(Clients, null=True, on_delete=models.CASCADE, verbose_name="Cliente")
    out_flight = models.ForeignKey(Flight, related_name="outbound_flight" ,null=True, on_delete=models.SET_NULL, verbose_name="Voo Ida")
    hotel = models.ForeignKey(Hotels, null=True, on_delete=models.SET_NULL, verbose_name="Hotel")
    in_flight = models.ForeignKey (Flight, related_name="inbound_flight", null=True, on_delete=models.SET_NULL, verbose_name="Voo Regresso")

forms.py: 

class UpdateTrip(ModelForm):

    def __init__(self, *args, **kwargs):
        super(UpdateTrip, self).__init__(*args, **kwargs)
        self.fields['trip_id'].widget = TextInput(attrs={'class': 'form-control'})
        self.fields['destination'].widget = TextInput(attrs={'class': 'form-control'})
        self.fields['client'].queryset = Clients.objects.all()


    class Meta:
        model = Trip
        fields = ('trip_id', 'destination', 'client', 'out_flight', 'hotel', 'in_flight')

И это views.py для этого: 

def trip_upd(request, trip_id):
    if request.method == 'POST':
        form = UpdateTrip(request.POST)
        if form.is_valid():
            form.save()
            return redirect('trips')
    else:
        form = UpdateTrip()
    return render(request, 'backend/trip_update.html', {'form': form})

Я использую форму barebones при тестировании: 

<form method="post">
    {% csrf_token %}
    {{ form }}
    <input type="submit" value="Submit">
</form>

Теперь это не обновляет объекты, а создает новый.

1 Ответ

0 голосов
/ 08 марта 2020

Вам необходимо передать экземпляр, который вы хотите обновить, с помощью параметра instance=&hellip;: 

def trip_upd(request, trip_id):
    if request.method == 'POST':
        <b>trip =</b> Trip.objects.get(trip_id=trip_id)
        form = UpdateTrip(request.POST<b>, instance=trip</b>)
        if form.is_valid():
            form.save()
            return redirect('trips')
    else:
        form = UpdateTrip()
    return render(request, 'backend/trip_update.html', {'form': form})

или вы можете работать с get_object_or_404 функцией [Django -doc] для возврата ошибки HTTP 404, если объект не существует: 

from django.shortcuts import get_object_or_404

def trip_upd(request, trip_id):
    if request.method == 'POST':
        trip = <b>get_object_or_404(</b>Trip, trip_id=trip_id<b>)</b>
        form = UpdateTrip(request.POST, instance=trip)
        if form.is_valid():
            form.save()
            return redirect('trips')
    else:
        form = UpdateTrip()
    return render(request, 'backend/trip_update.html', {'form': form})

Обратите внимание, что в форме вы можете указать виджеты в Meta, что более декларативный, чем запись в __init__: 

class UpdateTrip(ModelForm):

    class Meta:
        model = Trip
        fields = ('trip_id', 'destination', 'client', 'out_flight', 'hotel', 'in_flight')
        <b>widgets</b> = {
            'trip_id': TextInput(attrs={'class': 'form-control'}),
            'destination': TextInput(attrs={'class': 'form-control'})
        }
...