From 831d363141705adf24bd33a4e15e1132e41358f0 Mon Sep 17 00:00:00 2001 From: Geoffrey Fairchild Date: Thu, 25 Jun 2015 17:01:41 -0600 Subject: [PATCH] fixing Django 1.9 deprecations --- nested_inline/admin.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/nested_inline/admin.py b/nested_inline/admin.py index 6793771..f62ce96 100644 --- a/nested_inline/admin.py +++ b/nested_inline/admin.py @@ -165,7 +165,7 @@ def add_view(self, request, form_url='', extra_context=None): form_validated = False new_object = self.model() prefixes = {} - for FormSet, inline in zip(self.get_formsets(request), inline_instances): + for FormSet, inline in self.get_formsets_with_inlines(request): prefix = FormSet.get_default_prefix() prefixes[prefix] = prefixes.get(prefix, 0) + 1 if prefixes[prefix] != 1 or not prefix: @@ -195,7 +195,7 @@ def add_view(self, request, form_url='', extra_context=None): initial[k] = initial[k].split(",") form = ModelForm(initial=initial) prefixes = {} - for FormSet, inline in zip(self.get_formsets(request), inline_instances): + for FormSet, inline in self.get_formsets_with_inlines(request): prefix = FormSet.get_default_prefix() prefixes[prefix] = prefixes.get(prefix, 0) + 1 if prefixes[prefix] != 1 or not prefix: @@ -227,7 +227,7 @@ def add_view(self, request, form_url='', extra_context=None): context = { 'title': _('Add %s') % force_text(opts.verbose_name), 'adminform': adminForm, - 'is_popup': "_popup" in request.REQUEST, + 'is_popup': "_popup" in request.GET, 'show_delete': False, 'media': media, 'inline_admin_formsets': inline_admin_formsets, @@ -269,7 +269,7 @@ def change_view(self, request, object_id, form_url='', extra_context=None): form_validated = False new_object = obj prefixes = {} - for FormSet, inline in zip(self.get_formsets(request, new_object), inline_instances): + for FormSet, inline in self.get_formsets_with_inlines(request, new_object): prefix = FormSet.get_default_prefix() prefixes[prefix] = prefixes.get(prefix, 0) + 1 if prefixes[prefix] != 1 or not prefix: @@ -291,7 +291,7 @@ def change_view(self, request, object_id, form_url='', extra_context=None): else: form = ModelForm(instance=obj) prefixes = {} - for FormSet, inline in zip(self.get_formsets(request, obj), inline_instances): + for FormSet, inline in self.get_formsets_with_inlines(request, obj): prefix = FormSet.get_default_prefix() prefixes[prefix] = prefixes.get(prefix, 0) + 1 if prefixes[prefix] != 1 or not prefix: @@ -325,7 +325,7 @@ def change_view(self, request, object_id, form_url='', extra_context=None): 'adminform': adminForm, 'object_id': object_id, 'original': obj, - 'is_popup': "_popup" in request.REQUEST, + 'is_popup': "_popup" in request.GET, 'media': media, 'inline_admin_formsets': inline_admin_formsets, 'errors': helpers.AdminErrorList(form, formsets), @@ -364,9 +364,9 @@ def get_inline_instances(self, request, obj=None): inline_instances.append(inline) return inline_instances - def get_formsets(self, request, obj=None): + def get_formsets_with_inlines(self, request, obj=None): for inline in self.get_inline_instances(request): - yield inline.get_formset(request, obj) + yield inline.get_formset(request, obj), inline class NestedStackedInline(NestedInline):