Opened 5 years ago
Last modified 5 years ago
#53899 new enhancement
Escaping strings and screenshot URL on Themes screen
| Reported by: | vishitshah | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Awaiting Review |
| Component: | Themes | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
Attachments (1)
Change History (5)
#2
@
5 years ago
Concerning the screenshot URL, if we want to escape late the URL, we'll probably want to do it in the template model too:
<# if ( data.screenshot[0] ) { #>
<div class="theme-screenshot">
<img src="{{ data.screenshot[0] }}" alt="" />
</div>
<# } else { #>
<div class="theme-screenshot blank"></div>
<# } #>
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Thanks for the ticket and patch,
Just noting that the content of
$active_classis generated by the following code:$active_class = ''; if ( $theme['active'] ) { $active_class = ' active'; }Therefore, there is no way to anyone one to inject anything in this variable, so the current code seems 100% safe :)