Skip to content

ADMINTHEME-124 Select the v2 admin layout without aborting the request so postLayoutRender fires - #41

Merged
DominicWatson merged 2 commits into
stablefrom
feature-ADMINTHEME-124_fix-postlayoutrender-abort
Aug 21, 2026
Merged

ADMINTHEME-124 Select the v2 admin layout without aborting the request so postLayoutRender fires#41
DominicWatson merged 2 commits into
stablefrom
feature-ADMINTHEME-124_fix-postlayoutrender-abort

Conversation

@pixl8-jg

Copy link
Copy Markdown
Contributor

This PR fixes the v2 admin layout aborting the request before postLayoutRender can fire. The v2 admin and admin-login layouts selected their v2 chrome with an include followed by an abort; because the abort ends the request from inside renderLayout(), ColdBox never announced the postLayoutRender interception point, so no interceptor or extension could alter the rendered admin page. This replaces the include-and-abort with an if/else so renderLayout() returns normally and postLayoutRender fires, for both the v1 and v2 layouts. Rendered output is unchanged for both versions; only the request lifecycle changes.

@DominicWatson
DominicWatson merged commit f548269 into stable Aug 21, 2026
2 checks passed
@DominicWatson
DominicWatson deleted the feature-ADMINTHEME-124_fix-postlayoutrender-abort branch August 21, 2026 18:47
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