From 0fbac14d3ca69489b47eef8e7f5a1ab5ea8f9738 Mon Sep 17 00:00:00 2001 From: Karl Hobley Date: Fri, 8 Jun 2018 15:19:34 +0100 Subject: [PATCH] Remove shortcut queries --- wagtail/contrib/postgres_search/backend.py | 4 +--- wagtail/search/backends/db.py | 4 +--- wagtail/search/query.py | 12 ------------ 3 files changed, 2 insertions(+), 18 deletions(-) diff --git a/wagtail/contrib/postgres_search/backend.py b/wagtail/contrib/postgres_search/backend.py index 9fe0091b3..6ea414919 100644 --- a/wagtail/contrib/postgres_search/backend.py +++ b/wagtail/contrib/postgres_search/backend.py @@ -11,7 +11,7 @@ from django.utils.encoding import force_text from wagtail.search.backends.base import ( BaseSearchBackend, BaseSearchQueryCompiler, BaseSearchResults) from wagtail.search.index import RelatedFields, SearchField, get_indexed_models -from wagtail.search.query import And, Boost, MatchAll, Not, Or, PlainText, Prefix, SearchQueryShortcut, Term +from wagtail.search.query import And, Boost, MatchAll, Not, Or, PlainText, Prefix, Term from wagtail.search.utils import ADD, AND, OR from .models import SearchAutocomplete as PostgresSearchAutocomplete @@ -248,8 +248,6 @@ class PostgresSearchQueryCompiler(BaseSearchQueryCompiler): if isinstance(query, Boost): boost *= query.boost return self.build_database_query(query.subquery, config, boost=boost) - if isinstance(query, SearchQueryShortcut): - return self.build_database_query(query.get_equivalent(), config, boost=boost) if isinstance(query, Prefix): self.check_boost(query, boost=boost) self.is_autocomplete = True diff --git a/wagtail/search/backends/db.py b/wagtail/search/backends/db.py index 227132671..778656a92 100644 --- a/wagtail/search/backends/db.py +++ b/wagtail/search/backends/db.py @@ -5,7 +5,7 @@ from django.db.models.expressions import Value from wagtail.search.backends.base import ( BaseSearchBackend, BaseSearchQueryCompiler, BaseSearchResults) -from wagtail.search.query import And, Boost, MatchAll, Not, Or, PlainText, Prefix, SearchQueryShortcut, Term +from wagtail.search.query import And, Boost, MatchAll, Not, Or, PlainText, Prefix, Term from wagtail.search.utils import AND, OR @@ -77,8 +77,6 @@ class DatabaseSearchQueryCompiler(BaseSearchQueryCompiler): if isinstance(self.query, MatchAll): return models.Q() - if isinstance(query, SearchQueryShortcut): - return self.build_database_filter(query.get_equivalent(), boost=boost) if isinstance(query, Term): self.check_boost(query) return self.build_single_term_filter(query.term) diff --git a/wagtail/search/query.py b/wagtail/search/query.py index 9321c4930..413e71b26 100644 --- a/wagtail/search/query.py +++ b/wagtail/search/query.py @@ -54,13 +54,6 @@ class MultiOperandsSearchQueryOperator(SearchQueryOperator): yield from self.subqueries -class SearchQueryShortcut(SearchQuery): - def get_equivalent(self): - raise NotImplementedError - - def get_children(self): - yield self.get_equivalent() - # # Operators # @@ -149,9 +142,4 @@ class Fuzzy(SearchQuery): return func(self.__class__(self.term, self.max_distance, self.boost)) -# -# Shortcut query classes -# - - MATCH_ALL = MatchAll()