Skip to:
Content

bbPress.org

Ticket #2677: 2677.diff

File 2677.diff, 9.0 KB (added by netweb, 10 years ago)

  • src/includes/admin/metaboxes.php

     
    369369                <input name="menu_order" type="number" step="1" size="4" id="menu_order" value="<?php echo esc_attr( $menu_order ); ?>" />
    370370        </p>
    371371
    372         <input name="ping_status" type="hidden" id="ping_status" value="open" />
     372        <input name="ping_status" type="hidden" id="ping_status" value="closed" />
    373373
    374374        <?php
    375375        wp_nonce_field( 'bbp_forum_metabox_save', 'bbp_forum_metabox' );
    … …  
    445445                ) ); ?>
    446446        </p>
    447447
    448         <input name="ping_status" type="hidden" id="ping_status" value="open" />
     448        <input name="ping_status" type="hidden" id="ping_status" value="closed" />
    449449
    450450        <?php
    451451        wp_nonce_field( 'bbp_topic_metabox_save', 'bbp_topic_metabox' );
    … …  
    539539                <?php bbp_reply_to_dropdown( $post_id ); ?>
    540540        </p>
    541541
    542         <input name="ping_status" type="hidden" id="ping_status" value="open" />
     542        <input name="ping_status" type="hidden" id="ping_status" value="closed" />
    543543
    544544        <?php
    545545        wp_nonce_field( 'bbp_reply_metabox_save', 'bbp_reply_metabox' );

  • src/includes/core/filters.php

     
    5454// Force comments_status on bbPress post types
    5555add_filter( 'comments_open', 'bbp_force_comment_status' );
    5656
     57// Force ping_status on bbPress post types
     58add_filter( 'pings_open', 'bbp_force_ping_status' );
     59
    5760// Add post_parent__in to posts_where
    5861add_filter( 'posts_where', 'bbp_query_post_parent__in', 10, 2 );
    5962

  • src/includes/core/theme-compat.php

     
    357357                        'menu_order'            => 0,
    358358                        'pinged'                => '',
    359359                        'to_ping'               => '',
    360                         'ping_status'           => '',
     360                        'ping_status'           => 'closed',
    361361                        'comment_status'        => 'closed',
    362362                        'comment_count'         => 0,
    363363                        'filter'                => 'raw',
    … …  
    435435 * @uses bbp_set_theme_compat_template() To set the global theme compat template
    436436 */
    437437function bbp_template_include_theme_compat( $template = '' ) {
    438 
     438
    439439        /**
    440440         * Bail if a root template was already found. This prevents unintended
    441441         * recursive filtering of 'the_content'.
    … …  
    480480                        'post_title'     => bbp_get_displayed_user_field( 'display_name' ),
    481481                        'post_status'    => bbp_get_public_status_id(),
    482482                        'is_archive'     => false,
    483                         'comment_status' => 'closed'
     483                        'comment_status' => 'closed',
     484                        'ping_status'    => 'closed'
    484485                ) );
    485486
    486487        /** Forums ************************************************************/
    … …  
    535536                        'post_type'      => bbp_get_forum_post_type(),
    536537                        'post_status'    => bbp_get_public_status_id(),
    537538                        'is_archive'     => true,
    538                         'comment_status' => 'closed'
     539                        'comment_status' => 'closed',
     540                        'ping_status'    => 'closed'
    539541                ) );
    540542
    541543        // Single Forum
    … …  
    551553                        'post_type'      => bbp_get_forum_post_type(),
    552554                        'post_status'    => bbp_get_forum_visibility(),
    553555                        'is_single'      => true,
    554                         'comment_status' => 'closed'
     556                        'comment_status' => 'closed',
     557                        'ping_status'    => 'closed'
    555558                ) );
    556559
    557560        } elseif ( bbp_is_single_forum() ) {
    … …  
    566569                        'post_type'      => bbp_get_forum_post_type(),
    567570                        'post_status'    => bbp_get_forum_visibility(),
    568571                        'is_single'      => true,
    569                         'comment_status' => 'closed'
     572                        'comment_status' => 'closed',
     573                        'ping_status'    => 'closed'
    570574                ) );
    571575
    572576        /** Topics ************************************************************/
    … …  
    605609                        'post_type'      => bbp_get_topic_post_type(),
    606610                        'post_status'    => bbp_get_public_status_id(),
    607611                        'is_archive'     => true,
    608                         'comment_status' => 'closed'
     612                        'comment_status' => 'closed',
     613                        'ping_status'    => 'closed'
    609614                ) );
    610615
    611616        // Single Topic
    … …  
    638643                        'post_type'      => bbp_get_topic_post_type(),
    639644                        'post_status'    => bbp_get_topic_status(),
    640645                        'is_single'      => true,
    641                         'comment_status' => 'closed'
     646                        'comment_status' => 'closed',
     647                        'ping_status'    => 'closed'
    642648                ) );
    643649
    644650        /** Replies ***********************************************************/
    … …  
    655661                        'post_content'   => $bbp_shortcodes->display_reply_index(),
    656662                        'post_type'      => bbp_get_reply_post_type(),
    657663                        'post_status'    => bbp_get_public_status_id(),
    658                         'comment_status' => 'closed'
     664                        'comment_status' => 'closed',
     665                        'ping_status'    => 'closed'
    659666                ) );
    660667
    661668        // Single Reply
    … …  
    683690                        'post_content'   => $new_content,
    684691                        'post_type'      => bbp_get_reply_post_type(),
    685692                        'post_status'    => bbp_get_reply_status(),
    686                         'comment_status' => 'closed'
     693                        'comment_status' => 'closed',
     694                        'ping_status'    => 'closed'
    687695                ) );
    688696
    689697        /** Views *************************************************************/
    … …  
    699707                        'post_content'   => $bbp_shortcodes->display_view( array( 'id' => get_query_var( bbp_get_view_rewrite_id() ) ) ),
    700708                        'post_type'      => '',
    701709                        'post_status'    => bbp_get_public_status_id(),
    702                         'comment_status' => 'closed'
     710                        'comment_status' => 'closed',
     711                        'ping_status'    => 'closed'
    703712                ) );
    704713
    705714        /** Search ************************************************************/
    … …  
    715724                        'post_content'   => $bbp_shortcodes->display_search( array( 'search' => get_query_var( bbp_get_search_rewrite_id() ) ) ),
    716725                        'post_type'      => '',
    717726                        'post_status'    => bbp_get_public_status_id(),
    718                         'comment_status' => 'closed'
     727                        'comment_status' => 'closed',
     728                        'ping_status'    => 'closed'
    719729                ) );
    720730
    721731        /** Topic Tags ********************************************************/
    … …  
    745755                        'post_title'     => sprintf( __( 'Topic Tag: %s', 'bbpress' ), '<span>' . bbp_get_topic_tag_name() . '</span>' ),
    746756                        'post_status'    => bbp_get_public_status_id(),
    747757                        'comment_status' => 'closed',
     758                        'ping_status'    => 'closed',
    748759                        'is_tax'         => true
    749760                ) );
    750761        }
    … …  
    979990        // Allow override of the override
    980991        return apply_filters( 'bbp_force_comment_status', $retval, $open, $post_id, $post_type );
    981992}
     993
     994/**
     995 * Force ping_status to 'closed' for bbPress post types
     996 *
     997 * @since bbPress (rXXXX)
     998 *
     999 * @param bool $open True if open, false if closed
     1000 * @param int $post_id ID of the post to check
     1001 * @return bool True if open, false if closed
     1002 */
     1003function bbp_force_ping_status( $open, $post_id = 0 ) {
     1004
     1005        // Get the post type of the post ID
     1006        $post_type = get_post_type( $post_id );
     1007
     1008        // Default return value is what is passed in $open
     1009        $retval = $open;
     1010
     1011        // Only force for bbPress post types
     1012        switch ( $post_type ) {
     1013                case bbp_get_forum_post_type() :
     1014                case bbp_get_topic_post_type() :
     1015                case bbp_get_reply_post_type() :
     1016                        $retval = false;
     1017                        break;
     1018        }
     1019
     1020        // Allow override of the override
     1021        return apply_filters( 'bbp_force_ping_status', $retval, $open, $post_id, $post_type );
     1022}

  • src/includes/replies/functions.php

     
    3838                'post_content'   => '',
    3939                'post_title'     => '',
    4040                'menu_order'     => 0,
    41                 'comment_status' => 'closed'
     41                'comment_status' => 'closed',
     42                'ping_status'    => 'closed'
    4243        ), 'insert_reply' );
    4344
    4445        // Insert reply
    … …  
    361362                'post_parent'    => $topic_id,
    362363                'post_type'      => bbp_get_reply_post_type(),
    363364                'comment_status' => 'closed',
     365                'ping_status'    => 'closed',
    364366                'menu_order'     => bbp_get_topic_reply_count( $topic_id, true ) + 1
    365367        ) );
    366368
    … …  
    11631165 * @return array
    11641166 */
    11651167function bbp_get_reply_ancestors( $reply_id = 0 ) {
    1166 
     1168
    11671169        // Validation
    11681170        $reply_id  = bbp_get_reply_id( $reply_id );
    11691171        $ancestors = array();
    … …  
    14691471        }
    14701472
    14711473        // Remove reply_to from moved reply
    1472         delete_post_meta( $move_reply->ID, '_bbp_reply_to' );
     1474        delete_post_meta( $move_reply->ID, '_bbp_reply_to' );
    14731475
    14741476        // It is a new topic and we need to set some default metas to make
    14751477        // the topic display in bbp_has_topics() list
    … …  
    23342336
    23352337/**
    23362338 * Validate a `reply_to` field for hierarchical replies
    2337  *
     2339 *
    23382340 * Checks for 2 scenarios:
    23392341 * -- The reply to ID is actually a reply
    23402342 * -- The reply to ID does not match the current reply

  • src/includes/topics/functions.php

     
    3838                'post_content'   => '',
    3939                'post_title'     => '',
    4040                'comment_status' => 'closed',
     41                'ping_status'    => 'closed',
    4142                'menu_order'     => 0,
    4243        ), 'insert_topic' );
    4344
    … …  
    330331                'post_parent'    => $forum_id,
    331332                'post_type'      => bbp_get_topic_post_type(),
    332333                'tax_input'      => $terms,
    333                 'comment_status' => 'closed'
     334                'comment_status' => 'closed',
     335                'ping_status'    => 'closed'
    334336        ) );
    335337
    336338        // Insert topic