django 選択されたタグが入っている記事を一覧で取得する方法

Question

選択されたタグが入っている記事一覧で取得したいのですが、取得方法がわからず質問させていただきました。
下記のコードのように、直接tag1が入っている記事の取得はできたのですが、直接取得している為、選択されたタグを変えても同じ (tag1が入っている) 記事が一覧で取得されます。

tag1を変数に変更して、選択されたタグを変えても対応できるにはどうすれば良いしょうか？
アドバイスいただきたいです。

#Views.py
#全タグリスト
class TagsView(View):
    def get(self, request, *args, **kwargs):
        context = {}
        context['all_tag_list'] = Tag.objects.all().annotate(num_tags=Count('post'))
        return render(request, './tag_list.html', context) 

#選択されたタグが入っている記事一覧
class TagsPostListView(View):
    def get(self, request, *args, **kwargs):
        context = {}
        context['all_tag_post_list'] = Post.objects.filter(tags__name__in=["tag1"]).distinct()
        print(context)
        return render(request, './tag_post_list.html', context) 

tag_post_list.html

    {% for post in all_tag_post %}
	<li class="list-group-post">
        <div class="row">
            <div class="main-title col-12">
        <a href="{% url 'app:detail' post.pk %}">{{ post.title }}{{ post.tags__name__in }}</a>
            </div>
        </div>
        <div class="row">
            <div class="user-post col-2">
                <a href="{% url 'app:user_detail' user.user_name %}">{{ post.post_user }}</a>
            </div>
            <div class="date-list col-8">
                {% for tag in post.tags.all %}
                    <a href="{% url 'app:tags' tag %}">{{ tag }}</a>
                {% endfor %}            
            </div>
            <div class="date-list col-2">
                {{post.created_at|date:"Y/m/d H:i"}}
            </div>
        </div>
    </li>
	{% endfor %}

#urls.py
    path('tags_list/',views.TagsView.as_view(), name='tags_list'),
    path('tags_post/<str>/',views.TagsPostListView.as_view(), name='tags_post'),

そもそもの書き方、考え方、が間違っているなどありましたご指摘いただける幸いです。
よろしくお願いいたします。

#追記
'NoneType' object is not iterableのエラー内容を追記させていただきました。

None
Internal Server Error: /tags_post/
Traceback (most recent call last):
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/core/handlers/base.py", line 115, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/views/generic/base.py", line 71, in view
    return self.dispatch(request, *args, **kwargs)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/views/generic/base.py", line 97, in dispatch
    return handler(request, *args, **kwargs)
  File "/Users/name/Desktop/study/zigzag2/register/views.py", line 391, in get
    context['all_tag_post_list'] = Item.objects.filter(tags__name__in=tag).distinct()
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/manager.py", line 82, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/query.py", line 904, in filter
    return self._filter_or_exclude(False, *args, **kwargs)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/query.py", line 923, in _filter_or_exclude
    clone.query.add_q(Q(*args, **kwargs))
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/sql/query.py", line 1350, in add_q
    clause, _ = self._add_q(q_object, self.used_aliases)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/sql/query.py", line 1377, in _add_q
    child_clause, needed_inner = self.build_filter(
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/sql/query.py", line 1311, in build_filter
    condition = self.build_lookup(lookups, col, value)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/sql/query.py", line 1165, in build_lookup
    lookup = lookup_class(lhs, rhs)
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/lookups.py", line 22, in __init__
    self.rhs = self.get_prep_lookup()
  File "/Users/name/Desktop/study/zigzag2/myvenv/lib/python3.8/site-packages/django/db/models/lookups.py", line 217, in get_prep_lookup
    for rhs_value in self.rhs:
TypeError: 'NoneType' object is not iterable

Answer 1

方法はいろいろあると思いますが、こう言う時はGETパラメーターを用いて渡すのがいいのかなと思います。

例えばyourdomain.com/tag_post_list_url?tag=hogeのようなURLを叩かれた場合,hogeという名前のタグがついた記事データを返すと言った仕様です。

以下のようなコードになるのではないでしょうか。

from django.views.generic import View
#その他必要なものをimport

class TagsPostListView(View):
    def get(self, request,*args, **kwargs):
      # getパラメーターのtag=の部分を取得
      tag = request.GET.get('tag')

      context = {}
      context['all_tag_post_list'] = Post.objects.filter(tags__name__in=tag).distinct()
      print(context)
      return render(request, './tag_post_list.html', context) 

動作の検証などしていませんが、多分動くかと思います。
ご参考までに

Comments

[Mamiya_tom]

回答していただきありがとうございます。
実行してみたのですが、
'NoneType' object is not iterable
とエラーが出たのですが、この場合は、（TagsPostListView）ではなく（TagsView）に問題があるのでしょうか？

[switch0805]

もしかしてdjango-taggitをご使用でしょうか？その場合は
context['all_tag_post_list'] = Post.objects.filter(tags__name__in=tag).distinct()
を
context['all_tag_post_list'] = Post.objects.filter(tags__name__in=[tag]).distinct()
にすると動くかも知れないです。

それでも動かないか、そもそも先のパッケージを使用されていないなら、次のことも教えてください。
当該の事象は/tags_postのURLを叩いた時に出たのでしょうか？加えてエラー文をtracebackから提示いただけると精度良く回答できるかと思います。お手数ですが記載の程よろしくお願いいたします。

最後にurls.pyの存在を見落としてしまったので、そこを訂正させてください！
url.pyの
path('tags_post/<str>/',views.TagsPostListView.as_view(), name='tags_post')
を
path('tags_post/',views.TagsPostListView.as_view(), name='tags_post'),
に変更

お手数おかけします。

[switch0805]

それからちょっと本筋から離れますが、GETパラメーターに何も指定していない時の挙動も作り込まれるといいですね。（すみません、先に指摘しておくべきでした）
例えば何も指定しない場合（つまりtagの中身がNoneもしくは""のとき）は、全件取得したものを表示するなど・・・

[Mamiya_tom]

返信ありがとうございます。
/tags_postのURLを叩いた時にエラーが出ます。
django-taggitを使用しています。最初に書くべきでした申し訳ありません。

django-taggitの場合のコードに変更し、urlsも変更後エラーは無くなったのですが、記事が表示されません。
ちなみに、returnの前に`print(tag)`書いたのですがすでに出力がNoneでした。

[switch0805]

情報ありがとうございます。
おそらくGETパラメーターにtagが指定されていないことによるものと思われます。
tag1というタグはソースを見る限り存在するようですので、
/tags_post?tag=tag1
という風にURLを叩くと動くと思われます。

[Mamiya_tom]

URLに直打ちで表示させることができました！
調べてGETパラメーターにtagを指定していきたいと思います。
わかりやすく教えていただきありがとうございました。