Support disabling managed columns from a trigger

Author: labkey-nicka

api/src/org/labkey/api/data/AbstractTableInfo.java

@@ -2019,7 +2019,7 @@ public void fireRowTrigger(
                 break;
 
             // trackedRow should only be null when not manageColumns
-            if (trackedRow != null)
+            if (trackedRow != null && script.isManagedColumnsEnabled())
             {
                 var managed = script.getManagedColumns();
                 var managedCols = managed != null ? managed.getColumns(type) : null;

api/src/org/labkey/api/data/triggers/ScriptTrigger.java

@@ -66,6 +66,7 @@ public class ScriptTrigger implements Trigger
     @NotNull protected final TableInfo _table;
     @NotNull protected final ScriptReference _script;
     @Nullable protected volatile ManagedColumns _managedColumns = null;
+    protected volatile Boolean _isManagedColumnsEnabled = null;
 
     protected ScriptTrigger(@NotNull Container c, @NotNull TableInfo table, @NotNull ScriptReference script)
     {
@@ -119,25 +120,34 @@ public boolean canStream()
     public @Nullable ManagedColumns getManagedColumns()
     {
         if (_managedColumns == null)
-            _managedColumns = resolveManagedColumns();
+        {
+            var user = _table.getUserSchema() != null ? _table.getUserSchema().getUser() : null;
+            var result = _invokeTableScript(_container, user, Object.class, "managedColumns", null, () -> null);
+            _isManagedColumnsEnabled = !(result instanceof Boolean enabled) || enabled;
+
+            if (result instanceof Map<?, ?> map)
+            {
+                var insert = managedColumnsFromScriptMap(map, "insert");
+                var update = managedColumnsFromScriptMap(map, "update");
+                var ignored = managedColumnsFromScriptMap(map, "ignored");
+
+                _managedColumns = new ManagedColumns(insert, update, ignored);
+            }
+            else
+                _managedColumns = ManagedColumns.empty();
+        }
 
         return _managedColumns;
     }
 
-    private @NotNull ManagedColumns resolveManagedColumns()
+    @Override
+    public boolean isManagedColumnsEnabled()
     {
-        var user = _table.getUserSchema() != null ? _table.getUserSchema().getUser() : null;
-        var result = _invokeTableScript(_container, user, Object.class, "managedColumns", null, () -> null);
-        if (result instanceof Map<?, ?> map)
-        {
-            var insert = managedColumnsFromScriptMap(map, "insert");
-            var update = managedColumnsFromScriptMap(map, "update");
-            var ignored = managedColumnsFromScriptMap(map, "ignored");
-
-            return new ManagedColumns(insert, update, ignored);
-        }
+        // Ensure the flag is initialized
+        if (_isManagedColumnsEnabled == null)
+            getManagedColumns();
 
-        return ManagedColumns.empty();
+        return _isManagedColumnsEnabled;
     }
 
     private @NotNull Set<String> managedColumnsFromScriptMap(@NotNull Map<?, ?> map, String key)

api/src/org/labkey/api/data/triggers/Trigger.java

@@ -137,6 +137,14 @@ public static ManagedColumns empty()
         return null;
     }
 
+    /**
+     * Returns true if managed columns are enabled for this trigger.
+     */
+    default boolean isManagedColumnsEnabled()
+    {
+        return true;
+    }
+
     /**
      * Ensures all columns declared by {@link #getManagedColumns()} are present in {@code newRow}
      * before INSERT trigger fires.