Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#26285 closed Cleanup/optimization (fixed)

Deprecate the MySQL specific __search lookup

Reported by: Tim Graham Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Found the implementation in Marc Taymlyn's ​branch for PostgreSQL full text text. It should have a separate ticket.

Change History (4)

comment:1 by Tim Graham, 8 years ago

Has patch: set

​PR. As noted there, a remaining problem is that the __search lookup is ​used by the admin
for the search_fields = ['@fieldname'] syntax. I think #26184 would allow removing that code.

If we don't want to make that ticket a prerequisite of this one, I suppose we could leave that code for now, even if the search lookup is removed from Django itself.

comment:2 by Tim Graham <timograham@…>, 8 years ago

Resolution: → fixed
Status: new → closed

In 8ddc79a:

Fixed #26285 -- Deprecated the MySQL-specific search lookup.

comment:3 by Tim Graham <timograham@…>, 7 years ago

In 5139832:

Refs #26285 -- Removed MySQL search lookup per deprecation timeline.

comment:4 by Tim Graham <timograham@…>, 7 years ago

In 0a0c245:

Refs #26285 -- Removed postgresql's unused DatabaseOperations.fulltext_search_sql().

Should have been removed in 5139832398624be75ee5361a6fac9348fdb61093.

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