WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#21317 closed enhancement (fixed)

Should "_media_button" function be deprecated?

Reported by: martythornley Owned by: ryan
Milestone: 3.5 Priority: normal
Severity: normal Version:
Component: Media Keywords: needs-patch
Focuses: Cc:

Description

Looking at media uploads, looks like the function _media_button is not used anywhere? defined in media.php line 389 but can't find it used anywhere?

Change History (4)

#1 @scribu
5 years ago

  • Keywords needs-patch added

I guess since there's only one button now, we don't need it.

I think we can just remove it outright.

#2 @SergeyBiryukov
5 years ago

Unused since [18890].

Related: #20427

#3 @nacin
5 years ago

  • Milestone changed from Awaiting Review to 3.5

Like #20427, I'd like to at least leave it in deprecated.php in case it is called directly by a plugin. I found at least a few instances in a search of the plugins directory.

#4 @ryan
5 years ago

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

In [21353]:

Deprecate and stub _media_button. fixes #21317

Note: See TracTickets for help on using tickets.