Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 15 years ago

#3184 closed enhancement (fixed)

deprecate bp_core_add_admin_menu_page()?

Reported by: DJPaul Owned by: DJPaul
Priority: normal Milestone: 1.5
Component: Core Version:
Severity: Keywords:
Cc:

Description

BuddyPress has long had bp_core_add_admin_menu_page(), its own version of add_admin_menu_page(), because "[it] allows [better] positioning of menus."

However, I'm looking at add_menu_page() and it seems to have a parameter to let us position the menu where we want, and the rest of the code looks pretty similar. Could someone check? I'd like to deprecate bp_core_add_admin_menu_page() and call add_admin_menu_page() instead.

Of course, if add_admin_menu_page() does miss something we find useful, we can send a patch upstream.

Change History (2)

#1 @DJPaul
15 years ago

  • Milestone Awaiting Review → 1.3
  • Owner set to DJPaul
  • Status new → assigned

#2 @djpaul
15 years ago

  • Resolution → fixed
  • Status assigned → closed

(In [4309]) Deprecate bp_core_add_admin_menu_page() in favour of add_menu_page(). Fixes #3184

Note: See TracTickets for help on using tickets.