From b65098fe0d0b08aa5e34e5761450cd3592eb41ba Mon Sep 17 00:00:00 2001 From: David Wallace Date: Fri, 7 Aug 2026 10:40:53 +0200 Subject: [PATCH] Remove unused projects pagination code Signed-off-by: David Wallace --- .../projects/projects_pagination.html | 23 ------------------ rdmo/projects/tests/test_utils.py | 24 ------------------- rdmo/projects/utils.py | 10 -------- 3 files changed, 57 deletions(-) delete mode 100644 rdmo/projects/templates/projects/projects_pagination.html diff --git a/rdmo/projects/templates/projects/projects_pagination.html b/rdmo/projects/templates/projects/projects_pagination.html deleted file mode 100644 index d0c6e28250..0000000000 --- a/rdmo/projects/templates/projects/projects_pagination.html +++ /dev/null @@ -1,23 +0,0 @@ -{% load i18n %} - - {% if page_obj.has_other_pages %} -
- -
- {% endif %} diff --git a/rdmo/projects/tests/test_utils.py b/rdmo/projects/tests/test_utils.py index 0868852295..c21906be41 100644 --- a/rdmo/projects/tests/test_utils.py +++ b/rdmo/projects/tests/test_utils.py @@ -2,17 +2,14 @@ from django.contrib.auth.models import User from django.contrib.sites.models import Site -from django.http import QueryDict from rdmo.core.tests.utils import compute_checksum -from ..filters import ProjectFilter from ..models import Invite, Project, Value from ..utils import ( compute_set_prefix_from_set_value, copy_project, get_invite_email_project_path, - set_context_querystring_with_filter_and_page, ) GET_queries = [ @@ -34,27 +31,6 @@ ({'set_prefix': '0|0', 'set_index': 1}, {'set_prefix': '0|0|0'}, '0|0|1'), ] -@pytest.mark.parametrize('GET_query', GET_queries) -def test_set_context_querystring_with_filter_and_page(GET_query): - querydict = QueryDict(GET_query) - filter = ProjectFilter(querydict) - context = {'filter': filter} - context = set_context_querystring_with_filter_and_page(context) - - if 'page' in GET_query and 'title' in GET_query: - assert 'querystring' in context - assert context['querystring'] == 'title=project' - querydict_copy = querydict.copy() - del querydict_copy['page'] - assert context['querystring'] == querydict_copy.urlencode() - elif 'page' not in GET_query and 'title' in GET_query: - assert 'querystring' in context - assert context['querystring'] == 'title=project' - elif 'page' in GET_query and 'title' not in GET_query: - assert context.get('querystring', 'not-in-context') == '' - else: - assert context.get('querystring', 'not-in-context') == 'not-in-context' - def test_copy_project(db, files): project = Project.objects.get(id=1) diff --git a/rdmo/projects/utils.py b/rdmo/projects/utils.py index d407e48806..8289e0b97d 100644 --- a/rdmo/projects/utils.py +++ b/rdmo/projects/utils.py @@ -272,16 +272,6 @@ def send_invite_email(request, invite): send_mail(subject, message, to=[invite.email]) -def set_context_querystring_with_filter_and_page(context: dict) -> dict: - '''prepares the filter part of the querystring for the next and previous hyperlinks in the pagination''' - if context["filter"].data: - querystring = context["filter"].data.copy() - if context["filter"].data.get('page'): - del querystring['page'] - context['querystring'] = querystring.urlencode() - return context - - def get_upload_accept(): accept = defaultdict(set) for import_plugin in get_plugins('PROJECT_IMPORTS').values():