ProgramPageListView + article list view

aircox/views/api.py

@@ -25,7 +25,7 @@ class LiveAPIView(BaseLogListView, BaseAPIView, ListAPIView):
     queryset = Log.objects.all()
 
     def get(self, *args, **kwargs):
-        self.min_date = tz.now() - tz.timedelta(minutes=5)
+        self.min_date = tz.now() - tz.timedelta(minutes=20)
         return super().get(*args, **kwargs)
 
     def get_serializer(self, queryset, *args, **kwargs):

aircox/views/article.py

@@ -1,12 +1,14 @@
 from ..models import Article
-from .page import PageListView
+from .program import ProgramPageListView
 
 
 __all__ = ['ArticleListView']
 
 
-class ArticleListView(PageListView):
+class ArticleListView(ProgramPageListView):
     model = Article
+    template_name = 'aircox/article_list.html'
+    show_headline = True
     is_static = False
 
     def get_queryset(self):

aircox/views/episode.py

@@ -7,8 +7,7 @@ from django.views.generic import ListView
 
 from ..models import Diffusion, Episode, Page, Program, Sound
 from .base import BaseView
-from .page import PageListView
-from .program import ProgramPageDetailView
+from .program import ProgramPageDetailView, ProgramPageListView
 
 
 __all__ = ['EpisodeDetailView', 'DiffusionListView', 'TimetableView']
@@ -30,32 +29,11 @@ class EpisodeDetailView(ProgramPageDetailView):
         return super().get_context_data(**kwargs)
 
 
-# TODO: pagination: in template, only a limited number of pages displayed
-class EpisodeListView(PageListView):
+class EpisodeListView(ProgramPageListView):
     model = Episode
+    template_name = 'aircox/diffusion_list.html'
     item_template_name = 'aircox/episode_item.html'
     show_headline = True
-    template_name = 'aircox/diffusion_list.html'
-    program = None
-
-    def get(self, request, *args, **kwargs):
-        program_slug = kwargs.get('program_slug')
-        if program_slug:
-            self.program = get_object_or_404(Program, slug=program_slug)
-        return super().get(request, *args, **kwargs)
-
-    def get_queryset(self):
-        qs = super().get_queryset()
-        if self.program:
-            qs = qs.filter(program=self.program)
-        return qs
-
-    def get_context_data(self, **kwargs):
-        program = kwargs.setdefault('program', self.program)
-        if program is not None:
-            kwargs.setdefault('cover', program.cover)
-            kwargs.setdefault('parent', program)
-        return super().get_context_data(**kwargs)
 
 
 class TimetableView(BaseView, ListView):

aircox/views/page.py

@@ -1,4 +1,5 @@
 
+from django.core.exceptions import FieldDoesNotExist
 from django.http import Http404
 from django.views.generic import DetailView, ListView
 
@@ -45,10 +46,11 @@ class PageDetailView(BaseView, DetailView):
         return super().get_context_data(**kwargs)
 
 
+# TODO: pagination: in template, only a limited number of pages displayed
 class PageListView(BaseView, ListView):
     template_name = 'aircox/page_list.html'
     item_template_name = 'aircox/page_item.html'
-    paginate_by = 10
+    paginate_by = 20
     show_headline = True
     show_side_nav = True
     categories = None
@@ -82,3 +84,4 @@ class PageListView(BaseView, ListView):
         return super().get_context_data(**kwargs)
 
 
+

aircox/views/program.py

@@ -1,15 +1,19 @@
+from django.core.exceptions import ObjectDoesNotExist
+from django.shortcuts import get_object_or_404
 
 from aircox.models import Episode, Program
-from .page import PageDetailView
+from .page import PageDetailView, PageListView
 
 
 __all__ = ['ProgramPageDetailView', 'ProgramDetailView']
 
 
 class ProgramPageDetailView(PageDetailView):
-    """ Base view class for rendering content of a specific programs. """
+    """
+    Base view class for a page that is displayed as a program's child page.
+    """
     show_side_nav = True
-    list_count=5
+    list_count = 5
 
     def get_episodes_queryset(self, program):
         return program.episode_set.published().order_by('-date')
@@ -21,9 +25,40 @@ class ProgramPageDetailView(PageDetailView):
             program=program, episodes=episodes[:self.list_count], **kwargs)
 
 
+class ProgramPageListView(PageListView):
+    """
+    Base list view class rendering pages as a program's child page.
+    Retrieved program from it slug provided by `kwargs['program_slug']`.
+
+    This view class can be used with or without providing a program.
+    """
+    program = None
+
+    def get(self, request, *args, **kwargs):
+        slug = kwargs.get('program_slug', None)
+        if slug is not None:
+            self.program = get_object_or_404(
+                Program.objects.select_related('cover'), slug=slug)
+        return super().get(request, *args, **kwargs)
+
+    def get_queryset(self):
+        return super().get_queryset().filter(program=self.program) \
+               if self.program else super().get_queryset()
+
+    def get_context_data(self, **kwargs):
+        program = kwargs.setdefault('program', self.program)
+        if program is not None:
+            kwargs.setdefault('cover', program.cover)
+            kwargs.setdefault('parent', program)
+        return super().get_context_data(**kwargs)
+
+
 class ProgramDetailView(ProgramPageDetailView):
     model = Program
 
+    def get_articles_queryset(self, program):
+        return program.article_set.published().order_by('-date')
+
     def get_context_data(self, **kwargs):
         kwargs.setdefault('program', self.object)
         return super().get_context_data(**kwargs)