Changeset 6835 for trunk/bp-groups/bp-groups-admin.php

Timestamp:

03/04/2013 01:23:22 PM (12 years ago)

Author:

boonebgorges

Message:

Fix bug preventing groups from being deleted in bulk

Also fixes redirect after group deletion

Fixes #4861

Hat tip to imath for original patch

File:

• trunk/bp-groups/bp-groups-admin.php (modified) (1 diff)

trunk/bp-groups/bp-groups-admin.php

@@ -86 +86 @@
         check_admin_referer( 'bp-groups-delete' );
 
-        $group_id = (int) $_GET['gid'];
-
-        $result = groups_delete_group( $group_id );
-
-        if ( $result ) {
-            $redirect_to = add_query_arg( 'deleted', '1', $redirect_to );
-        } else {
-            $redirect_to = add_query_arg( array(
-                'deleted' => 0,
-                'action'  => 'edit',
-                'gid'     => $group_id
-            ) );
-        }
+        $group_ids = wp_parse_id_list( $_GET['gid'] );
+
+        $count = 0;
+        foreach ( $group_ids as $group_id ) {
+            if ( groups_delete_group( $group_id ) ) {
+                $count++;
+            }
+        }
+
+        $redirect_to = add_query_arg( 'deleted', $count, $redirect_to );
 
         bp_core_redirect( $redirect_to );