Skip to:
Content

bbPress.org

Ticket #2900: 2900.diff

File 2900.diff, 2.3 KB (added by milindmore22, 5 years ago)

added $argssep? with backward compatibility

  • src/includes/admin/tools/common.php

    diff --git src/includes/admin/tools/common.php src/includes/admin/tools/common.php
    index 55c24e9..ec0ab96 100644
    function bbp_get_admin_repair_tool_overhead_filters( $args = array() ) { 
    439439                'link_after'        => '</li>',
    440440                'count_before'      => ' <span class="count">(',
    441441                'count_after'       => ')</span>',
    442                 'separator'         => ' | ',
     442                'separator'         => false,
     443                'sep'               => ' | ',
    443444        ), 'get_admin_repair_tool_overhead_filters' );
    444 
     445       
     446        /**
     447         * Necessary for backwards compatibility
     448         * @see https://bbpress.trac.wordpress.org/ticket/2900
     449         */
     450        if( !empty( $r['separator'] ) ){
     451                $r['sep'] = $r['separator'];
     452        }
     453       
    445454        // Get page
    446455        $page = sanitize_key( $_GET['page'] );
    447456
    function bbp_get_admin_repair_tool_overhead_filters( $args = array() ) { 
    500509        }
    501510
    502511        // Surround output with before & after strings
    503         $output = $r['before'] . implode( $r['separator'], $links ) . $r['after'];
     512        $output = $r['before'] . implode( $r['sep'], $links ) . $r['after'];
    504513
    505514        // Filter & return
    506515        return apply_filters( 'bbp_get_admin_repair_tool_components', $output, $r, $args );
  • src/includes/forums/template.php

    diff --git src/includes/forums/template.php src/includes/forums/template.php
    index 8cc3076..668fe30 100644
    function bbp_list_forums( $args = array() ) { 
    774774                'count_before'      => ' (',
    775775                'count_after'       => ')',
    776776                'count_sep'         => ', ',
    777                 'separator'         => ', ',
     777                'separator'         => false,
     778                'sep'               => ', ',
    778779                'forum_id'          => '',
    779780                'show_topic_count'  => true,
    780781                'show_reply_count'  => true,
    781782        ), 'list_forums' );
    782783
     784        /**
     785         * Necessary for backwards compatibility
     786         * @see https://bbpress.trac.wordpress.org/ticket/2900
     787         */
     788        if( !empty( $r['separator'] ) ){
     789                $r['sep'] = $r['separator'];
     790        }
     791       
    783792        // Loop through forums and create a list
    784793        $sub_forums = bbp_forum_get_subforums( $r['forum_id'] );
    785794        if ( ! empty( $sub_forums ) ) {
    function bbp_list_forums( $args = array() ) { 
    796805                        $permalink = bbp_get_forum_permalink( $sub_forum->ID );
    797806                        $title     = bbp_get_forum_title( $sub_forum->ID );
    798807                        $show_sep  = ( $total_subs > $i )
    799                                 ? $r['separator']
     808                                ? $r['sep']
    800809                                : '';
    801810
    802811                        // Show topic count