Changeset 5930 for trunk/src/includes/admin/list-tables/topic-replies.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/list-tables/topic-replies.php (modified) (1 diff)

trunk/src/includes/admin/list-tables/topic-replies.php

@@ -230 +230 @@
         ) );
     }
+
+    /**
+     * Message to be displayed when there are no items
+     *
+     * @since bbPress (r5930)
+     */
+    public function no_items() {
+        esc_html_e( 'No replies to this topic.', 'bbpress' );
+    }
+
+    /**
+     * Display the list table
+     *
+     * This custom method is necessary because the one in `WP_List_Table` comes
+     * with a nonce and check that we do not need.
+     *
+     * @since bbPress (r5930)
+     */
+    public function display() {
+
+        // Top
+        $this->display_tablenav( 'top' ); ?>
+
+        <table class="wp-list-table <?php echo implode( ' ', $this->get_table_classes() ); ?>">
+            <thead>
+                <tr>
+                    <?php $this->print_column_headers(); ?>
+                </tr>
+            </thead>
+
+            <tbody id="the-list" data-wp-lists='list:<?php echo $this->_args['singular']; ?>'>
+                <?php $this->display_rows_or_placeholder(); ?>
+            </tbody>
+
+            <tfoot>
+                <tr>
+                    <?php $this->print_column_headers( false ); ?>
+                </tr>
+            </tfoot>
+        </table>
+
+        <?php
+
+        // Bottom
+        $this->display_tablenav( 'bottom' );
+    }
+
+    /**
+     * Generate the table navigation above or below the table
+     *
+     * This custom method is necessary because the one in `WP_List_Table` comes
+     * with a nonce and check that we do not need.
+     *
+     * @since bbPress (r5930)
+     *
+     * @param string $which
+     */
+    protected function display_tablenav( $which = '' ) {
+        ?>
+
+        <div class="tablenav <?php echo esc_attr( $which ); ?>">
+            <?php
+                $this->extra_tablenav( $which );
+                $this->pagination( $which );
+            ?>
+            <br class="clear" />
+        </div>
+
+        <?php
+    }
 }
 endif;