Allow inserts on ContainerScopedTable when pseudo-key is the real PK

ContainerScopedTable.init() hid and disabled every real PK column, assuming the real PK is always a surrogate key. When the pseudo-key is also the real PK (a natural key, like ehr_lookups.project_types.type), this made the key column hidden and non-editable, so inserts were impossible. Detect that case and leave the column visible and editable, hiding it only from the update form.

Author: labkey-martyp

LDK/api-src/org/labkey/api/ldk/table/ContainerScopedTable.java

@@ -80,22 +80,37 @@ public ContainerScopedTable<SchemaType> init()
     {
         super.init();
 
+        boolean realPkIsPseudoKey = false;
         for (String col : getRealTable().getPkColumnNames())
         {
             var existing = getMutableColumn(col);
             if (existing != null)
             {
-                existing.setKeyField(false);
-                existing.setUserEditable(false);
-                existing.setHidden(true);
                 _realPKs.add(col);
+                if (col.equalsIgnoreCase(_pseudoPk))
+                {
+                    // The pseudo-key is also the real PK (a natural key, like ehr_lookups.project_types.type),
+                    // so it must remain visible and editable for inserts to be possible
+                    realPkIsPseudoKey = true;
+                }
+                else
+                {
+                    existing.setKeyField(false);
+                    existing.setUserEditable(false);
+                    existing.setHidden(true);
+                }
             }
         }
 
         var newKey = getMutableColumn(_pseudoPk);
         assert newKey != null;
 
         newKey.setKeyField(true);
+        if (realPkIsPseudoKey)
+        {
+            // Enterable on insert, but hidden from the update form
+            newKey.setShownInUpdateView(false);
+        }
         return this;
     }