BaseYearArchiveView prevent ordering of queryset

[Thibault Serot <thibserot@…>]

In BaseYearArchiveView, the get_dated_items calls the get_dated_queryset with an extra parameter ordering="-%s" : qs = self.get_dated_queryset(ordering='-%s' % date_field, * * lookup_kwargs)

This behavior is not the same as in BaseMonthArchiveView (which do qs = self.get_dated_queryset(* * lookup_kwargs))
and prevents from using custom filtering in get_queryset.

Could you please remove the extra ordering="-%s" in the BaseYearArchiveView and let the user handle the filtering?

Thanks a lot

[Thibault Serot <thibserot@…>]

Hello,
I actually see that there is also a default filtering + ordering in BaseArchiveIndexView:

qs = self.get_dated_queryset(ordering='-%s' % self.get_date_field())
date_list = self.get_date_list(qs, ordering='DESC')

Should be replaced by:

qs = self.get_dated_queryset()
date_list = self.get_date_list()

[Tim Graham]

Is the current behavior documented? I see some ​references to ordering in the docs. Would this be a backwards incompatible change for people relying on the current behavior? If we can call it a bug, we may be able to change it (and sooner rather than later would be best). Can you submit a patch with a test that demonstrates the new functionality that we'd be adding?

[Aymeric Augustin]

#18355 looks related.

[Tim Graham]

Accepting on the basis of the problem, not sure about the proposed solution at this point.

[pjrharley]

Replying to aaugustin:

#18355 looks related.

I've added a PR there. I think it looks like it might close this ticket too if accepted. By default the behaviour is the same but the user can now easily use custom filtering (without overriding get_queryset either).

[Tim Graham <timograham@…>]

In 2724cdbff635a40819f206411de23e9b14867a58:

Fixed #18355 -- Added ordering options to list based generic views.

Added MultipleObjectMixin.ordering and get_ordering().

Refs #21450.

[Tim Graham]

Closing per above commit as I believe it solves it, but please reopen if not.