WordPress.org

Make WordPress Core

Opened 5 years ago

Last modified 15 months ago

#14900 new enhancement

link-template string builder cleanup

Reported by: niallkennedy Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: template Cc:

Description

Convert dynamic strings to static strings where quick wins are available.

Wrap output such as translated strings in an esc_attr or esc_html just in case HTML can better handle what a parent page's character encoding can't.

Attachments (1)

link-template.diff (5.8 KB) - added by niallkennedy 5 years ago.
staticize strings. escape some strings for HTML output.

Download all attachments as: .zip

Change History (4)

@niallkennedy5 years ago

staticize strings. escape some strings for HTML output.

comment:1 @dd325 years ago

  • Component changed from Permalinks to Template

Looks good to me so far. One thing of note is that I have the feeling the change of 'esc_url_raw' to 'esc_url' in the final patch chunk there may be wrong..

comment:2 @markjaquith4 years ago

  • Milestone changed from Awaiting Review to Future Release

Don't forget about the translation-enabled escaping functions, like esc_attr__()!

comment:3 @nacin15 months ago

  • Component changed from Template to General
  • Focuses template added
Note: See TracTickets for help on using tickets.