Skip to:

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#2701 closed defect (bug) (fixed)

BP_Group_Extension - Adding Navigation Item

Reported by: andrewteg's profile andrewteg Owned by: djpaul's profile DJPaul
Milestone: 1.2.7 Priority: major
Severity: Version:
Component: Core Keywords: BP_Group_Extension, nav_item, group extension api
Cc: andrewteg


I have found that group extension api is no longer functioning properly when upgrading to 1.2.6 in regards to creating navigation items for a group. In, the sample code to create new group tabs (attached) properly displays "My Group Extension" for all users, even logged out users. However, in 1.2.6, it only shows for an admin user. This is using the sample code from the codex at and the referenced loader file "out of the box" without any modification. I first created a forum post today at before I was able to verify it is indeed a bug. Below were my steps to recreate this bug:

Log of Actions

Install WP 3.0.1
Install BP
Change Permalinks
Active BP Default Theme
Active My Plugin (bp_plugin)
Create Group
Create Subscriber Level User
View as admin = OK on both versions!
View as Subscriber = Tab Missing on 1.2.6, OK on!


Attachments (1) (1.5 KB) - added by andrewteg 14 years ago.

Download all attachments as: .zip

Change History (7)

14 years ago

#1 @andrewteg
14 years ago

I have put the two test versions I did online at You can login as admin/admin or test/test to see the issue. You can see the issue demonstrated directly at these links:

No "My Group Extension" tab unless admin/admin @

Always "My Group Extension" tab @


#2 @andrewteg
14 years ago

  • Cc andrewteg added

I'm not sure if you take fixes or not, but here is what I believe will fix this bug:

File: bp-groups-classes.php
Class: class BP_Group_Extension (line 1136)
Function: function _register() (line 1183)
Fix: line 1198

Currently: if ( $this->enable_edit_item ) {
Proposed Fix: if ( $this->enable_edit_item && $bp->is_item_admin ) {

You can then remove the true problem lines of 1201-1203 where it returns false if a user is not an item admin. Most users should not be item admins but the default is properly set to enable_edit_item so admins can edit it. I believe this means non admins hit that return on line 1203 so the code to only show the tab to non-admins starting on line 1220 will never fire, making the group extension API non-visible to non-admins. By checking both cases initially it should allow non-admins to just skip that code block and let line 1220 fire based on settings but not have even direct URL admin access to the plugin's admin screen while admins can still view and admin just fine.

Thanks for considering this,

#3 @cnorris23
14 years ago

  • Milestone changed from 1.2.6 to 1.2.7

#4 @DJPaul
14 years ago

  • Owner set to DJPaul
  • Status changed from new to assigned

We love receiving fixes! This is open source development, after all. Thank you very much for the sample plugin to demonstrate the problem.

#5 @djpaul
14 years ago

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

(In [3374]) Fixes #2701, props andrewteg (branch)

#6 @djpaul
14 years ago

(In [3375]) Fixes #2701, props andrewteg (trunk)

Note: See TracTickets for help on using tickets.