dynamic pagination class + UI for submissions

Author: IdirLISN

src/apps/api/pagination.py

@@ -22,3 +22,64 @@ def get_paginated_response(self, data):
             'page_size': self.page_size,
             'results': data
         })
+
+
+class DynamicChoicePagination(PageNumberPagination):
+    """
+    Pagination dynamique pour l'UI :
+    - défaut : 50
+    - valeurs autorisées côté client : 50, 100, 500, all
+    - si page_size=all => on renvoie tous les objets (dans la limite de max_page_size)
+    """
+    page_size = 50
+    page_size_query_param = 'page_size'
+    max_page_size = 1000
+    _allowed_sizes = (50, 100, 500, 'all')
+
+    def get_page_size(self, request):
+        raw = request.query_params.get(self.page_size_query_param)
+        if raw is None:
+            return self.page_size
+
+        raw_lower = str(raw).lower()
+        if raw_lower == 'all':
+            return None
+
+        try:
+            val = int(raw)
+        except (TypeError, ValueError):
+            return self.page_size
+
+        if val in (50, 100, 500):
+            return val
+        return self.page_size
+
+    def paginate_queryset(self, queryset, request, view=None):
+        raw = request.query_params.get(self.page_size_query_param)
+        self.requested_page_size = raw if raw is not None else str(self.page_size)
+
+        if raw is not None and str(raw).lower() == 'all':
+            try:
+                total = queryset.count()
+            except Exception:
+                total = self.max_page_size
+            self.page_size = total if (isinstance(total, int) and total > 0) else 1
+        else:
+            page_size = self.get_page_size(request)
+            if page_size is None:
+                page_size = self.page_size
+            self.page_size = page_size
+
+        return super().paginate_queryset(queryset, request, view)
+
+    def get_paginated_response(self, data):
+        page_size_value = self.requested_page_size if getattr(self, 'requested_page_size', None) is not None else self.page_size
+
+        return Response({
+            'next': self.get_next_link(),
+            'previous': self.get_previous_link(),
+            'count': self.page.paginator.count,
+            'page_size': page_size_value,
+            'results': data,
+            'allowed_page_sizes': [50, 100, 500, 'all'],
+        })

src/apps/api/views/submissions.py

@@ -16,7 +16,7 @@
 from django.core.files.base import ContentFile
 
 from profiles.models import Organization, Membership
-from api.pagination import LargePagination
+from api.pagination import DynamicChoicePagination
 from tasks.models import Task
 from api.serializers.submissions import SubmissionCreationSerializer, SubmissionSerializer, SubmissionFilesSerializer, SubmissionDetailSerializer
 from competitions.models import Submission, SubmissionDetails, Phase, CompetitionParticipant
@@ -33,7 +33,7 @@ class SubmissionViewSet(ModelViewSet):
     filterset_fields = ('phase__competition', 'phase', 'status', 'is_soft_deleted')
     search_fields = ('data__data_file', 'description', 'name', 'owner__username')
     renderer_classes = api_settings.DEFAULT_RENDERER_CLASSES + [renderers.CSVRenderer]
-    pagination_class = LargePagination
+    pagination_class = DynamicChoicePagination
 
     def check_object_permissions(self, request, obj):
         if self.action in ['submission_leaderboard_connection']:

src/static/riot/competitions/detail/submission_manager.tag

@@ -204,7 +204,14 @@
             </button>
         </div>
 
-        <div style="display:flex; align-items:center; gap:8px; margin-left: auto;">
+        <div style="display:flex; align-items:center; gap:8px;">
+            <label>Per page</label>
+            <select class="ui dropdown" value="{ page_size }" onchange="{ change_page_size.bind(this) }">
+                <option value="50">50</option>
+                <option value="100">100</option>
+                <option value="500">500</option>
+                <option value="all">all</option>
+            </select>
 
             <div style="margin-right: 10px; color: #8c8c8c;">
                 <small>{ total_count || 0 } total</small>
@@ -239,6 +246,8 @@
                         show_visualization="{opts.competition.enable_detailed_results}"
                         submission="{child}"></submission-modal>
                 </div>
+
+
                 <div class="ui tab" style="height: 565px; overflow: auto;" data-tab="admin" if="{is_admin()}">
                     <submission-scores leaderboards="{leaderboards}"></submission-scores>
                 </div>
@@ -258,7 +267,7 @@
         self.show_is_soft_deleted = false
 
         self.page = 1
-        self.page_size = 20
+        self.page_size = 50
         self.total_count = 0
         self.total_pages = 1
         self.next = null
@@ -325,9 +334,22 @@
                         self.next = response.next || null
                         self.previous = response.previous || null
                         self.total_count = response.count || 0
-                        // prefer page_size from server if present
-                        self.page_size = response.page_size || self.page_size
-                        self.total_pages = Math.max(1, Math.ceil(self.total_count / self.page_size))
+                        if (response && typeof response.page_size !== 'undefined') {
+                            const rsp = String(response.page_size).toLowerCase()
+                            if (rsp === 'all') {
+                                self.page_size = 'all'
+                            } else {
+                                const n = Number(response.page_size)
+                                self.page_size = isNaN(n) ? self.page_size : n
+                            }
+                        }
+
+                        if (String(self.page_size).toLowerCase() === 'all') {
+                            self.total_pages = 1
+                        } else {
+                            const ps = Number(self.page_size) || 1
+                            self.total_pages = Math.max(1, Math.ceil(self.total_count / ps))
+                        }
                     } else {
                         results = response || []
                         self.next = null
@@ -377,9 +399,18 @@
         }
 
         self.change_page_size = function (e) {
-            const val = (e && e.target && e.target.value) ? parseInt(e.target.value, 10) : parseInt(self.page_size, 10)
-            if (!val || val <= 0) return
-            self.page_size = val
+            const raw = (e && e.target && typeof e.target.value !== 'undefined') ? String(e.target.value).toLowerCase() : String(self.page_size).toLowerCase()
+
+            if (raw === 'all') {
+                self.page_size = 'all'
+            } else {
+                const val = parseInt(raw, 10)
+                if (isNaN(val) || val <= 0) return
+                // n'autorise que 50,100,500
+                if (![50, 100, 500].includes(val)) return
+                self.page_size = val
+            }
+
             self.page = 1  // reset to first page when page size changes
             self.update_submissions()
         }