From 2c8d1e9ee39a7c937de3f4a894e1974dcfc3910e Mon Sep 17 00:00:00 2001 From: labkey-susanh Date: Wed, 31 Dec 2025 08:27:08 -0800 Subject: [PATCH 1/2] Remove lower casing of fields in audit details maps --- api/src/org/labkey/api/audit/AuditHandler.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/api/src/org/labkey/api/audit/AuditHandler.java b/api/src/org/labkey/api/audit/AuditHandler.java index 9c0f1f8f973..8ca20bf30c1 100644 --- a/api/src/org/labkey/api/audit/AuditHandler.java +++ b/api/src/org/labkey/api/audit/AuditHandler.java @@ -116,8 +116,6 @@ static Pair, Map> getOldAndNewRecordForMerge nameFromAlias = encodedInputColumn; } } - else - nameFromAlias = lcName; boolean isExtraAuditField = extraFieldsToInclude != null && extraFieldsToInclude.contains(nameFromAlias); if (!excludedFromDetailDiff.contains(nameFromAlias) && (row.containsKey(nameFromAlias) || isExpInput)) From cb480d172e4d8f3cec989a517583db7ca8f249e7 Mon Sep 17 00:00:00 2001 From: labkey-susanh Date: Wed, 31 Dec 2025 09:34:52 -0800 Subject: [PATCH 2/2] Change expected casing --- .../src/org/labkey/experiment/api/ExpDataClassDataTestCase.jsp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/experiment/src/org/labkey/experiment/api/ExpDataClassDataTestCase.jsp b/experiment/src/org/labkey/experiment/api/ExpDataClassDataTestCase.jsp index 42aeed754d7..26381bc8d2f 100644 --- a/experiment/src/org/labkey/experiment/api/ExpDataClassDataTestCase.jsp +++ b/experiment/src/org/labkey/experiment/api/ExpDataClassDataTestCase.jsp @@ -1257,7 +1257,7 @@ public void testUpdateAuditForLongField() throws Exception List events = AuditLogService.get().getAuditEvents(c, user, "QueryUpdateAuditEvent", filter, new Sort("-RowId")); assertEquals("Number of audit events not as expected", 2, events.size()); // should have one for insert and one for update String oldRecordMap = ((DetailedAuditTypeEvent) events.get(0)).getOldRecordMap(); - assertTrue("Old record map (" + oldRecordMap + ") does not contain expected field", oldRecordMap.contains(encodeURIComponent(fieldName.toLowerCase()) + "=Initial")); + assertTrue("Old record map (" + oldRecordMap + ") does not contain expected field", oldRecordMap.contains(encodeURIComponent(fieldName) + "=Initial")); } } %>