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

Timestamp:

09/02/2013 01:02:26 AM (11 years ago)

Author:

r-a-y

Message:

In the groups dashboard, fix bulk deletion. Fixes #5105.

• Use bp_admin_list_table_current_bulk_action() to reconcile what the proper action is when doing a bulk action like deleting.
• Fix copypasta from activity admin dashboard to use group IDs.

Props imath.

File:

• trunk/bp-groups/bp-groups-admin.php (modified) (2 diffs)

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

@@ -400 +400 @@
 function bp_groups_admin() {
     // Decide whether to load the index or edit screen
-    $doaction = ! empty( $_REQUEST['action'] ) ? $_REQUEST['action'] : '';
+    $doaction = bp_admin_list_table_current_bulk_action();
 
     // Display the single group edit screen
@@ -1286 +1286 @@
      */
     function column_cb( $item = array() ) {
-        printf( '<label class="screen-reader-text" for="aid-%1$d">' . __( 'Select group %1$d', 'buddypress' ) . '</label><input type="checkbox" name="aid[]" value="%1$d" id="aid-%1$d" />', $item['id'] );
+        printf( '<label class="screen-reader-text" for="gid-%1$d">' . __( 'Select group %1$d', 'buddypress' ) . '</label><input type="checkbox" name="gid[]" value="%1$d" id="gid-%1$d" />', $item['id'] );
     }