#34566: ModelAdmin get_field_queryset uses related admin ordering, but not
related
admin querysets.
-------------------------------+------------------------------------
Reporter: Mike J | Owner: nobody
Type: Bug | Status: new
Component: contrib.admin | Version: 4.2
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1
Needs tests: 1 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------+------------------------------------
Changes (by Mariusz Felisiak):
* cc: fisadev, Ramiro Morales (added)
Comment:
Replying to [comment:6 Natalia Bidart]:
> Replying to [comment:5 Mariusz Felisiak]:
> > This would be definitely backward incompatible. I'd start by checking
why `get_ordering()` was added like this in
d9330d5be2ee60b208dcab2616eb164ea2e6bf36.
>
> Do you have any suggestion about how to check why `get_ordering` was
added the way it was? Shall we contact the contributors? Shall we ask the
merger? Thanks!
Not sure why, maybe `get_queryset()` didn't exist then and the default
manager was the most reasonable choice, shrug. It's hard to tell. This
code hasn't changed in the last 10 years so we should be very careful. We
can try to ping Ramiro and Juan.
--
Ticket URL: <https://code.djangoproject.com/ticket/34566#comment:7>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/0107018825b38039-7892e6f0-1479-4fd1-b44c-f4d8480a2f9e-000000%40eu-central-1.amazonses.com.
