From a6182612a0c2c64fee884dda9b49b9319196cfbb Mon Sep 17 00:00:00 2001
From: bbimber <bbimber@gmail.com>
Date: Wed, 23 Oct 2024 15:55:44 -0700
Subject: [PATCH] Allow case-changes of the pseudo-PK for case-insensitive DBs

---
 .../labkey/api/ldk/table/ContainerScopedTable.java  | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/LDK/api-src/org/labkey/api/ldk/table/ContainerScopedTable.java b/LDK/api-src/org/labkey/api/ldk/table/ContainerScopedTable.java
index 0248d936..975f117e 100644
--- a/LDK/api-src/org/labkey/api/ldk/table/ContainerScopedTable.java
+++ b/LDK/api-src/org/labkey/api/ldk/table/ContainerScopedTable.java
@@ -204,8 +204,17 @@ protected Map<String, Object> updateRow(User user, Container container, Map<Stri
             Object oldValue = oldRow.get(_pseudoPk);
             Object newValue = row.get(_pseudoPk);
 
-            if (oldRow != null && newValue != null && !oldValue.equals(newValue) && _keyManager.rowExists(container,  newValue))
-                throw new ValidationException("There is already a record where " + _pseudoPk + " equals " + newValue);
+            if (newValue != null && !oldValue.equals(newValue) && _keyManager.rowExists(container,  newValue))
+            {
+                if (!getSqlDialect().isCaseSensitive() & String.valueOf(newValue).equalsIgnoreCase(String.valueOf(oldValue)) & !String.valueOf(newValue).equals(String.valueOf(oldValue)))
+                {
+                    // Changing case of the PK is allowable. If the DB is case-insensitive, then our DB test should tell us if this would create a violation
+                }
+                else
+                {
+                    throw new ValidationException("There is already a record where " + _pseudoPk + " equals " + newValue);
+                }
+            }
 
             return super.updateRow(user, container, row, oldRow, configParameters);
         }