Skip to:
Content

bbPress.org


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-topics.php

    r3688 r3714  
    525525            $success   = false;                      // Flag
    526526            $post_data = array( 'ID' => $topic_id ); // Prelim array
    527 
    528             if ( !$topic = bbp_get_topic( $topic_id ) ) // Which topic?
     527            $topic     = bbp_get_topic( $topic_id );
     528
     529            // Bail if topic is missing
     530            if ( empty( $topic ) )
    529531                wp_die( __( 'The topic was not found!', 'bbpress' ) );
    530532
     
    602604            $is_failure = !empty( $_GET['failed'] ) ? true : false; // Was that a failure?
    603605
    604             // Empty? No topic?
    605             if ( empty( $notice ) || empty( $topic_id ) || !$topic = bbp_get_topic( $topic_id ) )
     606            // Bais if no topic_id or notice
     607            if ( empty( $notice ) || empty( $topic_id ) )
     608                return;
     609
     610            // Bail if topic is missing
     611            $topic = bbp_get_topic( $topic_id );
     612            if ( empty( $topic ) )
    606613                return;
    607614
     
    716723                   
    717724                    // Forum Title
    718                     if ( !$forum_title = bbp_get_forum_title( $forum_id ) )
     725                    $forum_title = bbp_get_forum_title( $forum_id );
     726                    if ( empty( $forum_title ) ) {
    719727                        $forum_title = __( 'No Forum', 'bbpress' );
     728                    }
    720729
    721730                    // Output the title
     
    755764            case 'bbp_topic_freshness' :
    756765                $last_active = bbp_get_topic_last_active_time( $topic_id, false );
    757                 if ( !empty( $last_active ) )
     766                if ( !empty( $last_active ) ) {
    758767                    printf( __( '%s ago', 'bbpress' ), $last_active );
    759                 else
     768                } else {
    760769                    _e( 'No Replies', 'bbpress' ); // This should never happen
     770                }
    761771
    762772                break;
     
    946956     * @since bbPress (r3080)
    947957     *
    948      * @global WP_Query $post
    949958     * @global int $post_ID
    950959     * @uses get_post_type()
     
    959968     */
    960969    function updated_messages( $messages ) {
    961         global $post, $post_ID;
     970        global $post_ID;
    962971
    963972        if ( get_post_type( $post_ID ) != $this->post_type )
     
    966975        // URL for the current topic
    967976        $topic_url = bbp_get_topic_permalink( $post_ID );
     977
     978        // Current topic's post_date
     979        $post_date = bbp_get_global_post_field( 'post_date', 'raw' );
    968980
    969981        // Messages array
     
    10021014                    // translators: Publish box date format, see http://php.net/date
    10031015                    date_i18n( __( 'M j, Y @ G:i' ),
    1004                     strtotime( $post->post_date ) ),
     1016                    strtotime( $post_date ) ),
    10051017                    $topic_url ),
    10061018
Note: See TracChangeset for help on using the changeset viewer.