[buddypress-trac] [BuddyPress Trac] #4017: Group taxonomy

buddypress-trac noreply at wordpress.org
Tue Oct 28 03:58:47 UTC 2014


#4017: Group taxonomy
--------------------------+-----------------------------
 Reporter:  boonebgorges  |       Owner:  boonebgorges
     Type:  enhancement   |      Status:  new
 Priority:  normal        |   Milestone:  Future Release
Component:  Groups        |     Version:
 Severity:  major         |  Resolution:
 Keywords:  has-patch     |
--------------------------+-----------------------------

Comment (by imath):

 r-a-y,

 I've tested your patch. It's working fine. There's some notices due to
 {{{get_queried_object()}}}. I got rid of them resetting it from
 {{{bp_core_load_template()}}}. But not sure it's the right move.

 Personally, i don't really like to edit tags from the root blog admin, and
 then go back to network admin to edit a group, and then go back to root
 blog admin to edit tags... But i think it's the only way to do it without
 copy/pasting the edit-tags.php in the group tags admin function (what i've
 done in 4017.patch).

 I've tried another way, and thought i found an alternative to stay in
 network admin to edit tags, by using a function like this :

 {{{
 function bp_groups_admin_tags_load() {
         $post_type = 'bp_group';
         $taxnow  = $taxonomy = 'bp_group_tags';

         require_once( ABSPATH . 'wp-admin/edit-tags.php' );
         exit();
 }
 add_action( "load-$hook", 'bp_groups_admin_tags_load' );
 }}}

 At first it seemed to work fine, but AJAX, bulk actions, & search was
 failing. At several places WordPress checks for {{{'edit-tags.php'}}} :(
 So unless we have this file in network admin, i think your approach is the
 best.

 I'm adding an updated version of your patch to latest trunk.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4017#comment:22>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list