Opened 3 years ago

Closed 2 years ago

Last modified 10 months ago

#34547 closed Cleanup/optimization (fixed)

Deprecate DatabaseOperations.field_cast_sql().

Reported by: Mariusz Felisiak Owned by: David Smith
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords:
Cc: David Sanders Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

DatabaseOperations.field_cast_sql() is noop for all builtin backends since 1586a09b7949bbb7b0d84cb74ce1cadc25cbb355. I think we can deprecate it in favor of DatabaseOperations.lookup_cast() which is more flexible.

Change History (7)

comment:1 by Natalia Bidart, 3 years ago

Triage Stage: Unreviewed → Accepted

Accepting following the PR ​comments.

comment:2 by David Smith, 2 years ago

Owner: changed from nobody to David Smith
Status: new → assigned

comment:3 by Mariusz Felisiak, 2 years ago

Has patch: set
Patch needs improvement: set

​PR

comment:4 by Mariusz Felisiak, 2 years ago

Patch needs improvement: unset
Triage Stage: Accepted → Ready for checkin

comment:5 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

Resolution: → fixed
Status: assigned → closed

In 27b399d:

Fixed #34547 -- Deprecated DatabaseOperations.field_cast_sql().

comment:6 by nessita <124304+nessita@…>, 15 months ago

In a69f895d:

Refs #34547 -- Adjusted deprecation warning stacklevel in DatabaseOperations.field_cast_sql().

comment:7 by Sarah Boyce <42296566+sarahboyce@…>, 10 months ago

In 6b271ef2:

Refs #34547 -- Removed DatabaseOperations.field_cast_sql() per deprecation timeline.

Note: See TracTickets for help on using tickets.
Back to Top