Skip to content

[5.4] Don't re-escape a View instance passed as the default value to yield or section blade directives - #19884

Merged
taylorotwell merged 3 commits into
laravel:5.4from
ockle:do-not-escape-yield-default-if-view
Jul 3, 2017
Merged

taylorotwell merged 3 commits into
laravel:5.4from
ockle:do-not-escape-yield-default-if-view

Conversation

@ockle

@ockle ockle commented Jul 3, 2017

Copy link
Copy Markdown
Contributor

Based on feedback in #19643, this changes the handling of the default parameter of @yield to detect that a View instance has been passed and will no longer try to re-escape it. For consistency, this pull also changes the handling of the default parameter of @section to behave the same way.

@ockle ockle changed the title Don't re-escape a View instance passed as the default value to yield or section blade directives [5.4] Don't re-escape a View instance passed as the default value to yield or section blade directives Jul 3, 2017
@taylorotwell
taylorotwell merged commit 405fd27 into laravel:5.4 Jul 3, 2017
@vrgenl

vrgenl commented Jul 4, 2017

Copy link
Copy Markdown

Sweet!

salmanarshad321 pushed a commit to salmanarshad321/framework that referenced this pull request Sep 23, 2026
…yield or section blade directives (laravel#19884)

* don't re-escape a View instance passed as the default value to @yield

* don't re-escape a View instance passed as the default value to @section

* Update ManagesLayouts.php
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