ruff

Author: cunla

.github/workflows/test.yml

@@ -34,7 +34,7 @@ jobs:
     strategy:
       max-parallel: 6
       matrix:
-        python-version: [ '3.10', '3.11', '3.12', '3.13' ]
+        python-version: [ '3.11', '3.12', '3.13' ]
         django-version: [ '5.0.7', '5.1.7' ]
         broker: [ 'redis', 'fakeredis', 'valkey' ]
         include:

scheduler/views/queue_views.py

@@ -202,21 +202,6 @@ def queue_confirm_action(request: HttpRequest, queue_name: str) -> HttpResponse:
     return redirect(next_url)
 
 
-def _check_next_url(request: HttpRequest, default_next_url: str) -> str:
-    next_url = request.POST.get("next_url", default_next_url)
-    next_url = next_url.replace('\\', '')
-    if not url_has_allowed_host_and_scheme(next_url, allowed_hosts=None) or urlparse(next_url).netloc or urlparse(
-            next_url).scheme:
-        messages.warning(request, "Bad followup URL")
-        next_url = default_next_url
-    try:
-        resolve(next_url)
-    except Exception:
-        messages.warning(request, "Bad followup URL")
-        next_url = default_next_url
-    return next_url
-
-
 @never_cache
 @staff_member_required
 def queue_actions(request: HttpRequest, queue_name: str) -> HttpResponse:
@@ -326,3 +311,18 @@ def get_statistics(run_maintenance_tasks: bool = False) -> Dict[str, List[Dict[s
             continue
 
     return {"queues": [dataclasses.asdict(q) for q in queues]}
+
+
+def _check_next_url(request: HttpRequest, default_next_url: str) -> str:
+    next_url = request.POST.get("next_url", default_next_url)
+    next_url = next_url.replace('\\', '')
+    if not url_has_allowed_host_and_scheme(next_url, allowed_hosts=None) or urlparse(next_url).netloc or urlparse(
+            next_url).scheme:
+        messages.warning(request, "Bad followup URL")
+        next_url = default_next_url
+    try:
+        resolve(next_url)
+    except Exception:
+        messages.warning(request, "Bad followup URL")
+        next_url = default_next_url
+    return next_url