Skip to content

Commit f955ee4

Browse files
committed
bug #58919 [WebProfilerBundle] Twig deprecations (mazodude)
This PR was submitted for the 7.1 branch but it was merged into the 5.4 branch instead. Discussion ---------- [WebProfilerBundle] Twig deprecations | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Issues | N/A | License | MIT This fixes some twig deprecations in the Web Profiler Bundle. Since twig/twig 3.15: Add explicit parentheses around the "??" binary operator to avoid behavior change in the next major version as its precedence will change in "`@WebProfiler`/Collector/notifier.html.twig" at line 141 Not entirely sure which branch this should go against. Also not sure if this is a bugfix as it isn't really a bug. Please feel free to correct it Commits ------- 0a868c4 Fix twig deprecations in web profiler twig files
2 parents b815547 + 0a868c4 commit f955ee4

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/notifier.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@
138138
{{- 'Content: ' ~ notification.getContent() }}<br/>
139139
{{- 'Importance: ' ~ notification.getImportance() }}<br/>
140140
{{- 'Emoji: ' ~ (notification.getEmoji() is empty ? '(empty)' : notification.getEmoji()) }}<br/>
141-
{{- 'Exception: ' ~ notification.getException() ?? '(empty)' }}<br/>
141+
{{- 'Exception: ' ~ (notification.getException() ?? '(empty)') }}<br/>
142142
{{- 'ExceptionAsString: ' ~ (notification.getExceptionAsString() is empty ? '(empty)' : notification.getExceptionAsString()) }}
143143
</pre>
144144
</div>

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy