[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 dcavins):

 Replying to [comment:24 boonebgorges]:

 I agree that mixed access settings based on group status is not the norm,
 and also agree that a completely flexible approach (overriding
 `user_can_visit()` and `user_can_see_nav_item()`) will handle those cases.
 I think most plugins won't need to specify anything and can rely on your
 defaults (public groups => anyone, private and hidden => members). We can
 add an example to the Group Extension API page with an example switch
 based on group status and user membership role to show a more complicated
 example.

 Yes, I looked at the example extensions and like the directness of the new
 arguments. I mean, it loses some of the mystery and wonder for the
 developer, but I think everyone'll adjust. :)

 I'll test this patch asap. Thanks again.

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