django-фильтр возится с пустым полем
Я настраиваю фильтр django для фильтрации некоторых моих списков. Вот один из них, с пользовательской формой:
class BookingListFiltersForm(forms.Form):
state__in = forms.MultipleChoiceField(
choices=Booking.STATE_CHOICES, required=False,
label=_("État"), widget=forms.CheckboxSelectMultiple)
source__in = forms.ModelMultipleChoiceField(
queryset=Platform.objects.all(), required=False,
label=_("Source"), widget=ModelSelect2Multiple(
url='autocomplete:platform'))
class BookingManagerFilter(filters.FilterSet):
payments__date = filters.DateFilter(method='payments__date_filter')
payments__method = filters.ChoiceFilter(
method='payments__method_filter',
choices=BookingPayment.METHOD_CHOICES,
)
class Meta:
model = Booking
fields = {
'period': [
'endswith', 'endswith__gte', 'endswith__lte',
'startswith', 'startswith__gte', 'startswith__lte',
],
'state': ['in'],
'source': ['in'],
'booking_date': ['date', 'date__lte', 'date__gte'],
'accommodation': ['in'],
'guest': ['exact']
}
def get_form_class(self):
return BookingListFiltersForm
def payments__date_filter(self, queryset, name, value):
return queryset.filter(**{name: value})
def payments__method_filter(self, queryset, name, value):
return queryset.filter(**{name: value})
Форма представляется методом GET. Когда поле "source__in" пустое, запрос выглядит так: "state__in = 1". В этом случае у меня нет результата на моей странице (что неожиданно, если поле не заполнено, я бы ожидал, что результаты не будут отфильтрованы в этом поле).
Я просмотрел панель инструментов отладки, чтобы получить дополнительную информацию о выполненном SQL-запросе. Удивительно, но я не нашел SQL-запроса для соответствующего набора запросов! (в то время как если querystring является "? state__in = 1 & source__in = 2", например, результат будет таким, как ожидалось, и я могу найти связанные запросы в панели инструментов отладки)
Поэтому я попытался заставить впечатление SQL-запроса использовать print(str(filters.qs.query))
. Новый сюрприз: это вызвало исключение EmptyResultSet
:
Traceback:
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/core/handlers/exception.py" in inner
35. response = get_response(request)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/core/handlers/base.py" in _get_response
128. response = self.process_exception_by_middleware(e, request)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/core/handlers/base.py" in _get_response
126. response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/views/generic/base.py" in view
69. return self.dispatch(request, *args, **kwargs)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/utils/decorators.py" in _wrapper
62. return bound_func(*args, **kwargs)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/contrib/auth/decorators.py" in _wrapped_view
21. return view_func(request, *args, **kwargs)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/utils/decorators.py" in bound_func
58. return func.__get__(self, type(self))(*args2, **kwargs2)
File "/home/tony/Workspace/cocoonr/utils/views/manager.py" in dispatch
29. return super().dispatch(*args, **kwargs)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/views/generic/base.py" in dispatch
89. return handler(request, *args, **kwargs)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/views/generic/list.py" in get
142. self.object_list = self.get_queryset()
File "/home/tony/Workspace/cocoonr/booking/views/manager.py" in get_queryset
73. queryset = super().get_queryset()
File "/home/tony/Workspace/cocoonr/utils/views/common.py" in get_queryset
118. print(self.filters.qs.query)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/db/models/sql/query.py" in __str__
252. sql, params = self.sql_with_params()
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/db/models/sql/query.py" in sql_with_params
260. return self.get_compiler(DEFAULT_DB_ALIAS).as_sql()
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/db/models/sql/compiler.py" in as_sql
461. where, w_params = self.compile(self.where) if self.where is not None else ("", [])
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/db/models/sql/compiler.py" in compile
393. sql, params = node.as_sql(self, self.connection)
File "/home/tony/.venvs/cocoonr/lib/python3.6/site-packages/django/db/models/sql/where.py" in as_sql
98. raise EmptyResultSet
Exception Type: EmptyResultSet at /manager/booking/bookings/
Exception Value:
Теперь я застрял, я понятия не имею, что происходит не так и как отлаживать дальше.
Я попытался передать следующую цепочку для тестирования: "? State__in = 1 & source__in =". В этом случае фильтрация работает правильно, но в форме фильтра отображается ошибка "" "не является допустимым значением" для поля "source__in".
Кроме того, здесь находится соответствующий mixin в utils/views/common.py
:
class ListFilterMixin:
filters_class = None
default_filters = None
@cached_property
def filters(self):
return self.get_filters()
def get_filters(self):
if self.filters_class:
qstring = self.request.GET
if not qstring and self.default_filters:
qstring = QueryDict(self.default_filters)
return self.filters_class(
qstring, self.get_unfiltered_queryset(), request=self.request)
else:
return None
def get_queryset(self):
print(self.filters.qs.query) # <--- Line 118
# ...
def get_unfiltered_queryset(self):
return super().get_queryset()
А класс просмотра в booking/views/manager.py
:
class BookingListView(ListView):
"""List of all bookings."""
model = Booking
default_filters = 'state__in=1'
filters_class = BookingManagerFilter
paginate_by = 30
ordering = '-pk'
def get_queryset(self):
queryset = super().get_queryset() # <--- Line 73
# ...
Кроме того, у вас есть полное дерево наследования, обратите внимание, что ListView
используемый выше, является utils.views.manager.ListView
:
class ListView(BulkActionsMixin, ManagerMixin, BaseListView):
pass
И BaseListView
- utils.views.common.ListView
:
class ListView(ListFilterMixin, AgencyMixin, ContextMixin, BaseListView):
pass
Последним BaseListView
является django.views.generic.list.ListView
.
Используя ipdb
для отладки, как предложил Камиль, я заметил странную вещь, которая, вероятно, является причиной такого поведения:
ipdb> next
> /home.tony/.venvs/cocoonr/lib/python3.6/site-packages/django_filters/filters.py(167)filter()
166 def filter(self, qs, value):
--> 167 if value != self.null_value:
168 return super().filter(qs, value)
ipdb> self.null_value
'null'
ipdb> value
<QuerySet []>
ipdb> self.field_name
'source'
ipdb> self.lookup_expr
'in'
ipdb>
Поэтому последующий код считает source__in
не пустым и добавляет source__in=empty_queryset
к фильтрам. Я предполагаю, что django тогда догадывается, что результат не может оценить непустой запрос и сохраняет бесполезный запрос.
Является ли это ошибкой в django-filters
или я что-то делаю неправильно?
Ответы
Ответ 1
Я, наконец, понял эту проблему.
Видимо django-filters
не обрабатывают корректно на поиск in
для внешних ключей. Фильтр по умолчанию для source__in
например, является ModelChoiceFilter
. Поэтому я должен был явно определить его как ModelMultipleChoiceFilter
.
Однако я столкнулся с другой проблемой, которая заключается в том, что source__in=10&source__in=7
грубо переводит в Q(source__in=10) | Q(source__in=7)
Q(source__in=10) | Q(source__in=7)
. Это вызывает исключение, поскольку 10 и 7 не являются итерабельными. Так что я изменил мой код, чтобы использовать exact
поиск вместо in
, но по- прежнему использовать ModelMultipleChoiceFilter
. Что, в конце концов, дает следующее:
class BookingListFiltersForm(forms.Form):
state__in = forms.MultipleChoiceField(
choices=Booking.STATE_CHOICES, required=False,
label=_("État"), widget=forms.CheckboxSelectMultiple)
source = forms.ModelMultipleChoiceField(
queryset=Platform.objects.all(), required=False,
label=_("Source"), widget=ModelSelect2Multiple(
url='autocomplete:platform'))
class BookingManagerFilter(filters.FilterSet):
source = filters.ModelMultipleChoiceFilter(
queryset=Platform.objects.all())
payments__date = filters.DateFilter(method='payments__date_filter')
payments__method = filters.ChoiceFilter(
method='payments__method_filter',
choices=BookingPayment.METHOD_CHOICES,
)
class Meta:
model = Booking
fields = {
'period': [
'endswith', 'endswith__gte', 'endswith__lte',
'startswith', 'startswith__gte', 'startswith__lte',
],
'state': ['in'],
'source': ['exact'],
'booking_date': ['date', 'date__lte', 'date__gte'],
'accommodation': ['exact'],
'guest': ['exact']
}
def get_form_class(self):
return BookingListFiltersForm
Ответ 2
Я думаю, что документация отвечает на ваш вопрос:
Фильтрация пустой строкой
В настоящее время невозможно фильтровать пустую строку, поскольку пустые значения интерпретируются как пропущенный фильтр.
GET http://localhost/api/my-model?myfield=
Далее в документах приведены примеры возможных решений. Я помещаю здесь один из них
Решение 1. Магические значения
Вы можете переопределить метод filter() класса фильтра, чтобы специально проверить магические значения. Это похоже на обработку значения null ChoiceFilters.
GET http://localhost/api/my-model?myfield=EMPTY
class MyCharFilter(filters.CharFilter):
empty_value = 'EMPTY'
def filter(self, qs, value):
if value != self.empty_value:
return super(MyCharFilter, self).filter(qs, value)
qs = self.get_method(qs)(**{'%s__%s' % (self.name, self.lookup_expr): ""})
return qs.distinct() if self.distinct else qs
Сейчас я чувствую, что недостаточно информации для решения вашей проблемы. Я оставил комментарий по вашему вопросу. Если вы можете предоставить дополнительную информацию, это поможет понять, что происходит.
Вот несколько советов, которые помогут вам отслеживать эту ошибку:
Я подозреваю, что вы должны найти виновника в https://github.com/carltongibson/django-filter/blob/82a47fb7bbddedf179f110723003f3b28682d7fe/django_filters/filterset.py#L215
Вы можете сделать что-то подобное
class BookingManagerFilter(filters.FilterSet):
# your previous code here
def filter_queryset(self, queryset):
import ipdb;ipdb.set_trace()
return super(BookingManagerFilter, self)filter_queryset(queryset):
И запустите конечную точку, ipdb остановит приложение, и вы сможете войти в код и проверить его.