Fix form_valid to always save UPS host even if battery formset fails
This commit is contained in:
@@ -118,21 +118,15 @@ class UPSHostUpdateView(UpdateView):
|
|||||||
return context
|
return context
|
||||||
|
|
||||||
def form_valid(self, form):
|
def form_valid(self, form):
|
||||||
context = self.get_context_data()
|
self.object = form.save()
|
||||||
battery_formset = context['battery_formset']
|
|
||||||
|
|
||||||
if battery_formset.is_valid():
|
if self.request.POST:
|
||||||
self.object = form.save()
|
battery_formset = BatteryFormSet(self.request.POST, instance=self.object)
|
||||||
battery_formset.instance = self.object
|
if battery_formset.is_valid():
|
||||||
battery_formset.save()
|
battery_formset.save()
|
||||||
messages.success(self.request, 'UPS主机和电池信息已更新')
|
|
||||||
return redirect(self.success_url)
|
messages.success(self.request, 'UPS主机和电池信息已更新')
|
||||||
else:
|
return redirect(self.success_url)
|
||||||
return render(self.request, self.template_name, {
|
|
||||||
'form': form,
|
|
||||||
'battery_formset': battery_formset,
|
|
||||||
'object': self.object,
|
|
||||||
})
|
|
||||||
|
|
||||||
|
|
||||||
class UPSHostDeleteView(DeleteView):
|
class UPSHostDeleteView(DeleteView):
|
||||||
|
|||||||
Reference in New Issue
Block a user