Revert "Updated Settings UI"

This reverts commit 6436e37, reversing changes made to b76d0a3.
FEMR · Jun 6, 2024 · d0ab911 · d0ab911
1 parent 7a71044
commit d0ab911
Show file tree

Hide file tree

Showing 6 changed files with 30 additions and 62 deletions.
diff --git a/app/femr/business/services/system/UserService.java b/app/femr/business/services/system/UserService.java
@@ -241,7 +241,6 @@ public ServiceResponse<UserItem> updateUser(UserItem userItem, String newPasswor
             user.setRoles(newRoles);
             user.setPasswordReset(userItem.isPasswordReset());
             user.setPasswordCreatedDate(DateTime.now());
-            user.setLanguageCode(userItem.getLanguageCode());
             user = userRepository.updateUser(user);
             response.setResponseObject(itemModelMapper.createUserItem(user));
         } catch (Exception ex) {
@@ -250,7 +249,6 @@ public ServiceResponse<UserItem> updateUser(UserItem userItem, String newPasswor
         return response;
     }
 
-
     /**
      * {@inheritDoc}
      */

diff --git a/app/femr/common/models/UserItem.java b/app/femr/common/models/UserItem.java
@@ -36,6 +36,10 @@ public class UserItem {
     private Integer CreatedBy; //Sam Zanni
     private String languageCode;
 
+    public UserItem() {
+        this.roles = new ArrayList<>();
+    }
+
     public String getFirstName() {
         return firstName;
     }

diff --git a/app/femr/ui/controllers/SettingsController.java b/app/femr/ui/controllers/SettingsController.java
@@ -94,4 +94,4 @@ private EditViewModel convertToEditViewModel(UserItem userItem) {
         return editViewModel;
     }
 
-}
+}
diff --git a/app/femr/ui/views/partials/authenticated.scala.html b/app/femr/ui/views/partials/authenticated.scala.html
@@ -1,3 +1,4 @@
+@import femr.ui.controllers.SettingsController
 @(currentUser: femr.common.dtos.CurrentUser, assets: AssetsFinder)
 
 @import femr.ui.views.partials.helpers.AuthenticatedPartialHelper

