Changeset 13441 for trunk/src/bp-groups/screens/single/admin/delete-group.php

Timestamp:

03/23/2023 07:35:38 PM (18 months ago)

Author:

imath

Message:

Make BP_Core_Nav generate "BP Rewrites ready" navigation links

• Remove all components $parent_url attributes when setting sub nav items.
• Only use the bp_core_create_nav_link() $link attribute argument & the bp_core_create_subnav_link() $parent_url attribute argument if specified to preserve backward compatibility.
• Migrates the Community search feature so that it uses BP Rewrites.
• Perform some bp_loggedin_user_domain() in favor of bp_loggedin_user_url().
• Update some PHPUnit tests.

Props r-a-y, johnjamesjacoby, boonebgorges

Closes ​https://github.com/buddypress/buddypress/pull/77
See #4954

File:

• trunk/src/bp-groups/screens/single/admin/delete-group.php (modified) (3 diffs)

trunk/src/bp-groups/screens/single/admin/delete-group.php

@@ -15 +15 @@
 function groups_screen_group_admin_delete_group() {
 
-    if ( 'delete-group' != bp_get_group_current_admin_tab() )
+    if ( 'delete-group' != bp_get_group_current_admin_tab() ) {
         return false;
+    }
 
-    if ( ! bp_is_item_admin() && !bp_current_user_can( 'bp_moderate' ) )
+    if ( ! bp_is_item_admin() && !bp_current_user_can( 'bp_moderate' ) ) {
         return false;
+    }
 
     $bp = buddypress();
 
     if ( isset( $_REQUEST['delete-group-button'] ) && isset( $_REQUEST['delete-group-understand'] ) ) {
+        $groups_slug = bp_get_groups_slug();
+        $redirect    = bp_loggedin_user_url(
+            array(
+                'single_item_component' => bp_rewrites_get_slug( 'members', 'member_' . $groups_slug, $groups_slug ),
+            )
+        );
 
         // Check the nonce first.
-        if ( !check_admin_referer( 'groups_delete_group' ) ) {
+        if ( ! check_admin_referer( 'groups_delete_group' ) ) {
             return false;
         }
@@ -40 +48 @@
 
         // Group admin has deleted the group, now do it.
-        if ( !groups_delete_group( $bp->groups->current_group->id ) ) {
+        if ( ! groups_delete_group( $bp->groups->current_group->id ) ) {
             bp_core_add_message( __( 'There was an error deleting the group. Please try again.', 'buddypress' ), 'error' );
         } else {
@@ -53 +61 @@
              */
             do_action( 'groups_group_deleted', $bp->groups->current_group->id );
-
-            bp_core_redirect( trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() ) );
         }
 
-        bp_core_redirect( trailingslashit( bp_loggedin_user_domain() . bp_get_groups_slug() ) );
+        bp_core_redirect( $redirect );
     }