Changeset 2001

Timestamp:

10/20/2015 09:47:41 AM (9 years ago)

Author:

ocean90

Message:

Rosetta Roles: Add a 'per_page' screen option for the list table.

Location:

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles

Files:

• class-translation-editors-list-table.php (modified) (1 diff)
• rosetta-roles.php (modified) (3 diffs)

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/class-translation-editors-list-table.php

@@ -67 +67 @@
      */
     public function prepare_items() {
-        $search = isset( $_REQUEST['s'] ) ? wp_unslash( trim( $_REQUEST['s'] ) ) : '';
-        $per_page = 10;
-        $paged = $this->get_pagenum();
+        $search =   isset( $_REQUEST['s'] ) ? wp_unslash( trim( $_REQUEST['s'] ) ) : '';
+        $per_page = $this->get_items_per_page( 'translation_editors_per_page', 10 );
+        $paged =    $this->get_pagenum();
 
         $args = array(

sites/trunk/global.wordpress.org/public_html/wp-content/mu-plugins/roles/rosetta-roles.php

@@ -47 +47 @@
         add_action( 'admin_menu', array( $this, 'register_translation_editors_page' ) );
         add_filter( 'user_row_actions', array( $this, 'promote_user_to_translation_editor' ), 10, 2 );
+        add_filter( 'set-screen-option', array( $this, 'save_custom_screen_options' ), 10, 3 );
     }
 
@@ -210 +211 @@
 
         add_action( 'load-' . $this->translation_editors_page, array( $this, 'load_translation_editors_page' ) );
+        add_action( 'load-' . $this->translation_editors_page, array( $this, 'register_screen_options' ) );
         add_action( 'admin_print_scripts-' . $this->translation_editors_page, array( $this, 'enqueue_scripts' ) );
         add_action( 'admin_footer-' . $this->translation_editors_page, array( $this, 'print_js_templates' ) );
@@ -258 +260 @@
 
     /**
+     * Registers a 'per_page' screen option for the list table.
+     */
+    public function register_screen_options() {
+        $option = 'per_page';
+        $args   = array(
+            'default' => 10,
+            'option'  => 'translation_editors_per_page'
+        );
+        add_screen_option( $option, $args );
+    }
+
+    /**
+     * Adds the 'per_page' screen option to the whitelist so it gets saved.
+     *
+     * @param bool|int $new_value Screen option value. Default false to skip.
+     * @param string   $option    The option name.
+     * @param int      $value     The number of rows to use.
+     * @return bool|int New screen option value.
+     */
+    public function save_custom_screen_options( $new_value, $option, $value ) {
+        if ( 'translation_editors_per_page' !== $option ) {
+            return $new_value;
+        }
+
+        $value = (int) $value;
+        if ( $value < 1 || $value > 999 ) {
+            return $new_value;
+        }
+
+        return $value;
+    }
+
+    /**
      * Loads either the overview or the edit handler.
      */