Make WordPress Core

Opened 12 months ago

Closed 9 months ago

#38625 closed enhancement (fixed)

Explicitly specify the end of the variable name in get_page_template().

Reported by: kuck1u Owned by: SergeyBiryukov
Milestone: 4.8 Priority: low
Severity: normal Version:
Component: Themes Keywords: has-patch
Focuses: template Cc:


  • Explicitly specify the end of the variable name.

Attachments (1)

38625.diff (575 bytes) - added by kuck1u 12 months ago.

Download all attachments as: .zip

Change History (6)

12 months ago

#1 @kuck1u
12 months ago

  • Keywords has-patch added

#2 @kuck1u
12 months ago

  • Component changed from General to Themes
  • Focuses template added

#3 @johnbillion
12 months ago

  • Milestone changed from Awaiting Review to Future Release
  • Priority changed from normal to low
  • Version trunk deleted

Thanks @kuck1u.

This sort of change would normally fall under code refactoring, which the WordPress project typically avoids doing, but this does clarify the variable names and I think we can drop it in as an enhancement.

#4 @SergeyBiryukov
9 months ago

  • Milestone changed from Future Release to 4.8

#5 @SergeyBiryukov
9 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 39884:

Themes: Use curly braces for variables inside strings in `get_page_template() to explicitly specify the end of the variable name.

Props kuck1u.
Fixes #38625.

Note: See TracTickets for help on using tickets.