remove unnecessary code from patch before in ModelAdmin inlines save.

[Maxim Danilov]

In #33111, the following code was added:

form = ModelForm(request.POST, request.FILES, instance=obj)
formsets, inline_instances = self._create_formsets(request, form.instance if add else obj, change=not add)

it is all ok, but if we have:

form = ModelForm(request.POST, request.FILES, instance=obj)

this is superabundant:

form.instance if add else obj

I think, it should be:

form = ModelForm(request.POST, request.FILES, instance=obj)
formsets, inline_instances = self._create_formsets(request, form.instance, change=not add)
# or # formsets, inline_instances = self._create_formsets(request, obj, change=not add)

The same changes is possible to made in code-block for request.GET

[Tim Graham]

I don't see any test failures with the following patch, so if it's needed, perhaps a regression test would be useful.

django/contrib/admin/options.py

@@ -1787 +1787 @@
             form = ModelForm(request.POST, request.FILES, instance=obj)
             formsets, inline_instances = self._create_formsets(
                 request,
-                form.instance if add else obj,
+                form.instance,
                 change=not add,
             )
             form_validated = form.is_valid()