Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#34387 closed enhancement (wontfix)

New function: `the_theme_mod()`

Reported by:	sebastian.pisula	Owned by:
Priority:	normal	Milestone:
Component:	Themes	Version:
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:

Description

For example:

<img src="<?php the_theme_mod( 'logo' ); ?>" alt=""/>

OR

<div><?php the_theme_mod( 'copyright_text' ); ?></div>

Attachments (1)

34387.patch (802 bytes ) - added by sebastian.pisula 11 years ago.

Download all attachments as: .zip

Change History (6)

@sebastian.pisula
11 years ago

Attachment 34387.patch added

#1 @sebastian.pisula
11 years ago

Keywords has-patch added

#2 @swissspidy
11 years ago

You should always escape your output, which is not possible with a generic the_theme_mod function.

I suggest using something like echo esc_url( get_theme_mod( 'logo' ) ) and echo esc_html( get_theme_mod( 'copyright_text' ) ) instead.

#3 @swissspidy
11 years ago

Keywords close added

#4 @SergeyBiryukov
11 years ago

Keywords close removed
Milestone Awaiting Review
Resolution → wontfix
Status new → closed

#5 @ocean90
8 years ago

#45258 was marked as a duplicate.

Note: See TracTickets for help on using tickets.

Download in other formats: