From 07cab85fc979624dd051ff928e6d76d488cd583f Mon Sep 17 00:00:00 2001
From: ppettitau <paola.pettit@qut.edu.au>
Date: Mon, 28 Oct 2024 08:06:18 +1000
Subject: [PATCH] [ERP-2779] Fix Preferred Language Bugs * Remove duplicate
 imports for the bootstrap-select causing issues when typing to search for a
 language from the Languages dropdown * Fix logic that determines which
 preferred language to select based on the higher-level language
 (USER_REQUESTED_LANGUAGE) selected.

---
 .../registration_login_details.html           | 23 +++++++++----------
 rdrf/rdrf/views/registration_rdrf.py          |  3 ---
 2 files changed, 11 insertions(+), 15 deletions(-)

diff --git a/rdrf/rdrf/templates/registration/registration_login_details.html b/rdrf/rdrf/templates/registration/registration_login_details.html
index f79994ef3..9922ee0fb 100644
--- a/rdrf/rdrf/templates/registration/registration_login_details.html
+++ b/rdrf/rdrf/templates/registration/registration_login_details.html
@@ -4,25 +4,24 @@
 {% get_user_requested_language as USER_REQUESTED_LANGUAGE %}
 {% get_current_language as CURRENT_LANGUAGE %}
 
-<link rel="stylesheet" href="{% static 'vendor/bootstrap-select-1.14.0-beta3/css/bootstrap-select.min.css' %}">
-
 <script type="text/javascript" src="{% static 'js/show_password.js' %}"></script>
-<script type="text/javascript" src="{% static 'vendor/bootstrap-select-1.14.0-beta3/js/bootstrap-select.min.js' %}"></script>
-<script type="text/javascript" src="/static/vendor/bootstrap-select-1.14.0-beta3/js/i18n/defaults-{{ CURRENT_LANGUAGE|get_locale }}.js"></script>
 
 <script>
     function constructPreferredLanguages() {
-        const preferred_language = "{{ USER_REQUESTED_LANGUAGE }}";
-        const not_in_languages_array = !{{ all_language_codes|safe }}.map(language => language.toLowerCase()).includes(preferred_language);
+        const isMatchingLanguage = (code1, code2) => code1.toLowerCase() === code2.toLowerCase();
         const getPureLanguageCode = (language) => language.split('-')[0];
-        const pure_language_code = getPureLanguageCode(preferred_language);
-        const preferred_languages = {{ preferred_languages|safe }};
-        const selected_preferred_language = preferred_languages.find(
-            (language) => (language.code.toLowerCase() === preferred_language) || (not_in_languages_array && getPureLanguageCode(language.code) === pure_language_code)
+
+        const preferredLanguages = {{ preferred_languages|safe }};
+        const userLanguageCode = "{{ USER_REQUESTED_LANGUAGE }}";
+        const isUserLanguageInList = {{ preferred_languages|safe }}.find(language => isMatchingLanguage(language.code, userLanguageCode));
+        const pureUserLanguageCode = getPureLanguageCode(userLanguageCode);
+        const selectedPreferredLanguage = preferredLanguages.find(
+            (language) => isMatchingLanguage(language.code, userLanguageCode)
+                || (!isUserLanguageInList && isMatchingLanguage(getPureLanguageCode(language.code), pureUserLanguageCode))
         ) ?? { code: "{{ CURRENT_LANGUAGE }}" };
-        const options = preferred_languages.map((language) => `
+        const options = preferredLanguages.map((language) => `
             <option value="${language.code}" data-tokens="${language.code}"
-                ${language.code.toLowerCase() === selected_preferred_language.code.toLowerCase() ? 'selected' : ''}
+                ${isMatchingLanguage(language.code, selectedPreferredLanguage.code) ? 'selected' : ''}
             >${language.name}</option>
         `);
 
diff --git a/rdrf/rdrf/views/registration_rdrf.py b/rdrf/rdrf/views/registration_rdrf.py
index bf375b2d0..4429a55d5 100644
--- a/rdrf/rdrf/views/registration_rdrf.py
+++ b/rdrf/rdrf/views/registration_rdrf.py
@@ -56,9 +56,6 @@ def get_context_data(self, **kwargs):
             {"code": lang.code, "name": lang.name} for lang in languages_dict
         ]
         context["is_mobile"] = self.request.user_agent.is_mobile
-        context["all_language_codes"] = [
-            language.code for language in languages_dict
-        ]
 
         return context