Ticket #6123: 6123.03.patch

src/bp-core/bp-core-catchuri.php

@@ -269 +269 @@
                                         return;
                                 }
 
-                                // If the displayed user is marked as a spammer, 404 (unless logged-
-                                // in user is a super admin)
-                                if ( bp_displayed_user_id() && bp_is_user_spammer( bp_displayed_user_id() ) ) {
-                                        if ( bp_current_user_can( 'bp_moderate' ) ) {
+                                // If the displayed user is marked as a spammer, the account is not active or user is deleted, 404
+                                // (unless logged-in user is a super admin if the user is marked as spammer)
+                                if ( bp_displayed_user_id() && bp_is_user_inactive( bp_displayed_user_id() ) ) {
+
+                                        // Only display spammer if the user is a super admin
+                                        if ( bp_is_user_spammer( bp_displayed_user_id() ) && bp_current_user_can( 'bp_moderate' ) ) {
                                                 bp_core_add_message( __( 'This user has been marked as a spammer. Only site admins can view this profile.', 'buddypress' ), 'warning' );
                                         } else {
                                                 bp_do_404();

src/bp-members/bp-members-loader.php

@@ -193 +193 @@
                                 unset( $bp->canonical_stack['component'] );
                         }
 
-                        // if we're on a spammer's profile page, only users with the 'bp_moderate' cap
-                        // can view subpages on the spammer's profile
-                        //
-                        // users without the cap trying to access a spammer's subnav page will get
-                        // redirected to the root of the spammer's profile page.  this occurs by
-                        // by removing the component in the canonical stack.
-                        if ( bp_is_user_spammer( bp_displayed_user_id() ) && ! bp_current_user_can( 'bp_moderate' ) ) {
+                        // if we're on a not active user profile page, redirect to the root of the user's profile page.
+                        // this occurs by removing the component in the canonical stack.
+                        if ( bp_is_user_inactive( bp_displayed_user_id() ) && ! bp_current_user_can( 'bp_moderate' ) ) {
                                 unset( $bp->canonical_stack['component'] );
                         }
                 }