Skip to:
Content

BuddyPress.org

Opened 10 years ago

Closed 10 years ago

#3847 closed defect (bug) (fixed)

New BuddyPress bp-general-settings menu items have bad links

Reported by: modemlooper Owned by:
Milestone: 1.6 Priority: normal
Severity: major Version:
Component: Core Keywords:
Cc:

Description

The links seems to be missing: admin.php?page=

url.com/wp-admin/bp-share-it

should be

url.com/wp-admin/admin.php?page=bp-share-it

Change History (4)

#1 @modemlooper
10 years ago

forgot to add;

wp 3.2.1
bp 1.5.1 single upgraded to trunk
bp-default

#2 @modemlooper
10 years ago

	add_submenu_page(
			'bp-general-settings.php', //<--this doesn't work with .php at end **********************
			__( 'BuddyPress Share It', 'buddypress' ),
			__( 'BuddyPress Share It', 'buddypress' ),
			'manage_options',
			'bpsi-options',
			'bpsi_plugin_options'
		);

removing .php adds the menu item but link is wrong. changing bp-general-settings to options-general.php adds menu item to settings and link works fine.

#3 @DJPaul
10 years ago

  • Milestone changed from Awaiting Review to 1.6

If you hook with a priority of 99, it works. Shouldn't require that, needs fixing.

#4 @djpaul
10 years ago

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

(In [5557]) Register core admin menus earlier, so existing plugins can continue hooking in to the old top-level menu. Fixes #3847

Note: See TracTickets for help on using tickets.