Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#27641 closed defect (bug) (fixed)

Theme Install screen displays encoded entities

Reported by: johnbillion Owned by: nacin
Priority: normal Milestone: 3.9
Component: Themes Version: 3.9
Severity: normal Keywords: 2nd-opinion has-patch
Cc: Focuses: ui

Description

The theme description (that's displayed in the left hand panel when you view a theme's details) on the theme install screen is output in the template using {{ double curly braces }} which encodes the output instead of outputting it as HTML.

The result is that ampersands show up as & instead of &. You can see this in action if you view details of "Alexandria" in the "Featured" tab.

Is it safe to use {{{ triple braces }}} instead? It should be, but could do with a second opinion as I'm not sure how much sanitising is done in the theme repo.

Attachments (1)

27641.patch​ (612 bytes ) - added by johnbillion 13 years ago.

Download all attachments as: .zip

Change History (4)

@johnbillion
13 years ago

#1 @johnbillion
13 years ago

  • Keywords has-patch added; needs-patch removed

#2 @nacin
13 years ago

Hmm. Anything coming back from WordPress.org should be considered safe, but I do not really want to treat it as safe.

#3 @nacin
13 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In 27962:

Theme Installer: Don't encode description entities/HTML.

props johnbillion.
fixes #27641.

Note: See TracTickets for help on using tickets.