Django: TemplateViewからListViewへの変更

Question

実現したいこと

paginate_byを使用したいため、class内のTemplateViewからListViewに変更したいです。
どなたかご教授頂けると幸いです。

該当のソースコード

class Sample(LoginRequiredMixin, TemplateView):
    #model = Post
    paginate_by = 5
    template_name = 'blog/sample.html'

    def get(self, request, *args, **kwargs):
        context = super().get_context_data(**kwargs)
        context['member'] = Member.objects.get(user=self.request.user)
        context['object_list'] = Post.objects.filter(status=Status.objects.get(name=self.kwargs.get('stat'))).order_by('id').reverse()
        flag = 0

        for s in Member.objects.get(user=self.request.user).status.all():
            if(self.kwargs.get('stat') == str(s)):
                flag += 1
        
        if flag != 1:
            return redirect('authentication:home')
        else:
            return render(request, 'blog/sample.html', context)

試したこと

def get(self, request, *args, **kwargs):
        context = super().get_context_data(**kwargs)
        context['member'] = Member.objects.get(user=self.request.user)
        #context['object_list'] = Post.objects.filter(status=Status.objects.get(name=self.kwargs.get('stat'))).order_by('id').reverse()
        flag = 0

        for s in Member.objects.get(user=self.request.user).status.all():
            if(self.kwargs.get('stat') == str(s)):
                flag += 1
        
        if flag != 1:
            return redirect('authentication:home')
        else:
            return render(request, 'blog/sample.html', context)
    
    def get_queryset(self, **kwargs):
        return Post.objects.filter(status=Status.objects.get(name=self.kwargs.get('stat'))).order_by('id').reverse()

追記

class Sample(LoginRequiredMixin, ListView):
    #model = Post
    paginate_by = 5
    template_name = 'blog/sample.html'
    
    def get_queryset(self, **kwargs):
        print('get_queryser')
        return Post.objects.filter(status=Status.objects.get(name=self.kwargs.get('stat'))).order_by('id').reverse()    
    
    def get_context_data(self, **kwargs):
        print('get_context_data')
        context = super().get_context_data(**kwargs)
        context['member'] = Member.objects.get(user=self.request.user)
        print('get_context_data')
        return context
    
    def get(self, request, *args, **kwargs):
        print('get')
        #context = super().get_context_data(**kwargs)
        #context['member'] = Member.objects.get(user=self.request.user)
        #context['object_list'] = Post.objects.filter(status=Status.objects.get(name=self.kwargs.get('stat'))).order_by('id').reverse()
        flag = 0

        for s in Member.objects.get(user=self.request.user).status.all():
            if(self.kwargs.get('stat') == str(s)):
                flag += 1
        
        if flag != 1:
            return redirect('authentication:home')
        else:
            return render(request, 'blog/sample.html')

追記コンソール画面

System check identified no issues (0 silenced).
April 02, 2020 - 12:23:15
Django version 3.0.1, using settings 'mysite.settings'
Starting development server at http://127.0.0.1:8000/
Quit the server with CTRL-BREAK.
get
[02/Apr/2020 12:23:24] "GET /blog/Vlog/ HTTP/1.1" 200 1770

Answer 1

SampleビューがBaseListViewのgetメソッドをオーバーライドしているため、BaseListViewのgetメソッドで呼び出されているListView特有のメソッドが呼び出されていないと思います。

まずは、Sampleビューのgetメソッドを削除して実行してみてください。

また、Sampleビューのgetメソッドでは、アクセスしてきたユーザーの属性により表示するビューを変更しているようですが、この機能はUserPassesTestMixinをSampleビューのベースクラスに追加して、UserPassesTestMixinが提供するメソッドで、ユーザーのチェックをした方が良いと思います。

class Sample(LoginRequiredMixin, UserPassesTestMixin, ListView):
    login_url = reverse_lazy('authentication:home')
    def test_func(self):
        """
        アクセスを許可する場合はTrueを返却。
        login_urlへリダイレクトさせたい場合はFalseを返却。
        """    

Answer 2

コード挿入したかったのでこちらで失礼します。class内で

def get_queryset(self, **kwargs):
        return Post.objects.filter(status=Status.objects.get(name=self.kwargs.get('stat'))


def get_context_data(self, **kwargs):
        context = super().get_context_data(**kwargs)
        context['member'] = Member.objects.get(user=self.request.user)
        return context


def get(self, request, *args, **kwargs):
        flag = 0

        for s in Member.objects.get(user=self.request.user).status.all():
            if(self.kwargs.get('stat') == str(s)):
                flag += 1

        if flag != 1:
            return redirect('authentication:home')
        else:
            return render(request, 'blog/sample.html', context)

こちらでどうでしょうか？get_context_dataをgetの外で呼ぶ形です。

Comments

[退会済みユーザー]

何度も何度もすいません。

最後の行のreturn render(request, 'blog/sample.html', context)のcontextを削除した場合、エラーなく実行可能でした。
しかし、get_queryset()とget_context_data()で得られるobject_listとmemberがHTML内で機能しないです。{{member.status}}みたいな感じで今までは出力が得られていたのですが、何も表示されなくなってしまいました。

[netsuy]

{{ member }}だけだとオブジェクトは表示されますでしょうか？

[退会済みユーザー]

表示されません

[netsuy]

であればmemberがそもそもcontextに代入できているのか、printしたりobjects.getの結果を出してみたりとか、切り分けて調査結果を言っていただけませんか？get_querysetは呼ばれてる、get_context_dataは呼ばれてないとか。

[退会済みユーザー]

切り分け調査の結果、getは呼ばれてます。
get_querysetとget_context_dataは呼ばれていません。

[netsuy]

であればお手数ですが、追記で現状のclass内すべてのコードを載せていただけませんでしょうか？

[退会済みユーザー]

載せさせて頂きました。

Answer 3

class Sample(LoginRequiredMixin, ListView):

こちらそのままListViewに変更するので問題ないように思えます。

Comments

[退会済みユーザー]

AttributeError: 'Sample' object has no attribute 'object_list'
このようなエラー文が出てしまいます。

[netsuy]

大変失礼いたしました。
試したことに書かれている「def get_queryset(self, **kwargs):」も記載済みでしょうか？
あと、
context = {}
# context = super().get_context_data(**kwargs)
# context['member'] = Member.objects.get(user=self.request.user)
とするとどうなりますでしょうか？

[退会済みユーザー]

上記の場合は、最初のコードに対してで、
次のコードに対してListViewに変更すると、下記エラーが発生してしまいます。
ImproperlyConfigured at /blog/Vlog/
Sample is missing a QuerySet. Define Sample.model, Sample.queryset, or override Sample.get_queryset().

[netsuy]

もしかして「def get_queryset(self, **kwargs):」は、インデント的に「def get」内に記載されていますか？もしそうであればdef getと同じインデントにしてみてもらってもよろしいでしょうか

[退会済みユーザー]

'Sample' object has no attribute 'object_list'
このようなエラーが表示されます