Skip to content

[TASK] Establish proper API to set template layout #1090

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

s2b
Copy link
Contributor

@s2b s2b commented May 23, 2025

The ParsingState now provides a proper method to set the
layout of a template during parsing/compilation. Previously, the
magic variable name layoutName has been used for that
purpose, but with the new nodeInitializedEvent we can now
access the ParsingState object directly.

The usage of the old variable name has been deprecated and
will no longer be evaluated with Fluid v5. Its main usage besides
the <f:layout> ViewHelper (see above) has been the cache
warmup command, which will receive a more straightforward
implementation with Fluid v5.

@s2b s2b force-pushed the task/templateLayoutVariable branch 3 times, most recently from 481d724 to b511013 Compare May 23, 2025 16:32
The `ParsingState` now provides a proper method to set the
layout of a template during parsing/compilation. Previously, the
magic variable name `layoutName` has been used for that
purpose, but with the new `nodeInitializedEvent` we can now
access the `ParsingState` object directly.

The usage of the old variable name has been deprecated and
will no longer be evaluated with Fluid v5. Its main usage besides
the `<f:layout>` ViewHelper (see above) has been the cache
warmup command, which will receive a more straightforward
implementation with Fluid v5.
@s2b s2b force-pushed the task/templateLayoutVariable branch from b511013 to 540177b Compare May 23, 2025 16:40
@s2b
Copy link
Contributor Author

s2b commented May 23, 2025

TYPO3 Core CI is green (apart from composer integrity): https://review.typo3.org/c/Packages/TYPO3.CMS/+/89518

@s2b s2b mentioned this pull request Jun 7, 2025
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.

1 participant