Skip to:
Content

bbPress.org


Ignore:
Timestamp:
08/25/2015 03:07:14 PM (6 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:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/includes/admin/list-tables/topic-replies.php

    r5909 r5930  
    230230        ) );
    231231    }
     232
     233    /**
     234     * Message to be displayed when there are no items
     235     *
     236     * @since bbPress (r5930)
     237     */
     238    public function no_items() {
     239        esc_html_e( 'No replies to this topic.', 'bbpress' );
     240    }
     241
     242    /**
     243     * Display the list table
     244     *
     245     * This custom method is necessary because the one in `WP_List_Table` comes
     246     * with a nonce and check that we do not need.
     247     *
     248     * @since bbPress (r5930)
     249     */
     250    public function display() {
     251
     252        // Top
     253        $this->display_tablenav( 'top' ); ?>
     254
     255        <table class="wp-list-table <?php echo implode( ' ', $this->get_table_classes() ); ?>">
     256            <thead>
     257                <tr>
     258                    <?php $this->print_column_headers(); ?>
     259                </tr>
     260            </thead>
     261
     262            <tbody id="the-list" data-wp-lists='list:<?php echo $this->_args['singular']; ?>'>
     263                <?php $this->display_rows_or_placeholder(); ?>
     264            </tbody>
     265
     266            <tfoot>
     267                <tr>
     268                    <?php $this->print_column_headers( false ); ?>
     269                </tr>
     270            </tfoot>
     271        </table>
     272
     273        <?php
     274
     275        // Bottom
     276        $this->display_tablenav( 'bottom' );
     277    }
     278
     279    /**
     280     * Generate the table navigation above or below the table
     281     *
     282     * This custom method is necessary because the one in `WP_List_Table` comes
     283     * with a nonce and check that we do not need.
     284     *
     285     * @since bbPress (r5930)
     286     *
     287     * @param string $which
     288     */
     289    protected function display_tablenav( $which = '' ) {
     290        ?>
     291
     292        <div class="tablenav <?php echo esc_attr( $which ); ?>">
     293            <?php
     294                $this->extra_tablenav( $which );
     295                $this->pagination( $which );
     296            ?>
     297            <br class="clear" />
     298        </div>
     299
     300        <?php
     301    }
    232302}
    233303endif;
Note: See TracChangeset for help on using the changeset viewer.