fix: ColumnGroup doesn't accept HtmlString as label (#19152)

[olipayne]

Fixes #19152

Problem

When using an HtmlString as a ColumnGroup label (which is allowed by the type signature), PHP throws an error:

Cannot access offset of type Illuminate\Support\HtmlString on array

This happens because the code was trying to use the HtmlString object directly as an array key, but PHP only allows strings or integers as array keys.

Solution

I used e() instead of a (string) cast to match the existing pattern in the codebase. In HasColumnManager::mapTableColumnGroupToArray(), the same conversion is already done with e():

Testing

Added a test case that reproduces the bug and verifies the fix. Without the fix, the test fails with the exact error from the issue. With the fix, all tests pass.

✓ it can use HtmlString as label

[Copilot]

Pull request overview

This PR fixes a bug where using an HtmlString as a ColumnGroup label causes a PHP error, even though the type signature allows it. The fix converts the label to a string using e() before using it as an array key, matching the existing pattern used elsewhere in the codebase.

Changes:

• Modified HasColumns::pushColumns() to convert ColumnGroup labels to strings before using them as array keys
• Added a test case that verifies HtmlString can be used as a ColumnGroup label

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/tables/src/Table/Concerns/HasColumns.php	Fixed bug by converting label to string with e() before using as array key
tests/src/Tables/Columns/ColumnGroupTest.php	Added test case and test component class to verify HtmlString labels work correctly

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.