BaseGenericInlineFormSet.get_queryset calls .empty()

[Zak Johnson <zakj@…>]

In BaseGenericInlineFormSet.get_queryset():

        if self.instance is None:
            return self.model._default_manager.empty()

Managers don't have an empty() method; this should instead be get_empty_query_set(). Attached patch (against trunk revision 9836) includes a doctest to illustrate the problem.

[Brian Rosner]

Wow, that must have been a brain dead commit by me. You are correct empty() is not on a manager, but none() would have been correct here. Thanks for the patch!

[Alex Gaynor]

I think what you really want is .none()

[Antti Kaihola]

patch and tests with none() instead of empty()

[Brian Rosner]

(In [9886]) Fixed #10274 -- Fixed an embarrassing mistake in BaseGenericInlineFormSet.get_queryset. Thanks Zak Johnson for the report and akaihola for the patch.

[Brian Rosner]

(In [9887]) [1.0.X] Fixed #10274 -- Fixed an embarrassing mistake in BaseGenericInlineFormSet.get_queryset. Thanks Zak Johnson for the report and akaihola for the patch.

Backport of r9886 from trunk.