Skip to:
Content

Ignore:
Timestamp:
01/29/2012 08:23:20 PM (7 years ago)
Author:
johnjamesjacoby
Message:

Global $post usage clean-up:

  • Use bbp_get_global_post_field() where appropriate, core WP functions otherwise
  • Replace local $post variables with $post to avoid stomping the global
  • Audit usage of bbp_get_global_post_field() and make sure we are providing the correct context
  • Fixes #1733.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/plugin/bbp-admin/bbp-replies.php

    r3688 r3714  
    501501            $post_data = array( 'ID' => $reply_id ); // Prelim array
    502502
    503             if ( !$reply = bbp_get_reply( $reply_id ) ) // Which reply?
     503            // Get reply and die if empty
     504            $reply = bbp_get_reply( $reply_id );
     505            if ( empty( $reply ) ) // Which reply?
    504506                wp_die( __( 'The reply was not found!', 'bbpress' ) );
    505507
     
    559561
    560562            // Empty? No reply?
    561             if ( empty( $notice ) || empty( $reply_id ) || !$reply = bbp_get_reply( $reply_id ) )
     563            if ( empty( $notice ) || empty( $reply_id ) )
     564                return;
     565
     566            // Get reply and bail if empty
     567            $reply = bbp_get_reply( $reply_id );
     568            if ( empty( $reply ) )
    562569                return;
    563570
     
    653660
    654661                    // Topic Title
    655                     if ( !$topic_title = bbp_get_topic_title( $topic_id ) )
     662                    $topic_title = bbp_get_topic_title( $topic_id );
     663                    if ( empty( $topic_title ) ) {
    656664                        $topic_title = __( 'No Topic', 'bbpress' );
     665                    }
    657666
    658667                    // Output the title
     
    677686
    678687                    // Forum Title
    679                     if ( !$forum_title = bbp_get_forum_title( $reply_forum_id ) )
     688                    $forum_title = bbp_get_forum_title( $reply_forum_id );
     689                    if ( empty( $forum_title ) ) {
    680690                        $forum_title = __( 'No Forum', 'bbpress' );
     691                    }
    681692
    682693                    // Alert capable users of reply forum mismatch
     
    874885     * @since bbPress (r3080)
    875886     *
    876      * @global WP_Query $post
    877887     * @global int $post_ID
    878888     * @uses get_post_type()
     
    887897     */
    888898    function updated_messages( $messages ) {
    889         global $post, $post_ID;
     899        global $post_ID;
    890900
    891901        if ( get_post_type( $post_ID ) != $this->post_type )
     
    894904        // URL for the current topic
    895905        $topic_url = bbp_get_topic_permalink( bbp_get_reply_topic_id( $post_ID ) );
     906       
     907        // Current reply's post_date
     908        $post_date = bbp_get_global_post_field( 'post_date', 'raw' );
    896909
    897910        // Messages array
     
    930943                    // translators: Publish box date format, see http://php.net/date
    931944                    date_i18n( __( 'M j, Y @ G:i' ),
    932                     strtotime( $post->post_date ) ),
     945                    strtotime( $post_date ) ),
    933946                    $topic_url ),
    934947
Note: See TracChangeset for help on using the changeset viewer.