Opened 3 months ago
Closed 2 months ago
#23664 closed enhancement (fixed)
Add esc_url() to header_image()
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Themes | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
Spotted this in Twenty Thirteen: esc_url( header_image() ) (see #23663).
It doesn't work as intended, since header_image() echoes the value directly:
http://core.trac.wordpress.org/browser/tags/3.5.1/wp-includes/theme.php#L983
I guess we should add esc_url() to header_image() itself, like we did in [14949] for the_guid() and in [23527] for the_permalink().
Attachments (1)
Change History (2)
SergeyBiryukov — 3 months ago
comment:1
SergeyBiryukov — 2 months ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.

In 23633: