WordPress.org

Make WordPress Core

Opened 3 weeks ago

Last modified 6 days ago

#44490 new enhancement

get_the_date() should be calling get_post_time()

Reported by: Rarst Owned by:
Milestone: 5.0 Priority: normal
Severity: minor Version: 3.0
Component: Date/Time Keywords: has-patch has-unit-tests
Focuses: Cc:

Description

Date/Time template tags follow a certain hierarchy with one exception:

  • the_time() > get_the_time() > get_post_time() > mysql2date()
  • the_modified_time() > get_the_modified_time() > get_post_modified_time() > mysql2date()
  • the_date() > get_the_date() > mysql2date()
  • the_modified_date() > get_the_modified_date() > get_post_modified_time() > mysql2date()

There is get_the_date() > get_post_time() level missing. Which means that low level get_post_time filter in it will only affect time branch, but not date branch. This is inconsistent and potentially unexpected.

Will work on a patch to adjust get_the_date() implementation.

Attachments (2)

trac-44490.diff (595 bytes) - added by Rarst 3 weeks ago.
Changed get_the_date() to call get_post_time() for consistency.
trac-44490.2.diff (3.3 KB) - added by Rarst 6 days ago.
Added test for hooks fired.

Download all attachments as: .zip

Change History (5)

@Rarst
3 weeks ago

Changed get_the_date() to call get_post_time() for consistency.

#1 @Rarst
3 weeks ago

  • Keywords has-patch added

#2 @flixos90
3 weeks ago

  • Keywords needs-unit-tests added
  • Milestone changed from Awaiting Review to 5.0

A few tests here could be useful, ensuring that the intended filters are fired for all these functions.

@Rarst
6 days ago

Added test for hooks fired.

#3 @Rarst
6 days ago

  • Keywords has-unit-tests added; needs-unit-tests removed
Note: See TracTickets for help on using tickets.