diff --git a/zerver/lib/i18n.py b/zerver/lib/i18n.py index 7aeba76dba..add39c2327 100644 --- a/zerver/lib/i18n.py +++ b/zerver/lib/i18n.py @@ -85,7 +85,7 @@ def get_browser_language_code(request: HttpRequest) -> Optional[str]: return None -def get_default_language_for_new_user(request: HttpRequest, realm: Realm) -> str: +def get_default_language_for_new_user(realm: Realm, *, request: HttpRequest) -> str: browser_language_code = get_browser_language_code(request) if browser_language_code is not None: return browser_language_code diff --git a/zerver/tests/test_signup.py b/zerver/tests/test_signup.py index 5af3b967c4..b508fbf40f 100644 --- a/zerver/tests/test_signup.py +++ b/zerver/tests/test_signup.py @@ -3984,17 +3984,17 @@ class UserSignUpTest(ZulipTestCase): realm = get_realm("zulip") req = HostRequestMock() req.META["HTTP_ACCEPT_LANGUAGE"] = "de,en" - self.assertEqual(get_default_language_for_new_user(req, realm), "de") + self.assertEqual(get_default_language_for_new_user(realm, request=req), "de") do_set_realm_property(realm, "default_language", "hi", acting_user=None) realm.refresh_from_db() req = HostRequestMock() req.META["HTTP_ACCEPT_LANGUAGE"] = "de,en" - self.assertEqual(get_default_language_for_new_user(req, realm), "de") + self.assertEqual(get_default_language_for_new_user(realm, request=req), "de") req = HostRequestMock() req.META["HTTP_ACCEPT_LANGUAGE"] = "" - self.assertEqual(get_default_language_for_new_user(req, realm), "hi") + self.assertEqual(get_default_language_for_new_user(realm, request=req), "hi") class DeactivateUserTest(ZulipTestCase): diff --git a/zerver/views/registration.py b/zerver/views/registration.py index 203731c2e5..e45f82df7f 100644 --- a/zerver/views/registration.py +++ b/zerver/views/registration.py @@ -566,7 +566,7 @@ def registration_helper( do_change_user_setting( user_profile, "default_language", - get_default_language_for_new_user(request, realm), + get_default_language_for_new_user(realm, request=request), acting_user=None, ) # TODO: When we clean up the `do_activate_mirror_dummy_user` code path, @@ -583,7 +583,7 @@ def registration_helper( role=role, tos_version=settings.TERMS_OF_SERVICE_VERSION, timezone=timezone, - default_language=get_default_language_for_new_user(request, realm), + default_language=get_default_language_for_new_user(realm, request=request), default_stream_groups=default_stream_groups, source_profile=source_profile, realm_creation=realm_creation,