Skip to:
Content

bbPress.org


Ignore:
Timestamp:
07/04/2012 02:17:31 AM (12 years ago)
Author:
johnjamesjacoby
Message:

Admin:

  • Add nonce checks and additional sanity checks to converter, to prevent it being loaded or accessed outside of the converter screen.
  • Use 'load-' actions to conditionally load admin components on their respective screens.
  • Remove post_type checks that are no longer needed as a result of conditionally loading each component.
  • Use get_current_screen() to fill in any extra post_type checks.
  • Sweep through converter, make output more clean, add output method, and store last query in the DB in case of failure.
File:
1 edited

Legend:

Unmodified
Added
Removed

  • branches/plugin/bbp-admin/bbp-replies.php

    r4034 r4053  
    2626     * @var The post type of this admin component
    2727     */
    28     var $post_type = '';
     28    private $post_type = '';
    2929
    3030    /** Functions *************************************************************/
    … …  
    7676
    7777        // Check if there are any bbp_toggle_reply_* requests on admin_init, also have a message displayed
    78         add_action( 'bbp_admin_init', array( $this, 'toggle_reply'        ) );
     78        add_action( 'load-edit.php', array( $this, 'toggle_reply'        ) );
    7979        add_action( 'admin_notices',  array( $this, 'toggle_reply_notice' ) );
    8080
    … …  
    114114    public function edit_help() {
    115115
    116         $current_screen = get_current_screen();
    117         $post_type      = !empty( $_REQUEST['post_type'] ) ? $_REQUEST['post_type'] : '';
    118 
    119         // Bail if current screen could not be found
    120         if ( empty( $current_screen ) )
    121             return;
    122 
    123         // Bail if not the reply post type
    124         if ( $post_type != $this->post_type )
    125             return;
    126 
    127116        // Overview
    128         $current_screen->add_help_tab( array(
     117        get_current_screen()->add_help_tab( array(
    129118            'id'        => 'overview',
    130119            'title'     => __( 'Overview', 'bbpress' ),
    … …  
    134123
    135124        // Screen Content
    136         $current_screen->add_help_tab( array(
     125        get_current_screen()->add_help_tab( array(
    137126            'id'        => 'screen-content',
    138127            'title'     => __( 'Screen Content', 'bbpress' ),
    … …  
    148137
    149138        // Available Actions
    150         $current_screen->add_help_tab( array(
     139        get_current_screen()->add_help_tab( array(
    151140            'id'        => 'action-links',
    152141            'title'     => __( 'Available Actions', 'bbpress' ),
    … …  
    163152
    164153        // Bulk Actions
    165         $current_screen->add_help_tab( array(
     154        get_current_screen()->add_help_tab( array(
    166155            'id'        => 'bulk-actions',
    167156            'title'     => __( 'Bulk Actions', 'bbpress' ),
    … …  
    172161
    173162        // Help Sidebar
    174         $current_screen->set_help_sidebar(
     163        get_current_screen()->set_help_sidebar(
    175164            '<p><strong>' . __( 'For more information:', 'bbpress' ) . '</strong></p>' .
    176165            '<p>' . __( '<a href="http://bbpress.org/documentation/" target="_blank">bbPress Documentation</a>', 'bbpress' ) . '</p>' .
    … …  
    187176    public function new_help() {
    188177
    189         $current_screen = get_current_screen();
    190         $post_type      = !empty( $_REQUEST['post_type'] ) ? $_REQUEST['post_type'] : '';
    191 
    192         // Bail if current screen could not be found
    193         if ( empty( $current_screen ) )
    194             return;
    195 
    196         // Bail if not the reply post type
    197         if ( $post_type != $this->post_type )
    198             return;
    199 
    200178        $customize_display = '<p>' . __( 'The title field and the big reply editing Area are fixed in place, but you can reposition all the other boxes using drag and drop, and can minimize or expand them by clicking the title bar of each box. Use the Screen Options tab to unhide more boxes (Excerpt, Send Trackbacks, Custom Fields, Discussion, Slug, Author) or to choose a 1- or 2-column layout for this screen.', 'bbpress' ) . '</p>';
    201179
    202         $current_screen->add_help_tab( array(
     180        get_current_screen()->add_help_tab( array(
    203181            'id'      => 'customize-display',
    204182            'title'   => __( 'Customizing This Display', 'bbpress' ),
    … …  
    206184        ) );
    207185
    208         $current_screen->add_help_tab( array(
     186        get_current_screen()->add_help_tab( array(
    209187            'id'      => 'title-reply-editor',
    210188            'title'   => __( 'Title and Reply Editor', 'bbpress' ),
    … …  
    224202        }
    225203
    226         $current_screen->add_help_tab( array(
     204        get_current_screen()->add_help_tab( array(
    227205            'id'      => 'reply-attributes',
    228206            'title'   => __( 'Reply Attributes', 'bbpress' ),
    … …  
    235213        ) );
    236214
    237         $current_screen->add_help_tab( array(
     215        get_current_screen()->add_help_tab( array(
    238216            'id'      => 'publish-box',
    239217            'title'   => __( 'Publish Box', 'bbpress' ),
    … …  
    241219        ) );
    242220
    243         $current_screen->add_help_tab( array(
     221        get_current_screen()->add_help_tab( array(
    244222            'id'      => 'discussion-settings',
    245223            'title'   => __( 'Discussion Settings', 'bbpress' ),
    … …  
    249227        ) );
    250228
    251         $current_screen->set_help_sidebar(
     229        get_current_screen()->set_help_sidebar(
    252230            '<p><strong>' . __( 'For more information:', 'bbpress' ) . '</strong></p>' .
    253231            '<p>' . __( '<a href="http://bbpress.org/documentation/" target="_blank">bbPress Documentation</a>', 'bbpress' ) . '</p>' .
    … …  
    308286            return $reply_id;
    309287
    310         // Bail if post_type is not a reply
    311         if ( get_post_type( $reply_id ) != $this->post_type )
    312             return $reply_id;
    313 
    314288        // Current user cannot edit this reply
    315289        if ( !current_user_can( 'edit_reply', $reply_id ) )
    … …  
    347321
    348322        // Bail if post_type is not a reply
    349         if ( ( empty( $_GET['action'] ) || ( 'edit' != $_GET['action'] ) ) || ( get_post_type() != $this->post_type ) )
     323        if ( empty( $_GET['action'] ) || ( 'edit' != $_GET['action'] ) )
    350324            return;
    351325
    … …  
    393367            return $post_id;
    394368
    395         // Bail if post_type is not a topic or reply
    396         if ( get_post_type( $post_id ) != $this->post_type )
    397             return;
    398 
    399369        // Bail if user cannot edit replies or reply is not anonymous
    400370        if ( !current_user_can( 'edit_reply', $post_id ) )
    … …  
    424394     */
    425395    function admin_head() {
    426 
    427         if ( get_post_type() == $this->post_type ) : ?>
    428 
    429             <style type="text/css" media="screen">
    430             /*<![CDATA[*/
    431 
    432                 .column-bbp_forum_topic_count,
    433                 .column-bbp_forum_reply_count,
    434                 .column-bbp_topic_reply_count,
    435                 .column-bbp_topic_voice_count {
    436                     width: 8% !important;
    437                 }
    438 
    439                 .column-author,
    440                 .column-bbp_reply_author,
    441                 .column-bbp_topic_author {
    442                     width: 10% !important;
    443                 }
    444 
    445                 .column-bbp_topic_forum,
    446                 .column-bbp_reply_forum,
    447                 .column-bbp_reply_topic {
    448                     width: 10% !important;
    449                 }
    450 
    451                 .column-bbp_forum_freshness,
    452                 .column-bbp_topic_freshness {
    453                     width: 10% !important;
    454                 }
    455 
    456                 .column-bbp_forum_created,
    457                 .column-bbp_topic_created,
    458                 .column-bbp_reply_created {
    459                     width: 15% !important;
    460                 }
    461 
    462                 .status-closed {
    463                     background-color: #eaeaea;
    464                 }
    465 
    466                 .status-spam {
    467                     background-color: #faeaea;
    468                 }
    469 
    470             /*]]>*/
    471             </style>
    472 
    473         <?php endif;
    474 
     396        ?>
     397
     398        <style type="text/css" media="screen">
     399        /*<![CDATA[*/
     400
     401            .column-bbp_forum_topic_count,
     402            .column-bbp_forum_reply_count,
     403            .column-bbp_topic_reply_count,
     404            .column-bbp_topic_voice_count {
     405                width: 8% !important;
     406            }
     407
     408            .column-author,
     409            .column-bbp_reply_author,
     410            .column-bbp_topic_author {
     411                width: 10% !important;
     412            }
     413
     414            .column-bbp_topic_forum,
     415            .column-bbp_reply_forum,
     416            .column-bbp_reply_topic {
     417                width: 10% !important;
     418            }
     419
     420            .column-bbp_forum_freshness,
     421            .column-bbp_topic_freshness {
     422                width: 10% !important;
     423            }
     424
     425            .column-bbp_forum_created,
     426            .column-bbp_topic_created,
     427            .column-bbp_reply_created {
     428                width: 15% !important;
     429            }
     430
     431            .status-closed {
     432                background-color: #eaeaea;
     433            }
     434
     435            .status-spam {
     436                background-color: #faeaea;
     437            }
     438
     439        /*]]>*/
     440        </style>
     441
     442        <?php
    475443    }
    476444
    … …  
    815783    function filter_dropdown() {
    816784
    817         // Bail if not viewing the topics list
    818         if (
    819                 // post_type exists in _GET
    820                 empty( $_GET['post_type'] ) ||
    821 
    822                 // post_type is reply or topic type
    823                 ( $_GET['post_type'] != $this->post_type )
    824             )
    825             return;
    826 
    827785        // Add Empty Spam button
    828786        if ( !empty( $_GET['post_status'] ) && ( bbp_get_spam_status_id() == $_GET['post_status'] ) && current_user_can( 'moderate' ) ) {
    … …  
    856814        global $pagenow;
    857815
    858         // Avoid poisoning other requests
    859         if (
    860                 // Only look in admin
    861                 !is_admin()                 ||
    862 
    863                 // Make sure the current page is for post rows
    864                 ( 'edit.php' != $pagenow  ) ||
    865 
    866                 // Make sure we're looking for a post_type
    867                 empty( $_GET['post_type'] ) ||
    868 
    869                 // Make sure we're looking at bbPress topics
    870                 ( $_GET['post_type'] != $this->post_type )
    871             )
    872 
    873             // We're in no shape to filter anything, so return
    874             return $query_vars;
    875 
    876816        // Add post_parent query_var if one is present
    877817        if ( !empty( $_GET['bbp_forum_id'] ) ) {
    … …  
    890830     *
    891831     * @global int $post_ID
    892      * @uses get_post_type()
    893832     * @uses bbp_get_topic_permalink()
    894833     * @uses wp_post_revision_title()
    … …  
    902841    function updated_messages( $messages ) {
    903842        global $post_ID;
    904 
    905         if ( get_post_type( $post_ID ) != $this->post_type )
    906             return $messages;
    907843
    908844        // URL for the current topic
    … …  
    962898 * Setup bbPress Replies Admin
    963899 *
     900 * This is currently here to make hooking and unhooking of the admin UI easy.
     901 * It could use dependency injection in the future, but for now this is easier.
     902 *
    964903 * @since bbPress (r2596)
    965904 *
    … …  
    967906 */
    968907function bbp_admin_replies() {
     908    global $typenow;
     909
     910    if ( bbp_get_reply_post_type() == $typenow )
     911        return;
     912
    969913    bbpress()->admin->replies = new BBP_Replies_Admin();
    970914}
Note: See TracChangeset for help on using the changeset viewer.