Changeset 13937 for trunk/src/bp-groups/bp-groups-admin.php

Timestamp:

06/24/2024 05:29:42 PM (10 months ago)

Author:

imath

Message:

Manage BP specific WP List Tables pagination from a central function

• Introduces bp_admin_set_screen_options() to deal with WP List Table pagination

for Activity, Groups, Opt-outs & Site Invitations.

• Deprecates bp_activity_admin_screen_options() & bp_groups_admin_screen_options().

Props emaralive

Fixes #9195
Closes ​https://github.com/buddypress/buddypress/pull/320

File:

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

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

@@ -17 +17 @@
 if ( ! class_exists( 'WP_List_Table' ) ) {
     require ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
-}
-
-// The per_page screen option. Has to be hooked in extremely early.
-if ( is_admin() && ! empty( $_REQUEST['page'] ) && 'bp-groups' == $_REQUEST['page'] ) {
-    add_filter( 'set-screen-option', 'bp_groups_admin_screen_options', 10, 3 );
 }
 
@@ -525 +520 @@
 
 /**
- * Handle save/update of screen options for the Groups component admin screen.
- *
- * @since 1.7.0
- *
- * @param string $value     Will always be false unless another plugin filters it first.
- * @param string $option    Screen option name.
- * @param string $new_value Screen option form value.
- * @return string|int Option value. False to abandon update.
- */
-function bp_groups_admin_screen_options( $value, $option, $new_value ) {
-    if ( 'toplevel_page_bp_groups_per_page' != $option && 'toplevel_page_bp_groups_network_per_page' != $option ) {
-        return $value;
-    }
-
-    // Per page.
-    $new_value = (int) $new_value;
-    if ( $new_value < 1 || $new_value > 999 ) {
-        return $value;
-    }
-
-    return $new_value;
-}
-
-/**
  * Select the appropriate Groups admin screen, and output it.
  *