From a6848b2d684016dbd26cbd6c691f15c302c52696 Mon Sep 17 00:00:00 2001
From: Adam Rauch <adam@labkey.com>
Date: Wed, 19 Mar 2025 17:32:37 -0700
Subject: [PATCH] Don't hide the first column in a display column group

---
 api/src/org/labkey/api/data/DisplayColumnGroup.java | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/api/src/org/labkey/api/data/DisplayColumnGroup.java b/api/src/org/labkey/api/data/DisplayColumnGroup.java
index 251f8be7f03..34f8cd0a39a 100644
--- a/api/src/org/labkey/api/data/DisplayColumnGroup.java
+++ b/api/src/org/labkey/api/data/DisplayColumnGroup.java
@@ -64,7 +64,10 @@ public void writeSameCheckboxCell(RenderContext ctx, HtmlWriter out)
                 new InputBuilder<>().type("checkbox").name(id).id(id).appendTo(out);
                 StringBuilder onChange = new StringBuilder("b = this.checked;");
 
-                getColumns().forEach(col -> {
+                // Index starts at 1 -- always leave the first column visible
+                for (int i = 1; i < _columns.size(); i++)
+                {
+                    DisplayColumn col = getColumns().get(i);
                     ColumnInfo colInfo = col.getColumnInfo();
                     if (colInfo != null)
                     {
@@ -72,7 +75,7 @@ public void writeSameCheckboxCell(RenderContext ctx, HtmlWriter out)
                             .append(col.getFormFieldName(ctx))
                             .append("')[0].style.display = b ? 'none' : 'block';\n");
                     }
-                });
+                }
 
                 onChange.append(" if (b) { ")
                     .append(getGroupFormFieldName(ctx))