Skip to content

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

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 app/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 app/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

boxlang@be Test Results

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

Results for commit fa5e1f7. ± Comparison against base commit 4009e5e.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

boxlang@1 Test Results

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

Results for commit fa5e1f7. ± Comparison against base commit 4009e5e.

♻️ 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 fa5e1f7. ± Comparison against base commit 4009e5e.

♻️ 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 fa5e1f7. ± Comparison against base commit 4009e5e.

♻️ This comment has been updated with latest results.

@lmajano
lmajano merged commit 5efd24c into development Sep 29, 2026
13 of 15 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