From f6bf4ce6185edc13a6d5450c0c6fc22b530f1d97 Mon Sep 17 00:00:00 2001
From: _aandres <alice.andres@m4x.org>
Date: Sat, 31 Dec 2022 03:59:18 +0100
Subject: [PATCH] feat [front,api]: pagination calendar and admin list

- extract list for home in action to not paginate

- compute calendar list from parent to paginate
---
 front/components/movieCard/listPerMonth.vue   | 11 +--
 front/components/pagination.vue               | 88 +++++++++++++++++++
 front/composables/types.ts                    |  7 ++
 front/css/a11y.sass                           | 27 ++----
 front/css/app.sass                            |  1 +
 front/pages/admin/index.vue                   | 30 ++++++-
 front/pages/calendrier.vue                    | 56 ++++++++----
 front/pages/index.vue                         |  2 +-
 .../pagination/PagePaginationWithPageCount.py | 13 +++
 server/myapi/views/std_views.py               | 13 ++-
 server/ulm_cine_club_api/settings/base.py     |  2 +
 11 files changed, 203 insertions(+), 47 deletions(-)
 create mode 100644 front/components/pagination.vue
 create mode 100644 server/myapi/pagination/PagePaginationWithPageCount.py

diff --git a/front/components/movieCard/listPerMonth.vue b/front/components/movieCard/listPerMonth.vue
index 56d1d37..b4e7530 100644
--- a/front/components/movieCard/listPerMonth.vue
+++ b/front/components/movieCard/listPerMonth.vue
@@ -22,16 +22,11 @@
 
 <script setup lang="ts">
 import { FilmsByMonth } from "~/composables/types"
+import { PropType } from "@vue/runtime-core"
 
