WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#17182 closed enhancement (fixed)

string cleanup in general-template

Reported by: niallkennedy Owned by: nacin
Priority: normal Milestone: 3.2
Component: General Version:
Severity: minor Keywords: has-patch
Cc: Focuses:

Description

Some dynamic strings changed to static in general-template.

Also included an esc_html() of two strings passed in an args array to paginate_links(). The function recasts integer variables from the array just to be sure, and should probably escape HTML output from unknowns as well.

Attachments (1)

general-template.diff​ (4.4 KB ) - added by niallkennedy 15 years ago.
general-template string cleanup

Download all attachments as: .zip

Change History (4)

@niallkennedy
15 years ago

general-template string cleanup

#1 @scribu
15 years ago

  • Milestone Awaiting Review
  • Resolution → duplicate
  • Status new → closed

See #16474.

Feel free to re-open with only the part about extra escaping for paginate_links().

#2 @nacin
15 years ago

  • Owner set to nacin
  • Resolution duplicate → fixed

In [17989]:

Code readability improvements. props niallkennedy, fixes #17166, #17182, #17231, #16474.

#3 @nacin
15 years ago

  • Milestone → 3.2

I disagree about the escaping. Integers that are supposed to be integers are one thing -- but HTML into paginate_links() seems it may be desirable. Feel free to open a new ticket if I'm reading that wrong, as I may very well be.

Note: See TracTickets for help on using tickets.