From 442b47ced71f9e698502b8a2fdab0d74d777a80c Mon Sep 17 00:00:00 2001
From: Adam Rauch <adam@labkey.com>
Date: Wed, 2 Jul 2025 07:36:04 -0700
Subject: [PATCH] Fix messages "Reset to folder default setting" checkbox

---
 .../org/labkey/announcements/emailPreferences.jsp | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/announcements/src/org/labkey/announcements/emailPreferences.jsp b/announcements/src/org/labkey/announcements/emailPreferences.jsp
index c7afa2262d7..74d68139303 100644
--- a/announcements/src/org/labkey/announcements/emailPreferences.jsp
+++ b/announcements/src/org/labkey/announcements/emailPreferences.jsp
@@ -15,7 +15,6 @@
  * limitations under the License.
  */
 %>
-<%@ page import="org.apache.commons.lang3.StringUtils" %>
 <%@ page import="org.labkey.announcements.AnnouncementsController.EmailPreferencesAction" %>
 <%@ page import="org.labkey.api.announcements.EmailOption" %>
 <%@ page extends="org.labkey.announcements.EmailPreferencesPage" %>
@@ -24,14 +23,14 @@
 <script type="text/javascript" nonce="<%=getScriptNonce()%>">
     function toggleOptions()
     {
-        var checkBoxElem = document.getElementById("resetFolderDefault");
+        const checkBoxElem = document.getElementById("resetFolderDefault");
 
-        var isReset = checkBoxElem.checked;
         if (checkBoxElem != null)
         {
-            var inputs = document.getElementsByTagName('input');
-            for(var i = 0; i < inputs.length; i++) {
-                if(inputs[i].type.toLowerCase() === 'radio') {
+            const isReset = checkBoxElem.checked;
+            const inputs = document.getElementsByTagName('input');
+            for (let i = 0; i < inputs.length; i++) {
+                if (inputs[i].type.toLowerCase() === 'radio') {
                     inputs[i].disabled = isReset;
                 }
             }
@@ -54,8 +53,8 @@
     <input type="radio" value="<%=EmailOption.MESSAGES_DAILY_DIGEST.getValue()%>" name="notificationType"<%=checked(notificationType == EmailOption.MESSAGES_DAILY_DIGEST.getValue())%>>
     <b>Daily Digest</b> - send one email each day that summarizes all posts<br>
 
-   <br><input type=checkbox id="resetFolderDefault" name="resetFolderDefault"><b> Reset</b> - Reset to folder default setting<br>
-    <% addHandler("resetFolderDefault", "click", "toggleOption();"); %>
+    <br><input type=checkbox id="resetFolderDefault" name="resetFolderDefault"><b> Reset</b> - Reset to folder default setting<br>
+    <% addHandler("resetFolderDefault", "click", "toggleOptions();"); %>
 
     <br><%=generateReturnUrlFormField(returnUrl)%>
     <br><input type=hidden name="srcIdentifier" value="<%=h(srcIdentifier)%>"/>