-const props = defineProps({
-  past: { type: Boolean, default: false },
+defineProps({
+  filmsByMonth: { type: Array as PropType<FilmsByMonth[]>, default: [] },
 })
-const filmsByMonth = ref<FilmsByMonth[]>()
-filmsByMonth.value = (
-  await apiGet<FilmsByMonth[]>(`films/calendar/`, {
-    period: props.past ? "past" : "future",
-  })
-).data.value
 </script>
 
 <style scoped lang="sass">
diff --git a/front/components/pagination.vue b/front/components/pagination.vue
new file mode 100644
index 0000000..dad8f14
--- /dev/null
+++ b/front/components/pagination.vue
@@ -0,0 +1,88 @@
+<template>
+  <nav class="pagination" role="navigation" aria-label="pagination">
+    <nuxt-link
+      class="pagination-previous"
+      :disabled="!prev ? 'disabled' : undefined"
+      :tabindex="!prev ? -1 : undefined"
+      :to="genRouteQuery(prev)"
+      @click="checkAndUpdate(prev)"
+    >
+      Précédent
+    </nuxt-link>
+    <nuxt-link
+      class="pagination-next"
+      :disabled="!next ? 'disabled' : undefined"
+      :tabindex="!next ? -1 : undefined"
+      :to="genRouteQuery(next)"
+      @click="checkAndUpdate(next)"
+    >
+      Suivant
+    </nuxt-link>
+    <ul class="pagination-list">
+      <li v-for="(i, index) in items" :key="index">
+        <span v-if="i === ELLIPSIS" class="pagination-ellipsis">&hellip;</span>
+        <nuxt-link
+          v-else
+          class="pagination-link"
+          :class="{ 'is-current': current === i }"
+          :aria-label="current === i ? `Page ${i}` : `Aller à la page ${i}`"
+          :aria-current="current === i ? 'page' : undefined"
+          :to="genRouteQuery(i)"
+          @click="checkAndUpdate(i)"
+        >
+          {{ i }}
+        </nuxt-link>
+      </li>
+    </ul>
+  </nav>
+</template>
+
+<script setup lang="ts">
+const router = useRouter()
+const props = defineProps({
+  nbToShow: { type: Number, default: 2 },
+  max: { type: Number, required: true },
+  modelValue: { type: Number, required: true },
+})
+const current = useModel<number>("modelValue")
+const ELLIPSIS = "&hellip;"
+
+const prev = computed<number | null>(() =>
+  current.value === 1 ? null : current.value - 1
+)
+const next = computed<number | null>(() =>
+  current.value === props.max ? null : current.value + 1
+)
+const items = reactive<(number | string)[]>([1])
+
+watch(current, () => paginate)
+paginate()
+
+function paginate() {
+  items.splice(1)
+  if (current.value === 1 && props.max === 1) return
+  if (current.value > 4) items.push(ELLIPSIS)
+
+  const r = props.nbToShow,
+    rLeft = current.value - r,
+    rRight = current.value + r
+
+  for (let i = rLeft > r ? rLeft : r; i <= Math.min(props.max, rRight); i++)
+    items.push(i)
+
+  if (rRight + 1 < props.max) items.push(ELLIPSIS)
+  if (rRight < props.max) items.push(props.max)
+}
+
+function checkAndUpdate(page: number | null) {
+  if (page) current.value = page
+  paginate()
+}
+
+function genRouteQuery(page: number) {
+  if (!page) return router.currentRoute
+  return { query: { ...router.currentRoute.value.query, page } }
+}
+</script>
+
+<style scoped lang="sass"></style>
diff --git a/front/composables/types.ts b/front/composables/types.ts
index 065f7ac..ce5793b 100644
--- a/front/composables/types.ts
+++ b/front/composables/types.ts
@@ -40,3 +40,10 @@ export type FilmsByMonth = {
   projectionMonth: string
   films: Film[]
 }
+
+export type PaginatedResponse<T> = {
+  results: T[]
+  currentPage: number
+  totalResults: number
+  totalPages: number
+}
diff --git a/front/css/a11y.sass b/front/css/a11y.sass
index 23553d3..fadd6f0 100644
--- a/front/css/a11y.sass
+++ b/front/css/a11y.sass
@@ -4,45 +4,36 @@
 
  // thanks @cityssm/bulma-a11y
 
-a:focus,
-button:focus 
-  outline-color: $black
-  outline-offset: 4px
-  outline-style: dotted !important
-  outline-width: 2px !important
+a, button, .pagination-link, .pagination-next, .pagination-previous
+  &:focus
+    outline-color: $black
+    outline-offset: 4px
+    outline-style: dotted !important
+    outline-width: 2px !important
 
   .has-background-black &,
-  .has-background-dark & 
+  .has-background-dark &
     outline-color: $white
   
-a:focus 
-
+a:focus
   .has-text-white 
     outline-color: $white
   
-.button:focus 
-
+.button:focus
   &.is-black 
     outline-color: $black
-  
   &.is-danger 
     outline-color: $danger-dark
-  
   &.is-dark 
     outline-color: $dark
-  
   &.is-info 
     outline-color: $info-dark
-  
   &.is-link 
     outline-color: $link-dark
-  
   &.is-primary 
     outline-color: $primary-dark
-  
   &.is-success 
     outline-color: $success-dark
-  
   &.is-warning 
     outline-color: $warning-dark
   
diff --git a/front/css/app.sass b/front/css/app.sass
index a422811..57d1226 100644
--- a/front/css/app.sass
+++ b/front/css/app.sass
@@ -8,6 +8,7 @@
 @import "bulma/sass/components/card"
 @import "bulma/sass/components/modal"
 @import "bulma/sass/components/navbar"
+@import "bulma/sass/components/pagination"
 @import "bulma/sass/components/tabs"
 @import "bulma/sass/elements/button"
 @import "bulma/sass/elements/container"
diff --git a/front/pages/admin/index.vue b/front/pages/admin/index.vue
index 2df7a61..6b24c61 100644
--- a/front/pages/admin/index.vue
+++ b/front/pages/admin/index.vue
@@ -18,18 +18,44 @@
         </div>
       </li>
     </ul>
+    <Pagination
+      :model-value="currentPage"
+      :max="totalPages"
+      @update:model-value="changePage"
+    />
   </main>
 </template>
 
 <script setup lang="ts">
-import { ShortFilm } from "~/composables/types"
+import { ShortFilm, PaginatedResponse } from "~/composables/types"
 
 definePageMeta({ layout: "admin" })
 useHead({ title: "Liste des séances" })
+const route = useRoute()
+const router = useRouter()
+
+const currentPage = ref<number>(parseInt(route.query?.page as string) || 1)
+const totalPages = ref<number>(1)
 
 const films = reactive<ShortFilm[]>([])
 
-Object.assign(films, (await apiGet("admin/films/")).data.value)
+async function changePage(newPage: number) {
+  await loadFilms(newPage)
+  router.replace({ query: { ...route.query, page: currentPage.value } })
+}
+
+await loadFilms(currentPage.value)
+
+async function loadFilms(page = 1) {
+  const res = (
+    await apiGet<PaginatedResponse<ShortFilm>>(`admin/films/`, {
+      page: page,
+    })
+  ).data.value
+  Object.assign(films, res?.results || [])
+  totalPages.value = res?.totalPages || 1
+  currentPage.value = res?.currentPage || 1
+}
 </script>
 
 <style lang="sass">
diff --git a/front/pages/calendrier.vue b/front/pages/calendrier.vue
index f427f95..0066ffc 100644
--- a/front/pages/calendrier.vue
+++ b/front/pages/calendrier.vue
@@ -6,39 +6,65 @@
       <div class="tabs is-toggle is-small">
         <ul>
           <li :class="isPast ? 'is-active' : undefined">
-            <a @click="view = 'past'">
+            <a @click="changeView('past')">
               <span>passées</span>
             </a>
           </li>
           <li :class="isPast ? undefined : 'is-active'">
-            <a @click="view = 'future'">
+            <a @click="changeView('future')">
               <span>à venir</span>
             </a>
           </li>
         </ul>
       </div>
     </header>
-    <KeepAlive>
-      <MovieCardListPerMonth v-if="isPast" past />
-    </KeepAlive>
-    <KeepAlive>
-      <MovieCardListPerMonth v-if="!isPast" />
-    </KeepAlive>
+    <MovieCardListPerMonth :films-by-month="filmsByMonth" />
+    <Pagination
+      :model-value="currentPage"
+      :max="totalPages"
+      @update:model-value="changePage"
+    />
   </section>
 </template>
 
 <script setup lang="ts">
+import { FilmsByMonth, PaginatedResponse } from "~/composables/types"
+
 useHead({ title: "Calendrier" })
 const router = useRouter()
 const route = useRoute()
 
-const view = computed<string>({
-  get: () => (route.query?.view as string) || "future",
-  set(value) {
-    router.replace({ query: { ...route.query, view: value } })
-  },
-})
-const isPast = computed<boolean>(() => view.value === "past")
+const currentPage = ref<number>(parseInt(route.query?.page as string) || 1)
+const totalPages = ref<number>(1)
+const currentView = ref<string>((route.query?.view as string) || "future")
+const isPast = computed<boolean>(() => currentView.value === "past")
+
+const filmsByMonth = ref<FilmsByMonth[]>([])
+
+async function changePage(newPage: number) {
+  await loadFilms(newPage, isPast.value)
+  router.replace({ query: { ...route.query, page: currentPage.value } })
+}
+
+async function changeView(newView: string) {
+  await loadFilms(1, newView === "past")
+  currentView.value = newView
+  router.replace({ query: { view: newView } })
+}
+
+await loadFilms(currentPage.value, isPast.value)
+
+async function loadFilms(page = 1, isPast = false) {
+  const res = (
+    await apiGet<PaginatedResponse<FilmsByMonth>>(`films/calendar/`, {
+      period: isPast ? "past" : "future",
+      page: page,
+    })
+  ).data.value
+  filmsByMonth.value = res?.results || []
+  totalPages.value = res?.totalPages || 1
+  currentPage.value = res?.currentPage || 1
+}
 </script>
 
 <style scoped lang="sass">
diff --git a/front/pages/index.vue b/front/pages/index.vue
index 0ce8472..54414c5 100644
--- a/front/pages/index.vue
+++ b/front/pages/index.vue
@@ -38,7 +38,7 @@ useHead({ title: "Accueil" })
 const modules = ref<SwiperModule[]>([Navigation])
 
 const films = ref<Film[]>()
-films.value = ((await apiGet<Film[]>(`films/`)).data.value || []) as Film[]
+films.value = ((await apiGet<Film[]>(`films/home/`)).data.value || []) as Film[]
 const firstFilm = computed(() => films.value?.[0])
 const nextFilms = computed(() => films.value?.slice(1))
 </script>
diff --git a/server/myapi/pagination/PagePaginationWithPageCount.py b/server/myapi/pagination/PagePaginationWithPageCount.py
new file mode 100644
index 0000000..b188f99
--- /dev/null
+++ b/server/myapi/pagination/PagePaginationWithPageCount.py
@@ -0,0 +1,13 @@
+from rest_framework import pagination
+from rest_framework.response import Response
+
+
+class PagePaginationWithPageCount(pagination.PageNumberPagination):
+
+    def get_paginated_response(self, data):
+        return Response({
+            'current_page': self.page.number,
+            'total_results': self.page.paginator.count,
+            'total_pages': self.page.paginator.num_pages,
+            'results': data
+        })
diff --git a/server/myapi/views/std_views.py b/server/myapi/views/std_views.py
index 66e64a4..0038d9e 100644
--- a/server/myapi/views/std_views.py
+++ b/server/myapi/views/std_views.py
@@ -51,7 +51,7 @@ class AdminFilmViewSet(viewsets.ModelViewSet):
         return self.general_com_view(com.ics)
 
 
-class FilmViewSet(mixins.RetrieveModelMixin, mixins.ListModelMixin, GenericViewSet):
+class FilmViewSet(mixins.RetrieveModelMixin, GenericViewSet):
     serializer_class = FilmSerializer
 
     def get_queryset(self):
@@ -65,12 +65,19 @@ class FilmViewSet(mixins.RetrieveModelMixin, mixins.ListModelMixin, GenericViewS
         ordering = f"{'-' if past else ''}projection_date"
         return queryset.filter(date_filter).order_by(ordering)
 
+    @action(detail=False, methods=["GET"])
+    def home(self, request, *args, **kwargs):
+        queryset = self.filter_queryset(self.get_queryset())[:5]
+        serializer = self.get_serializer(queryset, many=True)
+        return Response(serializer.data)
+
     @action(detail=False, methods=["GET"])
     def calendar(self, request):
         qs = self.get_queryset().annotate(
             projection_month=TruncMonth("projection_date")
         )
-        grouped = groupby(qs, lambda f: f.projection_month)
+        page = self.paginate_queryset(qs) or qs
+        grouped = groupby(page, lambda f: f.projection_month)
         data = [
             {
                 "projection_month": month.strftime("%B %Y"),
@@ -78,4 +85,4 @@ class FilmViewSet(mixins.RetrieveModelMixin, mixins.ListModelMixin, GenericViewS
             }
             for (month, monthly_data) in grouped
         ]
-        return Response(data)
+        return self.get_paginated_response(data)
diff --git a/server/ulm_cine_club_api/settings/base.py b/server/ulm_cine_club_api/settings/base.py
index 687df5a..fc7ed23 100644
--- a/server/ulm_cine_club_api/settings/base.py
+++ b/server/ulm_cine_club_api/settings/base.py
@@ -126,6 +126,8 @@ REST_FRAMEWORK = {
         "djangorestframework_camel_case.parser.CamelCaseMultiPartParser",
         "djangorestframework_camel_case.parser.CamelCaseJSONParser",
     ),
+    'DEFAULT_PAGINATION_CLASS': 'myapi.pagination.PagePaginationWithPageCount.PagePaginationWithPageCount',
+    'PAGE_SIZE': 2
 }
 
 TMDB_API_KEY = config.getstr("tmdb.api_key")