Re: [Django] #34015: Registering lookups on relation fields should be supported.

29 views
Skip to first unread message

Django

unread,
Sep 19, 2022, 9:29:59 AM9/19/22
to django-...@googlegroups.com
#34015: Registering lookups on relation fields should be supported.
-------------------------------------+-------------------------------------
Reporter: Thomas | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Normal | Resolution:
Keywords: ORM lookup | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Simon Charette):

I'm not sure I understand the rationale behind making this a release
blocker as lookups on related fields were never supported? I remember
having to remove this check a while ago when trying to add an `__exists`
lookup to work around #10060 in a feature branch.

I think this a limitation we should lift with proper test coverage but I
fail to see how it relates to 10178197d57476f69688d4535e550a1ea3a5eac5

--
Ticket URL: <https://code.djangoproject.com/ticket/34015#comment:9>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Sep 20, 2022, 7:44:50 AM9/20/22
to django-...@googlegroups.com
#34015: Registering lookups on relation fields should be supported.
-------------------------------------+-------------------------------------
Reporter: Thomas | Owner: Mariusz
| Felisiak
Type: New feature | Status: assigned

Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Normal | Resolution:
Keywords: ORM lookup | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* owner: nobody => Mariusz Felisiak
* status: new => assigned
* has_patch: 0 => 1


Comment:

[https://github.com/django/django/pull/16076 PR]

Agreed, it's not a release blocker.

--
Ticket URL: <https://code.djangoproject.com/ticket/34015#comment:10>

Django

unread,
Sep 21, 2022, 6:17:56 PM9/21/22
to django-...@googlegroups.com
#34015: Registering lookups on relation fields should be supported.
-------------------------------------+-------------------------------------
Reporter: Thomas | Owner: Mariusz
| Felisiak
Type: New feature | Status: closed

Component: Database layer | Version: 4.1
(models, ORM) |
Severity: Normal | Resolution: fixed

Keywords: ORM lookup | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by GitHub <noreply@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"ce6230aa976e8d963226a3956b45a8919215dbd8" ce6230a]:
{{{
#!CommitTicketReference repository=""
revision="ce6230aa976e8d963226a3956b45a8919215dbd8"
Fixed #34015 -- Allowed filtering by transforms on relation fields.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/34015#comment:11>

Reply all
Reply to author
Forward
0 new messages