Ticket #7608: 7608.01.diff

src/bp-activity/bp-activity-screens.php

@@ -230 +230 @@
                         return;
                 }
 
-                // Check to see if the group is not public, if so, check the
-                // user has access to see this activity.
+                // Check to see if the user has access to to the activity's parent group.
                 if ( $group = groups_get_group( $activity->item_id ) ) {
-
-                        // Group is not public.
-                        if ( 'public' != $group->status ) {
-
-                                // User is not a member of group.
-                                if ( !groups_is_user_member( bp_loggedin_user_id(), $group->id ) ) {
-                                        $has_access = false;
-                                }
-                        }
+                        $has_access = $group->user_has_access;
                 }
         }
 

src/bp-groups/bp-groups-actions.php

@@ -41 +41 @@
 
         $current_group   = groups_get_current_group();
         $user_has_access = $current_group->user_has_access;
+        $is_visible      = $current_group->is_visible;
         $no_access_args  = array();
 
-        if ( ! $user_has_access && 'hidden' !== $current_group->status ) {
+        // The user can know about the group but doesn't have full access.
+        if ( ! $user_has_access && $is_visible ) {
                 // Always allow access to home and request-membership.
                 if ( bp_is_current_action( 'home' ) || bp_is_current_action( 'request-membership' ) ) {
                         $user_has_access = true;
@@ -90 +92 @@
                 return;
         }
 
-        // Hidden groups should return a 404 for non-members.
+        // Groups that the user cannot know about should return a 404 for non-members.
         // Unset the current group so that you're not redirected
         // to the default group tab.
-        if ( 'hidden' == $current_group->status ) {
+        if ( ! $is_visible ) {
                 buddypress()->groups->current_group = 0;
                 buddypress()->is_single_item        = false;
                 bp_do_404();
@@ -488 +490 @@
                         bp_core_add_message( __( 'You successfully left the group.', 'buddypress' ) );
                 }
 
-                $redirect = bp_get_group_permalink( groups_get_current_group() );
+                $group = groups_get_current_group();
+                $redirect = bp_get_group_permalink( $group );
 
-                if( 'hidden' == $bp->groups->current_group->status ) {
+                if ( ! $group->is_visible ) {
                         $redirect = trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() );
                 }
 

src/bp-groups/bp-groups-template.php

@@ -485 +485 @@
                 $group =& $groups_template->group;
         }
 
-        if ( 'public' == $group->status ) {
+        if ( ! empty( $group->is_visible ) ) {
                 return true;
-        } else {
-                if ( groups_is_user_member( bp_loggedin_user_id(), $group->id ) ) {
-                        return true;
-                }
         }
 
         return false;