Re: [Django] #34012: QuerySet.order_by() silently skips non-existing fields on related fields with Meta.ordering.

2022-10-04 Thread Django 
#34012: QuerySet.order_by() silently skips non-existing fields on related fields
with Meta.ordering.
-+-
 Reporter:  Klaas van Schelven   |Owner:  David
 Type:   |  Sanders
  Cleanup/optimization   |   Status:  closed
Component:  Database layer   |  Version:  4.1
  (models, ORM)  |
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak ):

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


Comment:

 In [changeset:"4771a1694b3b54c7309602820881d3ec9cc2c809" 4771a169]:
 {{{
 #!CommitTicketReference repository=""
 revision="4771a1694b3b54c7309602820881d3ec9cc2c809"
 Fixed #34012 -- Made QuerySet.order_by() apply transforms on related
 fields for models with Meta.ordering.

 This makes QuerySet.order_by() no longer ignore trailing transforms for
 models with Meta.ordering. As a consequence, FieldError is raised in
 such cases for non-existent fields.

 Thanks to Klaas van Schelven for the report and Mariusz Felisiak for the
 review and advice.
 }}}

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070183a1c27a61-7b2feadf-f8dc-4775-9473-1b63bfdeb95d-00%40eu-central-1.amazonses.com.

Re: [Django] #34012: QuerySet.order_by() silently skips non-existing fields on related fields with Meta.ordering.

2022-10-03 Thread Django 
#34012: QuerySet.order_by() silently skips non-existing fields on related fields
with Meta.ordering.
-+-
 Reporter:  Klaas van Schelven   |Owner:  David
 Type:   |  Sanders
  Cleanup/optimization   |   Status:  assigned
Component:  Database layer   |  Version:  4.1
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * needs_better_patch:  1 => 0
 * stage:  Accepted => Ready for checkin


-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070183a11fde81-710e2bff-bd8e-40f8-bb28-4c8e0bdb9a65-00%40eu-central-1.amazonses.com.

Re: [Django] #34012: QuerySet.order_by() silently skips non-existing fields on related fields with Meta.ordering.

2022-09-22 Thread Django 
#34012: QuerySet.order_by() silently skips non-existing fields on related fields
with Meta.ordering.
-+-
 Reporter:  Klaas van Schelven   |Owner:  David
 Type:   |  Sanders
  Cleanup/optimization   |   Status:  assigned
Component:  Database layer   |  Version:  4.1
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by GitHub ):

 In [changeset:"37a13cc92d51d5a57b42240b9a5dd250648efa24" 37a13cc9]:
 {{{
 #!CommitTicketReference repository=""
 revision="37a13cc92d51d5a57b42240b9a5dd250648efa24"
 Refs #34012 -- Added test for ordering by transform of related fields.
 }}}

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018364d717ff-34e7ddc4-1214-4e64-8c72-02811269-00%40eu-central-1.amazonses.com.

Re: [Django] #34012: QuerySet.order_by() silently skips non-existing fields on related fields with Meta.ordering.

2022-09-19 Thread Django 
#34012: QuerySet.order_by() silently skips non-existing fields on related fields
with Meta.ordering.
-+-
 Reporter:  Klaas van Schelven   |Owner:  David
 Type:   |  Sanders
  Cleanup/optimization   |   Status:  assigned
Component:  Database layer   |  Version:  4.1
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * needs_better_patch:  0 => 1


-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018354ef03e3-b2e93354-5f7b-456f-85f3-44ec08ef40b3-00%40eu-central-1.amazonses.com.

Re: [Django] #34012: QuerySet.order_by() silently skips non-existing fields on related fields with Meta.ordering. (was: QuerySet.order_by() silently skips non-existing fields on related fields.)

2022-09-19 Thread Django 
#34012: QuerySet.order_by() silently skips non-existing fields on related fields
with Meta.ordering.
-+-
 Reporter:  Klaas van Schelven   |Owner:  David
 Type:   |  Sanders
  Cleanup/optimization   |   Status:  assigned
Component:  Database layer   |  Version:  4.1
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

-- 
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018354ce7c1f-fa704b98-2ee9-43a7-bc9f-0029867510ab-00%40eu-central-1.amazonses.com.