#39710 closed enhancement (fixed)
Deprecate unused `WP_Media_List_Table::column_desc()`
Reported by: | iandunn | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | trivial | Version: | 4.3 |
Component: | Media | Keywords: | needs-dev-note has-patch commit |
Focuses: | Cc: |
Description
As far as I can tell, WP_Media_List_Table::column_desc()
never gets called by anything in Core. It doesn't look like there's ever been a Description
column in that table.
I think it was just accidentally added when the table was broken out into it's own file (list-table-media.php
at the time) from default-list-tables.php
in r15491. It might go back even further, but that's when I stopped digging :)
Attachments (2)
Change History (21)
This ticket was mentioned in Slack in #core-media by antpb. View the logs.
2 years ago
#3
@
2 years ago
- Milestone changed from Awaiting Review to 6.2
This ticket has had no movement in some years and it seems that only a handful of plugins are using WP_Media_List_Table: https://wpdirectory.net/search/01GE2BXQRQC39EN3RJ0MP3X7B2 and of those we were not able to find any using column_desc
in a spot check.
It was decided in the recent media component meeting during a bug scrub of older tickets that we should tackle this in 6.2 to stop any future use of this function.
This ticket was mentioned in Slack in #core-media by antpb. View the logs.
2 years ago
This ticket was mentioned in Slack in #core by costdev. View the logs.
2 years ago
#6
@
2 years ago
- Keywords has-patch commit added
- Owner set to audrasjb
- Status changed from new to accepted
Indeed, let's deprecate this unused method.
Self assigning for commit
.
#7
@
2 years ago
- Keywords needs-dev-note added
Adding needs-dev-note
so we can mention this deprecation in a quick note in the Misc chanfges (or Media changes) dev note.
This ticket was mentioned in PR #3935 on WordPress/wordpress-develop by @audrasjb.
2 years ago
#9
@audrasjb commented on PR #3935:
2 years ago
#12
Committed in https://core.trac.wordpress.org/changeset/55159
#13
@
2 years ago
- Keywords has-patch commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
More frequently than not, WordPress maintains the functionality of a deprecated function for backward compatibility, see https://core.trac.wordpress.org/changeset/49146#file1 as an example.
There are some exceptions when it's not possible but I don't think this is such a case.
I've reopened the ticket for consideration.
This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.
2 years ago
#15
@
2 years ago
- Keywords needs-patch added
- Version set to 4.3
This ticket was discussed in the recent bug scrub.
This needs a follow-up patch to address Peter's feedback.
Props to @costdev
This ticket was mentioned in PR #4018 on WordPress/wordpress-develop by @costdev.
2 years ago
#16
- Keywords has-patch added; needs-patch removed
Trac ticket: https://core.trac.wordpress.org/ticket/39710
#17
@
2 years ago
- Keywords commit added
I have submitted PR 4018 to address @peterwilsoncc's feedback.
Adding for commit
consideration.
@audrasjb commented on PR #4018:
2 years ago
#19
committed in https://core.trac.wordpress.org/changeset/55249
It looks like nothing uses it, but it might be used by anyone who might be extending the class. I don't see any harm in keeping it in there for those purposes, but otherwise, it could be removed.