WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#12024 closed defect (bug) (fixed)

Move admin-only deprecated functions to wp-admin/includes/deprecated.php

Reported by: nacin Owned by:
Milestone: 3.0 Priority: normal
Severity: normal Version:
Component: Inline Docs Keywords: has-patch
Focuses: Cc:

Description

We have a few admin-only functions that were deprecated and moved to wp-includes/deprecated.php without checking for is_admin() before declaring the function.

These should be moved from wp-includes/deprecated.php to wp-admin/includes/deprecated.php.

I'll go through the ones we have there now and figure out which ones ought to be moved.

Attachments (1)

12024.diff (1.9 KB) - added by nacin 4 years ago.

Download all attachments as: .zip

Change History (3)

comment:1 nacin4 years ago

  • Keywords has-patch added

Apparently, there's only two:

function tinymce_include()
function documentation_link()

There are a few more that need to be deprecated to begin with, but that can be handled in #11388.

Patch attached.

nacin4 years ago

comment:2 ryan4 years ago

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

(In [12871]) Move deprecated admin functions to wp-admin/includes/deprecated.php. Props nacin. fixes #12024

Note: See TracTickets for help on using tickets.