WordPress.org

Make WordPress Core

Opened 4 years ago

Last modified 2 years ago

#42942 new feature request

Provide a post_date_column_time hook for media files

Reported by: ivanlutrov Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 5.1
Component: Administration Keywords: has-patch dev-feedback
Focuses: Cc:

Description

Class class-wp-posts-list-table.php has column_date() function which uses post_date_column_time hook to filter the date.

Class class-wp-media-list-table.php has column_date() function which has no hook to filter the date.

Attachments (2)

42942.diff (716 bytes) - added by lopo 3 years ago.
Add a media_date_column_time filter
42942.2.diff (733 bytes) - added by lopo 11 months ago.
Patch refreshed

Download all attachments as: .zip

Change History (11)

@lopo
3 years ago

Add a media_date_column_time filter

#1 @lopo
3 years ago

  • Keywords has-patch added

#2 @Mte90
3 years ago

  • Keywords dev-feedback added

#3 @SergeyBiryukov
3 years ago

  • Milestone changed from Awaiting Review to 5.0

#4 @ivanlutrov
3 years ago

Is there a reason we can't just use post_date_column_time for this? That filter is used by WP for posts and pages, as well as custom post types so it makes sense just to use the same filter for all of them.

#5 @lopo
3 years ago

I proposed a new filter for a number of related reasons:

  • UI for media list and post list is different, we are talking of two different classes/files after all;
  • defining the same hook in two different files can be misleading and poorly maintainable (if we ever have to change it, it will have to be done in both locations, etc.);
  • in class-wp-posts-list-table.php the filter has a $mode parameter which is not defined for media:
    apply_filters( 'post_date_column_time', $h_time, $post, 'date', $mode )
    
  • it's true that media are stored and managed as WP_Post objects, but they're quite different in various regards (they usually have inherit status, their post_date is usually set at upload and doesn't change...)
  • it could happen that you want to use the hook for posts/pages/custom type items but not for media, and with two distinct hooks you can do it easily without having to resort to if/switch/whatever: for example when you are dealing with future posts, which makes no sense for media (I think it's one of the most likely use cases);
  • conversely, if you want to hook the same function to both filters it can be done with just one line more.
Last edited 2 years ago by lopo (previous) (diff)

This ticket was mentioned in Slack in #core-media by joemcgill. View the logs.


3 years ago

#7 @peterwilsoncc
3 years ago

  • Milestone changed from 5.0 to 5.1

Moving to the 5.1 milestone due to the WordPress 5.0 focus on the new
editor (Gutenberg).

This ticket was mentioned in Slack in #core-media by joemcgill. View the logs.


2 years ago

#9 @desrosj
2 years ago

  • Component changed from Media to Administration
  • Milestone changed from 5.1 to Future Release

This one still needs a review. It also feels like more of an Administration component request, so updating accordingly. Going to punt it so that the component maintainers can milestone it according to their ability to address this.

@lopo
11 months ago

Patch refreshed

Note: See TracTickets for help on using tickets.