Ignore mutating sql warning while running LabSummaryNotification (#57)

Author: labkey-ankurj

LDK/src/org/labkey/ldk/notification/SiteSummaryNotification.java

@@ -22,6 +22,7 @@
 import org.apache.log4j.Logger;
 import org.json.JSONArray;
 import org.json.JSONObject;
+import org.labkey.api.action.SpringActionController;
 import org.labkey.api.audit.AuditLogService;
 import org.labkey.api.audit.AuditTypeProvider;
 import org.labkey.api.data.CompareType;
@@ -168,7 +169,12 @@ private void saveValues(Container c, Map<String, String> saved, Map<String, Stri
         newValues.put(lastSave, String.valueOf(new Date().getTime()));
         map.putAll(newValues);
 
-        map.save();
+        // this is recording when the report was last run, which is similar to audit logging and similar activities
+        // that we are comfortable treating as non-mutating.
+        try (var ignored = SpringActionController.ignoreSqlUpdates())
+        {
+            map.save();
+        }
     }
 
     @Override

laboratory/src/org/labkey/laboratory/notification/LabSummaryNotification.java

@@ -5,6 +5,7 @@
 import org.apache.log4j.Logger;
 import org.json.JSONArray;
 import org.json.JSONObject;
+import org.labkey.api.action.SpringActionController;
 import org.labkey.api.data.Container;
 import org.labkey.api.data.CoreSchema;
 import org.labkey.api.data.DbSchema;
@@ -116,7 +117,12 @@ private void saveValues(Container c, Map<String, String> saved, Map<String, Stri
         newValues.put(lastSave, String.valueOf(new Date().getTime()));
         map.putAll(newValues);
 
-        map.save();
+        // this is recording when the report was last run, which is similar to audit logging and similar activities
+        // that we are comfortable treating as non-mutating.
+        try (var ignored = SpringActionController.ignoreSqlUpdates())
+        {
+            map.save();
+        }
     }
 
     private String getPctChange(Long oldVal, Long newVal, double threshold)