Changeset 5930 for trunk/src/includes/admin/metaboxes.php

Timestamp:

08/25/2015 03:07:14 PM (8 years ago)

Author:

johnjamesjacoby

Message:

Topics: sanity check $_REQUEST and introduce list-table methods for reply table-navigation.

This changeset fixes a potential debug notice introduced in r5886, and removes unused functionality inherited from WP_List_Table.

Fixes #2842. See #2587.

File:

• trunk/src/includes/admin/metaboxes.php (modified) (2 diffs)

trunk/src/includes/admin/metaboxes.php

@@ -569 +569 @@
     }
 
+    // Look for pagination value
+    $page = isset( $_REQUEST['page'] )
+        ? (int) $_REQUEST['page']
+        : 0;
+
     // Load up the list table
     $replies_list_table = new BBP_Topic_Replies_List_Table();
@@ -574 +579 @@
 
     <form id="bbp-topic-replies" method="get">
-        <input type="hidden" name="page" value="<?php echo esc_attr( $_REQUEST['page'] ); ?>" />
+        <input type="hidden" name="page" value="<?php echo esc_attr( $page ); ?>" />
 
         <?php $replies_list_table->display(); ?>