Skip to content
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

[TASK] Add a note about the special variable layoutName #447

Merged
merged 3 commits into from
Apr 9, 2019

Conversation

dogawaf
Copy link
Contributor

@dogawaf dogawaf commented Apr 8, 2019

Maybe the good way would be to have not any reserved variable names. But in the meantime, this one should be documented.

dogawaf referenced this pull request Apr 8, 2019
Rather than analyse a whole stack of nodes and compile those to return this single value, the value is now pulled directly from the variablecontainer.

This also means that dynamic layout names can be achieved without touching Fluid itself - simply assign a `layoutName` variable to the view before rendering and it the value of it will be used as layout name.
@coveralls
Copy link

coveralls commented Apr 8, 2019

Pull Request Test Coverage Report for Build 1172

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.389%

Totals Coverage Status
Change from base Build 1158: 0.0%
Covered Lines: 2611
Relevant Lines: 2681

💛 - Coveralls

@mbrodala mbrodala merged commit 9bb86bf into TYPO3:master Apr 9, 2019
@mbrodala
Copy link
Member

mbrodala commented Apr 9, 2019

Thanks for the note.

NamelessCoder pushed a commit to NamelessCoder/Fluid that referenced this pull request May 21, 2019
* [TASK] Add a note about the special variable layoutName

* [TASK] Simplify wording

* [TASK] Add more explanation
NamelessCoder pushed a commit that referenced this pull request Sep 8, 2019
* [TASK] Add a note about the special variable layoutName

* [TASK] Simplify wording

* [TASK] Add more explanation
NamelessCoder pushed a commit that referenced this pull request Sep 8, 2019
* [TASK] Add a note about the special variable layoutName

* [TASK] Simplify wording

* [TASK] Add more explanation
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.

3 participants