diff --git a/app/femr/ui/views/triage/index.scala.html b/app/femr/ui/views/triage/index.scala.html
@@ -93,6 +93,13 @@
                 function updateGenderLabels(languageData, language) {
                     // Check if the language data exists for the selected language
                     if(languageData[language]) {
+                        // Update the Male button
+                        // if(languageData[language]['triage_index_Gender']) {
+                        //     const genderLabelElement = document.querySelector("label[for='genderBtns']");
+                        //     if (genderLabelElement) {
+                        //         genderLabelElement.textContent = languageData[language]['triage_index_Gender'] + " ";
+                        //     }
+                        // }
                         if(languageData[language]['triage_index_Male']) {
                             const maleBtn = document.getElementById('maleBtn');
                             if (maleBtn) {
@@ -108,44 +115,24 @@
                         }
                     }
                 }
-                function updateAgeClassificationLabels(languageData, language) {
-                    const classifications = document.querySelectorAll('#classificationRadioWrap label');
-                    classifications.forEach(label => {
-                        // Extracting the key from the label's ID (label_infant -> infant)
-                        const classification = label.id.replace('label_', '');
-                        const span = label.querySelector('span'); // Finds the span inside the current label
-                        const translationKey = "ageClassification_" + classification; // Constructs the translation key
+                function updateFileInputText(languageData, language) {
+                    const fileInputLabel = document.getElementById('chooseFileLabel');
+                    const fileChosenText = document.getElementById('fileChosenText');
 
-                        // Check if the translation exists for the given key and language
-                        if(languageData[language] && languageData[language][translationKey]) {
-                            const labelText = languageData[language][translationKey];
-                            // Updating the text inside the span
-                            if (span) {
-                                span.textContent = labelText;
-                            }
-                        }
-                    });
-                }
+                    if (fileInputLabel && languageData[language] && languageData[language]['triage_index_choose']) {
+                        fileInputLabel.textContent = languageData[language]['triage_index_choose'];
+                    }
 
-                function updateLabels(languageData, language) {
-                    // Update gender labels
-                    if (languageData[language]) {
-                        // Update the Male button
-                        if (languageData[language]['triage_index_Male']) {
-                            const maleBtn = document.getElementById('maleBtn');
-                            if (maleBtn) {
-                                maleBtn.nextSibling.textContent = languageData[language]['triage_index_Male'];
-                            }
-                        }
-                        // Update the Female button
-                        if (languageData[language]['triage_index_Female']) {
-                            const femaleBtn = document.getElementById('femaleBtn');
-                            if (femaleBtn) {
-                                femaleBtn.nextSibling.textContent = languageData[language]['triage_index_Female'];
-                            }
-                        }
+                    if (fileChosenText && languageData[language] && languageData[language]['triage_index_No']) {
+                        fileChosenText.textContent = languageData[language]['triage_index_No'];
                     }
-                    // Update age classification labels
+                }
+                document.getElementById('photoInput').addEventListener('change', function() {
+                    const fileChosenText = document.getElementById('fileChosenText');
+                    fileChosenText.textContent = this.files.length ? this.files[0].name : languageData[currentLanguage]['triage_index_No'];
+                });
+
+                function updateAgeClassificationLabels(languageData, language) {
                     const classifications = document.querySelectorAll('#classificationRadioWrap label');
                     classifications.forEach(label => {
                         // Extracting the key from the label's ID (label_infant -> infant)
@@ -154,7 +141,7 @@
                         const translationKey = "ageClassification_" + classification; // Constructs the translation key
 
                         // Check if the translation exists for the given key and language
-                        if (languageData[language] && languageData[language][translationKey]) {
+                        if(languageData[language] && languageData[language][translationKey]) {
                             const labelText = languageData[language][translationKey];
                             // Updating the text inside the span
                             if (span) {
@@ -163,25 +150,6 @@
                         }
                     });
                 }
-
-                function updateFileInputText(languageData, language) {
-                    const fileInputLabel = document.getElementById('chooseFileLabel');
-                    const fileChosenText = document.getElementById('fileChosenText');
-
-                    if (fileInputLabel && languageData[language] && languageData[language]['triage_index_choose']) {
-                        fileInputLabel.textContent = languageData[language]['triage_index_choose'];
-                    }
-
-                    if (fileChosenText && languageData[language] && languageData[language]['triage_index_No']) {
-                        fileChosenText.textContent = languageData[language]['triage_index_No'];
-                    }
-                }
-                document.getElementById('photoInput').addEventListener('change', function() {
-                    const fileChosenText = document.getElementById('fileChosenText');
-                    fileChosenText.textContent = this.files.length ? this.files[0].name : languageData[currentLanguage]['triage_index_No'];
-                });
-
-
                 function updateAgeInputPlaceholders(languageCode) {
                     // Update placeholders
                     const yearsInput = document.getElementById('yearsInput'); // You need to ensure this is the correct ID

diff --git a/conf/routes b/conf/routes
@@ -72,10 +72,7 @@ GET         /search/typeahead/medicationsWithID                @femr.ui.controll
 GET         /search/typeahead/medicationAdministrations        @femr.ui.controllers.SearchController.typeaheadMedicationAdministrationsJSONGet()
 
 #Settings
-GET        /settings                                          @femr.ui.controllers.SettingsController.index()
-POST        /settings/update                                   @femr.ui.controllers.SettingsController.update()
-
-
+GET         /settings                                          @femr.ui.controllers.SettingsController.index()
 #Photo
 GET         /photo/patient/:id                                 @femr.ui.controllers.PhotoController.GetPatientPhoto(id: Integer, showDefault: Boolean ?= false)
 GET         /photo/encounter/:id                               @femr.ui.controllers.PhotoController.GetPhoto(id: Int)