Как фильтровать выборку в шаблоне?

Question

[maxclax]

Имею:

class Data(models.Model):
    user = models.ForeignKey(settings.AUTH_USER_MODEL, verbose_name=_('User'))
    number = models.CharField(_('Number'), unique=True, max_length=6)

class Wallet(models.Model):
    user = models.ForeignKey(settings.AUTH_USER_MODEL, verbose_name=_('User'))
    data = models.ForeignKey(m.Data, null=True, verbose_name=_('Data'))

class DataListView(LoginRequiredMixin, TemplateView, BaseContextMixin):
    template_name = "data/list.html"
    login_url = reverse_lazy('users:login')

    def get_context_data(self, **kwargs):
        context = super(DataListView, self).get_context_data(**kwargs)
        context['my_data'] = self.request.user.data_set.all()
        return context

далее я в шаблоне вывожу:

{% for one in my_data %}

{% endfor %}

И получаю data текущего юзера: context['my_data'] = self.request.user.data_set.all()
Все отлично, но мне внутри цикла for необходимо вывести все wallet этого юзера по этой date. Использую

{% for wallet in one.wallet_set.all %}

{% endfor %}

В итоге я получаю все wallet даже не пренадлежащие текушему юзеру. Как настроить фильтр чтобы wallet были только от текущего user? Как в шаблонах использовать filter по моделям?

Answer 1

[sim3x]

Как фильтровать выборку в шаблоне?

никак.
Отвратительная практика
Фильтруй в модели, в крайнем случае, во вью

Comments

[zigen]

Во вью топик стартер может сделать context['my_wallet'] = Wallet.objects.filter(user=self.requet.user), а как в модели?

[sim3x]

zigen:
Не для данного случая, передаем юзера в менеджер или в метод модели

Для данного случая нужно унаследоваться от AbstractUser, чтоб получить возможноссть делать request.user.data.all() or request.user.data.my_custom_manager

[zigen]

sim3x: А чем плоха практика добавления нескольких дополнительных кверисетов через get_context_data(self, **kwargs) ? Не привязываясь к конкретному случаю. Просто хочу понять, для чего всё запихивать в модели.

[sim3x]

zigen: тестировать так проще

Есть сложный запрос с логикой, если его тестировать во вью, в возвращаемом контексте, то придется тестировать не отдельный запрос, а уже весь контекст

Как бонус, при чтении кода сразу понимаешь почему модель такая и какие задачи она выполняет, а не собирать инфу по куче вью