[buddypress-trac] [BuddyPress Trac] #4785: Decouple "visibility" and "access" properties

[buddypress-trac]

#4785: Decouple "visibility" and "access" properties
-------------------------+------------------
 Reporter:  smninja      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  2.1
Component:  Groups       |     Version:  1.7
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------

Comment (by boonebgorges):

 > Don't show site admins every possible tab.

 Yes, this looks reasonable to me.

 >  Include "noone" access and show_tab cases

 Sweet, looks great.

 > Which reminds me that we'll need to add to the API an example of how to
 add a function for enabling plugins per group (I'm guessing you'd want to
 set the access level to something other than "noone" for "enabled" or
 "noone" for "disabled" based on the group id.)

 The changes in this ticket aren't really designed to enable this. The
 changes being considered here are really relevant only to the `display()`
 method of `BP_Group_Extension` plugins; group-specific stuff would ideally
 happen in a more general way than that. See #2673 for background. That
 said, it would be possible to do what you're suggesting using custom
 `user_can_see_nav_item()` and `user_can_visit()` methods (something like:
 `return in_array( $this->group_id,
 $array_of_groups_that_should_see_this;`)

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