feat: rework the pagination with per-model setting

Signed-off-by: tdruez <tdruez@aboutcode.org>

Author: tdruez

component_catalog/views.py

@@ -353,7 +353,6 @@ class ComponentListView(
     template_list_table = "component_catalog/tables/component_list_table.html"
     include_reference_dataspace = True
     put_results_in_session = True
-    paginate_by = settings.PAGINATE_BY or 200
     group_name_version = True
 
     table_headers = (
@@ -1734,9 +1733,9 @@ class ScanListView(
     AddPackagePermissionMixin,
     APIWrapperListView,
 ):
-    paginate_by = 50
     template_name = "component_catalog/scan_list.html"
     template_list_table = "component_catalog/tables/scan_list_table.html"
+    paginate_by = settings.DEJACODE_PAGINATE_BY.get("scan", 50)
 
     def dispatch(self, request, *args, **kwargs):
         user = self.request.user

dejacode/settings.py

@@ -387,6 +387,23 @@ def gettext_noop(s):
 PAGINATE_BY = env.int("PAGINATE_BY", default=None)
 TAB_PAGINATE_BY = env.int("TAB_PAGINATE_BY", default=100)
 
+# List views pagination, controls the number of items displayed per page.
+# Syntax in .env: DEJACODE_PAGINATE_BY=product=20,package=100
+DEJACODE_PAGINATE_BY = env.dict(
+    "DEJACODE_PAGINATE_BY",
+    default={
+        "product": 50,
+        "component": PAGINATE_BY or 100,
+        "package": 100,
+        "license": 100,
+        "owner": 100,
+        "report": 50,
+        "request": 50,
+        "scan": 50,
+        "vulnerability": 100,
+    },
+)
+
 ADMIN_FORMS_CONFIGURATION = env.dict("ADMIN_FORMS_CONFIGURATION", default={})
 
 # Location of the changelog file

dje/views.py

@@ -260,8 +260,29 @@ def get_queryset(self):
         return qs.scope(dataspace, include_reference=self.include_reference_dataspace)
 
 
-class PreviousNextPaginationMixin:
+class PaginationMixin:
     query_dict_page_param = "page"
+    paginate_by = None
+    default_paginate_by = 100
+
+    def get_paginate_by(self, queryset):
+        """
+        Determine the number of items per page.
+
+        Resolution order:
+            1. ``paginate_by`` set directly on the view instance.
+            2. Per-model value from the ``DEJACODE_PAGINATE_BY`` setting.
+            3. ``default_paginate_by`` as a fallback.
+        """
+        if self.paginate_by:
+            return self.paginate_by
+
+        if self.model:
+            model_name = self.model._meta.model_name
+            if paginate_by := settings.DEJACODE_PAGINATE_BY.get(model_name):
+                return paginate_by
+
+        return self.default_paginate_by
 
     def get_previous_next(self, page_obj):
         """Return url links for the previous and next navigation."""
@@ -330,12 +351,11 @@ class DataspacedFilterView(
     GetDataspaceMixin,
     HasPermissionMixin,
     TableHeaderMixin,
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     FilterView,
 ):
     template_name = "object_list_base.html"
     template_list_table = None
-    paginate_by = settings.PAGINATE_BY or 100
     # Required if `show_previous_and_next_object_links` enabled on the
     # details view.
     put_results_in_session = False
@@ -2246,7 +2266,7 @@ def page(self, number):
 
 
 class APIWrapperListView(
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     ListView,
 ):
     paginate_by = 100

product_portfolio/views.py

@@ -99,7 +99,7 @@
 from dje.views import Header
 from dje.views import LicenseDataForBuilderMixin
 from dje.views import ObjectDetailsView
-from dje.views import PreviousNextPaginationMixin
+from dje.views import PaginationMixin
 from dje.views import SendAboutFilesView
 from dje.views import TabContentView
 from dje.views import TabField
@@ -164,7 +164,6 @@ class ProductListView(
     filterset_class = ProductFilterSet
     template_name = "product_portfolio/product_list.html"
     template_list_table = "product_portfolio/tables/product_list_table.html"
-    paginate_by = 50
     put_results_in_session = False
     group_name_version = True
     table_headers = (
@@ -733,7 +732,7 @@ def get_context_data(self, **kwargs):
 class ProductTabInventoryView(
     LoginRequiredMixin,
     BaseProductViewMixin,
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     TabContentView,
 ):
     template_name = "product_portfolio/tabs/tab_inventory.html"
@@ -978,7 +977,7 @@ def inject_scan_data(scancodeio, feature_grouped, dataspace_uuid):
 class ProductTabCodebaseView(
     LoginRequiredMixin,
     BaseProductViewMixin,
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     TabContentView,
 ):
     template_name = "product_portfolio/tabs/tab_codebase.html"
@@ -1059,7 +1058,7 @@ def has_any_values(field_name):
 class ProductTabDependenciesView(
     LoginRequiredMixin,
     BaseProductViewMixin,
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     TableHeaderMixin,
     TabContentView,
 ):
@@ -1138,7 +1137,7 @@ def get_context_data(self, **kwargs):
 class ProductTabVulnerabilitiesView(
     LoginRequiredMixin,
     BaseProductViewMixin,
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     TableHeaderMixin,
     TabContentView,
 ):

reporting/views.py

@@ -34,7 +34,7 @@
 from dje.views import DataspacedFilterView
 from dje.views import DownloadableMixin
 from dje.views import HasPermissionMixin
-from dje.views import PreviousNextPaginationMixin
+from dje.views import PaginationMixin
 from reporting.filters import ReportFilterSet
 from reporting.forms import RuntimeFilterBaseFormSet
 from reporting.forms import RuntimeFilterForm
@@ -81,7 +81,7 @@ def get(self, request, *args, **kwargs):
 
 class ReportDetailsView(
     LoginRequiredMixin,
-    PreviousNextPaginationMixin,
+    PaginationMixin,
     BootstrapCSSMixin,
     DownloadableMixin,
     HasPermissionMixin,
@@ -294,7 +294,6 @@ class ReportListView(
     filterset_class = ReportFilterSet
     template_name = "reporting/report_list.html"
     template_list_table = "reporting/includes/report_list_table.html"
-    paginate_by = 50
 
     def get_queryset(self):
         qs = super().get_queryset()

workflow/views.py

@@ -45,7 +45,6 @@ class RequestListView(
     filterset_class = RequestFilterSet
     template_name = "workflow/request_list.html"
     template_list_table = "workflow/includes/request_list_table.html"
-    paginate_by = 50
 
     def get_queryset(self):
         """