Skip to content

Fix main view for struct based RegisteredHandlers (ColdBox 8.2) - #30

Merged
lmajano merged 1 commit into
developmentfrom
ccr-682a144b-oo3m2o
Sep 29, 2026
Merged

lmajano merged 1 commit into
developmentfrom
ccr-682a144b-oo3m2o

Conversation

@lmajano

@lmajano lmajano commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Description

ColdBox 8.2 changed getSetting( "RegisteredHandlers" ) from a list to a struct. The default views/main/index looped it with list=, which fails with Can't cast Struct to a string.

  • Loop the setting with collection= / item=
  • Bump coldbox dependency to ^8.2.0 since the view now requires the struct shape

Type of change

  • Bug Fix

🤖 Generated with Claude Code

https://claude.ai/code/session_01436jYJmdqfg6WZ62HQeoso


Generated by Claude Code

ColdBox 8.2 stores RegisteredHandlers as a struct instead of a list, which made
the default index view fail with "Can't cast Struct to a string". Loop the
setting as a collection and require coldbox ^8.2.0.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01436jYJmdqfg6WZ62HQeoso

lmajano commented Sep 29, 2026

Copy link
Copy Markdown
Contributor Author

Checks Source Code Formatting fails, but not because of this PR.

The format:check script runs !boxlang format --check ..., and the Ortus-Solutions/commandbox-action image has no boxlang binary (/bin/sh: boxlang: not found, exit 127). This PR only touches views/main/index.cfm and box.json, not the script or the workflow, so every PR against development hits the same failure. It's deterministic, so I didn't re-run it.

No fix exists yet. Proposed patch (separate PR): install the BoxLang CLI in the formatCheck job of .github/workflows/pr.yml before running format:check, or change the script back to a formatter available in the CommandBox image.


Generated by Claude Code

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

lucee@5 Test Results

9 tests  ±0   9 ✅ ±0   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit f7de67d. ± Comparison against base commit 58c1c60.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

boxlang Test Results

9 tests  ±0   9 ✅ ±0   1s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit f7de67d. ± Comparison against base commit 58c1c60.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

lucee@6 Test Results

9 tests  ±0   9 ✅ ±0   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit f7de67d. ± Comparison against base commit 58c1c60.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

adobe@2023 Test Results

9 tests  ±0   9 ✅ ±0   1s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit f7de67d. ± Comparison against base commit 58c1c60.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

adobe@2025 Test Results

9 tests  ±0   9 ✅ ±0   1s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit f7de67d. ± Comparison against base commit 58c1c60.

♻️ This comment has been updated with latest results.

@lmajano
lmajano merged commit 3858036 into development Sep 29, 2026
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants