Remove request parameter from get_locale_or_redirect

mozilla · Nov 17, 2023 · aef18d4 · aef18d4
1 parent 7a039c5
commit aef18d4
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 6 deletions.
diff --git a/pontoon/base/utils.py b/pontoon/base/utils.py
@@ -639,7 +639,7 @@ def get_project_or_redirect(
             raise Http404
 
 
-def get_locale_or_redirect(code, request, redirect_view_name, url_arg_name, **kwargs):
+def get_locale_or_redirect(code, redirect_view_name, url_arg_name, **kwargs):
     """
     Attempts to retrieve a locale using the given code. If the locale does not exist, it checks the LocaleCodeHistory 
     for a record of the old code. If an entry is found, it redirects to the view specified by redirect_view_name 

diff --git a/pontoon/contributors/views.py b/pontoon/contributors/views.py
@@ -314,7 +314,7 @@ def settings(request):
     # Set custom homepage selector value
     custom_homepage_code = profile.custom_homepage
     custom_homepage_locale = get_locale_or_redirect(
-        custom_homepage_code, request, "pontoon.contributors.views.settings", "locale"
+        custom_homepage_code, "pontoon.contributors.views.settings", "locale"
     )
     if isinstance(custom_homepage_locale, HttpResponseRedirect):
         return custom_homepage_locale
@@ -326,7 +326,7 @@ def settings(request):
     # Set preferred source locale
     preferred_source_code = profile.preferred_source_locale
     preferred_source_locale = get_locale_or_redirect(
-        preferred_source_code, request, "pontoon.contributors.views.settings", "locale"
+        preferred_source_code, "pontoon.contributors.views.settings", "locale"
     )
     if isinstance(preferred_source_locale, HttpResponseRedirect):
         return preferred_source_locale

diff --git a/pontoon/localizations/views.py b/pontoon/localizations/views.py
@@ -26,7 +26,7 @@
 def localization(request, code, slug):
     """Locale-project overview."""
     locale = get_locale_or_redirect(
-        code, request, "pontoon.localizations.localization", "code", slug=slug
+        code, "pontoon.localizations.localization", "code", slug=slug
     )
     if isinstance(locale, HttpResponseRedirect):
         return locale

diff --git a/pontoon/teams/views.py b/pontoon/teams/views.py
@@ -51,7 +51,7 @@ def teams(request):
 
 def team(request, locale):
     """Team dashboard."""
-    locale = get_locale_or_redirect(locale, request, "pontoon.teams.team", "locale")
+    locale = get_locale_or_redirect(locale, "pontoon.teams.team", "locale")
     if isinstance(locale, HttpResponseRedirect):
         return locale
     available_count = locale.project_set.available().visible_for(request.user).count()

diff --git a/pontoon/translate/views.py b/pontoon/translate/views.py
@@ -44,7 +44,6 @@ def translate(request, locale, project, resource):
     # Validate Locale
     locale = get_locale_or_redirect(
         locale,
-        request,
         "pontoon.translate",
         "locale",
         project=project,