Make WordPress Core

Opened 17 months ago

Closed 17 months ago

Last modified 17 months ago

#56667 closed defect (bug) (fixed)

Twenty Ten: Escape 'get_permalink' where appropriate in 'twentyten/functions.php'

Reported by: hztyfoon's profile hztyfoon Owned by: audrasjb's profile audrasjb
Milestone: 6.1 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch commit
Focuses: coding-standards Cc:


In 'wp-content/themes/twentyten/functions.php' there are multiple instances where I think esc_url should be used.

To me, it's also a basic safety practice to use 'esc_url' where appropriate.
So, I think they should be escaped.

Change History (11)

This ticket was mentioned in PR #3343 on WordPress/wordpress-develop by hz-tyfoon.

17 months ago

  • Keywords has-patch added


Trac ticket: 56667

#2 @mukesh27
17 months ago

  • Component changed from General to Bundled Theme

#3 @robinwpdeveloper
17 months ago

PR Looks good to me.
Thanks @hztyfoon

#4 @mukesh27
17 months ago

Hi there! Welcome to the Trac.

Thanks for the Ticket and PR!

The PR looks good to me. Approved.

#5 @rudlinkon
17 months ago

Thank you @hztyfoon the PR looks also good to me.

fuadragib01 commented on PR #3343:

17 months ago

Ok, LGTM. Thank you @hz-tyfoon

#7 @SergeyBiryukov
17 months ago

  • Milestone changed from Awaiting Review to 6.1
  • Summary changed from Coding Standards: Escape 'get_permalink' where appropriate in 'twentyten/functions.php' to Twenty Ten: Escape 'get_permalink' where appropriate in 'twentyten/functions.php'

#8 @audrasjb
17 months ago

  • Owner set to audrasjb
  • Status changed from new to reviewing

Self assigning for commit consideration.

#9 @audrasjb
17 months ago

  • Keywords commit added

OK, this would bring more consistency with coding standards, add also with other bundled themes.

#10 @audrasjb
17 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 54341:

Twenty Ten: Escape get_permalink() where appropriate in functions.php file.

This changeset adds missing esc_url() to some instances of get_permalink() used in Twenty Ten, as per WordPress Coding Standards.

Props hztyfoon, robinwpdeveloper, mukesh27, rudlinkon, fuadragib01.
Fixes #56667.

Note: See TracTickets for help on using tickets.