Skip to:
Content

bbPress.org

Changeset 6506


Ignore:
Timestamp:
06/09/2017 04:45:11 AM (9 years ago)
Author:
johnjamesjacoby
Message:

Widgets: Audit all widget classes:

  • Use escaping GetText functions
  • Fix typos
  • Use more efficient queries for forum/topic/reply meta-boxes

Needs testing. See #3070.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/includes/common/widgets.php

    r6384 r6506  
    3737                $widget_ops = apply_filters( 'bbp_login_widget_options', array(
    3838                        'classname'   => 'bbp_widget_login',
    39                         'description' => __( 'A simple login form with optional links to sign-up and lost password pages.', 'bbpress' )
     39                        'description' => esc_html__( 'A simple login form with optional links to sign-up and lost password pages.', 'bbpress' )
    4040                ) );
    4141
    42                 parent::__construct( false, __( '(bbPress) Login Widget', 'bbpress' ), $widget_ops );
     42                parent::__construct( false, esc_html__( '(bbPress) Login Widget', 'bbpress' ), $widget_ops );
    4343        }
    4444
    … …  
    8787                        <form method="post" action="<?php bbp_wp_login_action( array( 'context' => 'login_post' ) ); ?>" class="bbp-login-form">
    8888                                <fieldset>
    89                                         <legend><?php _e( 'Log In', 'bbpress' ); ?></legend>
     89                                        <legend><?php esc_html_e( 'Log In', 'bbpress' ); ?></legend>
    9090
    9191                                        <div class="bbp-username">
    92                                                 <label for="user_login"><?php _e( 'Username', 'bbpress' ); ?>: </label>
     92                                                <label for="user_login"><?php esc_html_e( 'Username', 'bbpress' ); ?>: </label>
    9393                                                <input type="text" name="log" value="<?php bbp_sanitize_val( 'user_login', 'text' ); ?>" size="20" id="user_login" />
    9494                                        </div>
    9595
    9696                                        <div class="bbp-password">
    97                                                 <label for="user_pass"><?php _e( 'Password', 'bbpress' ); ?>: </label>
     97                                                <label for="user_pass"><?php esc_html_e( 'Password', 'bbpress' ); ?>: </label>
    9898                                                <input type="password" name="pwd" value="<?php bbp_sanitize_val( 'user_pass', 'password' ); ?>" size="20" id="user_pass" autocomplete="off" />
    9999                                        </div>
    … …  
    101101                                        <div class="bbp-remember-me">
    102102                                                <input type="checkbox" name="rememberme" value="forever" <?php checked( bbp_get_sanitize_val( 'rememberme', 'checkbox' ), true, true ); ?> id="rememberme" />
    103                                                 <label for="rememberme"><?php _e( 'Remember Me', 'bbpress' ); ?></label>
     103                                                <label for="rememberme"><?php esc_html_e( 'Remember Me', 'bbpress' ); ?></label>
    104104                                        </div>
    105105
    … …  
    108108                                                <?php do_action( 'login_form' ); ?>
    109109
    110                                                 <button type="submit" name="user-submit" id="user-submit" class="button submit user-submit"><?php _e( 'Log In', 'bbpress' ); ?></button>
     110                                                <button type="submit" name="user-submit" id="user-submit" class="button submit user-submit"><?php esc_html_e( 'Log In', 'bbpress' ); ?></button>
    111111
    112112                                                <?php bbp_user_login_fields(); ?>
    … …  
    120120                                                        <?php if ( ! empty( $settings['register'] ) ) : ?>
    121121
    122                                                                 <a href="<?php echo esc_url( $settings['register'] ); ?>" title="<?php esc_attr_e( 'Register', 'bbpress' ); ?>" class="bbp-register-link"><?php _e( 'Register', 'bbpress' ); ?></a>
     122                                                                <a href="<?php echo esc_url( $settings['register'] ); ?>" title="<?php esc_attr_e( 'Register', 'bbpress' ); ?>" class="bbp-register-link"><?php esc_html_e( 'Register', 'bbpress' ); ?></a>
    123123
    124124                                                        <?php endif; ?>
    … …  
    126126                                                        <?php if ( ! empty( $settings['lostpass'] ) ) : ?>
    127127
    128                                                                 <a href="<?php echo esc_url( $settings['lostpass'] ); ?>" title="<?php esc_attr_e( 'Lost Password', 'bbpress' ); ?>" class="bbp-lostpass-link"><?php _e( 'Lost Password', 'bbpress' ); ?></a>
     128                                                                <a href="<?php echo esc_url( $settings['lostpass'] ); ?>" title="<?php esc_attr_e( 'Lost Password', 'bbpress' ); ?>" class="bbp-lostpass-link"><?php esc_html_e( 'Lost Password', 'bbpress' ); ?></a>
    129129
    130130                                                        <?php endif; ?>
    … …  
    183183
    184184                <p>
    185                         <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'bbpress' ); ?>
     185                        <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'bbpress' ); ?>
    186186                        <input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo esc_attr( $settings['title'] ); ?>" /></label>
    187187                </p>
    188188
    189189                <p>
    190                         <label for="<?php echo $this->get_field_id( 'register' ); ?>"><?php _e( 'Register URI:', 'bbpress' ); ?>
     190                        <label for="<?php echo $this->get_field_id( 'register' ); ?>"><?php esc_html_e( 'Register URI:', 'bbpress' ); ?>
    191191                        <input class="widefat" id="<?php echo $this->get_field_id( 'register' ); ?>" name="<?php echo $this->get_field_name( 'register' ); ?>" type="text" value="<?php echo esc_url( $settings['register'] ); ?>" /></label>
    192192                </p>
    193193
    194194                <p>
    195                         <label for="<?php echo $this->get_field_id( 'lostpass' ); ?>"><?php _e( 'Lost Password URI:', 'bbpress' ); ?>
     195                        <label for="<?php echo $this->get_field_id( 'lostpass' ); ?>"><?php esc_html_e( 'Lost Password URI:', 'bbpress' ); ?>
    196196                        <input class="widefat" id="<?php echo $this->get_field_id( 'lostpass' ); ?>" name="<?php echo $this->get_field_name( 'lostpass' ); ?>" type="text" value="<?php echo esc_url( $settings['lostpass'] ); ?>" /></label>
    197197                </p>
    … …  
    241241                $widget_ops = apply_filters( 'bbp_views_widget_options', array(
    242242                        'classname'   => 'widget_display_views',
    243                         'description' => __( 'A list of registered optional topic views.', 'bbpress' )
     243                        'description' => esc_html__( 'A list of registered optional topic views.', 'bbpress' )
    244244                ) );
    245245
    246                 parent::__construct( false, __( '(bbPress) Topic Views List', 'bbpress' ), $widget_ops );
     246                parent::__construct( false, esc_html__( '(bbPress) Topic Views List', 'bbpress' ), $widget_ops );
    247247        }
    248248
    … …  
    286286                $settings['title'] = apply_filters( 'bbp_view_widget_title', $settings['title'], $instance, $this->id_base );
    287287
     288                // Start an output buffer
     289                ob_start();
     290
    288291                echo $args['before_widget'];
    289292
    … …  
    292295                } ?>
    293296
    294                 <ul>
     297                <ul class="bbp-views-widget">
    295298
    296299                        <?php foreach ( array_keys( bbp_get_views() ) as $view ) : ?>
    … …  
    303306
    304307                <?php echo $args['after_widget'];
     308
     309                // Output the current buffer
     310                echo ob_get_clean();
    305311        }
    306312
    … …  
    335341
    336342                <p>
    337                         <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'bbpress' ); ?>
     343                        <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'bbpress' ); ?>
    338344                                <input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo esc_attr( $settings['title'] ); ?>" />
    339345                        </label>
    … …  
    382388                $widget_ops = apply_filters( 'bbp_search_widget_options', array(
    383389                        'classname'   => 'widget_display_search',
    384                         'description' => __( 'The bbPress forum search form.', 'bbpress' )
     390                        'description' => esc_html__( 'The bbPress forum search form.', 'bbpress' )
    385391                ) );
    386392
    387                 parent::__construct( false, __( '(bbPress) Forum Search Form', 'bbpress' ), $widget_ops );
     393                parent::__construct( false, esc_html__( '(bbPress) Forum Search Form', 'bbpress' ), $widget_ops );
    388394        }
    389395
    … …  
    464470
    465471                <p>
    466                         <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'bbpress' ); ?>
     472                        <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'bbpress' ); ?>
    467473                                <input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo esc_attr( $settings['title'] ); ?>" />
    468474                        </label>
    … …  
    482488        public function parse_settings( $instance = array() ) {
    483489                return bbp_parse_args( $instance, array(
    484                         'title' => __( 'Search Forums', 'bbpress' )
     490                        'title' => esc_html__( 'Search Forums', 'bbpress' )
    485491                ), 'search_widget_settings' );
    486492        }
    … …  
    511517                $widget_ops = apply_filters( 'bbp_forums_widget_options', array(
    512518                        'classname'   => 'widget_display_forums',
    513                         'description' => __( 'A list of forums with an option to set the parent.', 'bbpress' )
     519                        'description' => esc_html__( 'A list of forums with an option to set the parent.', 'bbpress' )
    514520                ) );
    515521
    516                 parent::__construct( false, __( '(bbPress) Forums List', 'bbpress' ), $widget_ops );
     522                parent::__construct( false, esc_html__( '(bbPress) Forums List', 'bbpress' ), $widget_ops );
    517523        }
    518524
    … …  
    560566                // bbp_pre_get_posts_normalize_forum_visibility action and function.
    561567                $widget_query = new WP_Query( array(
    562                         'post_type'           => bbp_get_forum_post_type(),
    563                         'post_parent'         => $settings['parent_forum'],
    564                         'post_status'         => bbp_get_public_status_id(),
    565                         'posts_per_page'      => get_option( '_bbp_forums_per_page', 50 ),
    566                         'ignore_sticky_posts' => true,
    567                         'no_found_rows'       => true,
    568                         'orderby'             => 'menu_order title',
    569                         'order'               => 'ASC'
     568
     569                        // What and how
     570                        'post_type'      => bbp_get_forum_post_type(),
     571                        'post_status'    => bbp_get_public_status_id(),
     572                        'post_parent'    => $settings['parent_forum'],
     573                        'posts_per_page' => (int) get_option( '_bbp_forums_per_page', 50 ),
     574
     575                        // Order
     576                        'orderby' => 'menu_order title',
     577                        'order'   => 'DESC',
     578
     579                        // Performance
     580                        'ignore_sticky_posts'    => true,
     581                        'no_found_rows'          => true,
     582                        'nopaging'               => true,
     583                        'update_post_term_cache' => false,
     584                        'update_post_meta_cache' => false
    570585                ) );
    571586
    … …  
    581596                } ?>
    582597
    583                 <ul>
     598                <ul class="bbp-forums-widget">
    584599
    585600                        <?php while ( $widget_query->have_posts() ) : $widget_query->the_post(); ?>
    … …  
    637652
    638653                <p>
    639                         <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'bbpress' ); ?>
     654                        <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'bbpress' ); ?>
    640655                                <input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo esc_attr( $settings['title'] ); ?>" />
    641656                        </label>
    … …  
    643658
    644659                <p>
    645                         <label for="<?php echo $this->get_field_id( 'parent_forum' ); ?>"><?php _e( 'Parent Forum ID:', 'bbpress' ); ?>
     660                        <label for="<?php echo $this->get_field_id( 'parent_forum' ); ?>"><?php esc_html_e( 'Parent Forum ID:', 'bbpress' ); ?>
    646661                                <input class="widefat" id="<?php echo $this->get_field_id( 'parent_forum' ); ?>" name="<?php echo $this->get_field_name( 'parent_forum' ); ?>" type="text" value="<?php echo esc_attr( $settings['parent_forum'] ); ?>" />
    647662                        </label>
    … …  
    649664                        <br />
    650665
    651                         <small><?php _e( '"0" to show only root - "any" to show all', 'bbpress' ); ?></small>
     666                        <small><?php esc_html_e( '"0" to show only root - "any" to show all', 'bbpress' ); ?></small>
    652667                </p>
    653668
    … …  
    665680        public function parse_settings( $instance = array() ) {
    666681                return bbp_parse_args( $instance, array(
    667                         'title'        => __( 'Forums', 'bbpress' ),
     682                        'title'        => esc_html__( 'Forums', 'bbpress' ),
    668683                        'parent_forum' => 0
    669684                ), 'forum_widget_settings' );
    … …  
    695710                $widget_ops = apply_filters( 'bbp_topics_widget_options', array(
    696711                        'classname'   => 'widget_display_topics',
    697                         'description' => __( 'A list of recent topics, sorted by: newness, popularity, or recent replies.', 'bbpress' )
     712                        'description' => esc_html__( 'A list of recent topics, sorted by: newness, popularity, or recent replies.', 'bbpress' )
    698713                ) );
    699714
    700                 parent::__construct( false, __( '(bbPress) Recent Topics', 'bbpress' ), $widget_ops );
     715                parent::__construct( false, esc_html__( '(bbPress) Recent Topics', 'bbpress' ), $widget_ops );
    701716        }
    702717
    … …  
    743758                        case 'freshness' :
    744759                                $topics_query = array(
    745                                         'post_type'           => bbp_get_topic_post_type(),
    746                                         'post_parent'         => $settings['parent_forum'],
    747                                         'posts_per_page'      => (int) $settings['max_shown'],
    748                                         'post_status'         => bbp_get_public_topic_statuses(),
    749                                         'ignore_sticky_posts' => true,
    750                                         'no_found_rows'       => true,
    751                                         'meta_key'            => '_bbp_last_active_time',
    752                                         'meta_type'           => 'DATETIME',
    753                                         'orderby'             => 'meta_value',
    754                                         'order'               => 'DESC',
     760
     761                                        // What and how
     762                                        'post_type'      => bbp_get_topic_post_type(),
     763                                        'post_status'    => bbp_get_public_topic_statuses(),
     764                                        'post_parent'    => $settings['parent_forum'],
     765                                        'posts_per_page' => (int) $settings['max_shown'],
     766                                        'meta_query'     => array( array(
     767                                                'key'  => '_bbp_last_active_time',
     768                                                'type' => 'DATETIME'
     769                                        ) ),
     770
     771                                        // Ordering
     772                                        'orderby' => 'meta_value',
     773                                        'order'   => 'DESC',
     774
     775                                        // Performance
     776                                        'ignore_sticky_posts'    => true,
     777                                        'no_found_rows'          => true,
     778                                        'nopaging'               => true,
     779                                        'update_post_term_cache' => false,
     780                                        'update_post_meta_cache' => false
    755781                                );
    756782                                break;
    … …  
    759785                        case 'popular' :
    760786                                $topics_query = array(
    761                                         'post_type'           => bbp_get_topic_post_type(),
    762                                         'post_parent'         => $settings['parent_forum'],
    763                                         'posts_per_page'      => (int) $settings['max_shown'],
    764                                         'post_status'         => bbp_get_public_topic_statuses(),
    765                                         'ignore_sticky_posts' => true,
    766                                         'no_found_rows'       => true,
    767                                         'meta_key'            => '_bbp_reply_count',
    768                                         'meta_type'           => 'NUMERIC',
    769                                         'orderby'             => 'meta_value_num',
    770                                         'order'               => 'DESC'
     787
     788                                        // What and how
     789                                        'post_type'      => bbp_get_topic_post_type(),
     790                                        'post_status'    => bbp_get_public_topic_statuses(),
     791                                        'post_parent'    => $settings['parent_forum'],
     792                                        'posts_per_page' => (int) $settings['max_shown'],
     793                                        'meta_query'     => array( array(
     794                                                'key'  => '_bbp_reply_count',
     795                                                'type' => 'NUMERIC'
     796                                        ) ),
     797
     798                                        // Ordering
     799                                        'orderby' => 'meta_value_num',
     800                                        'order'   => 'DESC',
     801
     802                                        // Performance
     803                                        'ignore_sticky_posts'    => true,
     804                                        'no_found_rows'          => true,
     805                                        'nopaging'               => true,
     806                                        'update_post_term_cache' => false,
     807                                        'update_post_meta_cache' => false
    771808                                );
    772809                                break;
    … …  
    776813                        default :
    777814                                $topics_query = array(
    778                                         'post_type'           => bbp_get_topic_post_type(),
    779                                         'post_parent'         => $settings['parent_forum'],
    780                                         'posts_per_page'      => (int) $settings['max_shown'],
    781                                         'post_status'         => bbp_get_public_topic_statuses(),
    782                                         'ignore_sticky_posts' => true,
    783                                         'no_found_rows'       => true,
    784                                         'order'               => 'DESC'
     815
     816                                        // What and how
     817                                        'post_type'      => bbp_get_topic_post_type(),
     818                                        'post_status'    => bbp_get_public_topic_statuses(),
     819                                        'post_parent'    => $settings['parent_forum'],
     820                                        'posts_per_page' => (int) $settings['max_shown'],
     821
     822                                        // Ordering
     823                                        'orderby' => 'date',
     824                                        'order'   => 'DESC',
     825
     826                                        // Performance
     827                                        'ignore_sticky_posts'    => true,
     828                                        'no_found_rows'          => true,
     829                                        'nopaging'               => true,
     830                                        'update_post_term_cache' => false,
     831                                        'update_post_meta_cache' => false
    785832                                );
    786833                                break;
    … …  
    796843                }
    797844
     845                // Start an output buffer
     846                ob_start();
     847
    798848                echo $args['before_widget'];
    799849
    … …  
    802852                } ?>
    803853
    804                 <ul>
     854                <ul class="bbp-topics-widget <?php echo esc_attr( $settings['order_by'] ); ?>">
    805855
    806856                        <?php while ( $widget_query->have_posts() ) :
    … …  
    820870                                        <?php if ( ! empty( $author_link ) ) : ?>
    821871
    822                                                 <?php printf( _x( 'by %1$s', 'widgets', 'bbpress' ), '<span class="topic-author">' . $author_link . '</span>' ); ?>
     872                                                <?php printf( esc_html_x( 'by %1$s', 'widgets', 'bbpress' ), '<span class="topic-author">' . $author_link . '</span>' ); ?>
    823873
    824874                                        <?php endif; ?>
    … …  
    840890                // Reset the $post global
    841891                wp_reset_postdata();
     892
     893                // Output the current buffer
     894                echo ob_get_clean();
    842895        }
    843896
    … …  
    881934                $settings = $this->parse_settings( $instance ); ?>
    882935
    883                 <p><label for="<?php echo $this->get_field_id( 'title'     ); ?>"><?php _e( 'Title:',                  'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'title'     ); ?>" name="<?php echo $this->get_field_name( 'title'     ); ?>" type="text" value="<?php echo esc_attr( $settings['title']     ); ?>" /></label></p>
    884                 <p><label for="<?php echo $this->get_field_id( 'max_shown' ); ?>"><?php _e( 'Maximum topics to show:', 'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'max_shown' ); ?>" name="<?php echo $this->get_field_name( 'max_shown' ); ?>" type="text" value="<?php echo esc_attr( $settings['max_shown'] ); ?>" /></label></p>
     936                <p><label for="<?php echo $this->get_field_id( 'title'     ); ?>"><?php esc_html_e( 'Title:',                  'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'title'     ); ?>" name="<?php echo $this->get_field_name( 'title'     ); ?>" type="text" value="<?php echo esc_attr( $settings['title']     ); ?>" /></label></p>
     937                <p><label for="<?php echo $this->get_field_id( 'max_shown' ); ?>"><?php esc_html_e( 'Maximum topics to show:', 'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'max_shown' ); ?>" name="<?php echo $this->get_field_name( 'max_shown' ); ?>" type="text" value="<?php echo esc_attr( $settings['max_shown'] ); ?>" /></label></p>
    885938
    886939                <p>
    887                         <label for="<?php echo $this->get_field_id( 'parent_forum' ); ?>"><?php _e( 'Parent Forum ID:', 'bbpress' ); ?>
     940                        <label for="<?php echo $this->get_field_id( 'parent_forum' ); ?>"><?php esc_html_e( 'Parent Forum ID:', 'bbpress' ); ?>
    888941                                <input class="widefat" id="<?php echo $this->get_field_id( 'parent_forum' ); ?>" name="<?php echo $this->get_field_name( 'parent_forum' ); ?>" type="text" value="<?php echo esc_attr( $settings['parent_forum'] ); ?>" />
    889942                        </label>
    … …  
    891944                        <br />
    892945
    893                         <small><?php _e( '"0" to show only root - "any" to show all', 'bbpress' ); ?></small>
     946                        <small><?php esc_html_e( '"0" to show only root - "any" to show all', 'bbpress' ); ?></small>
    894947                </p>
    895948
    896                 <p><label for="<?php echo $this->get_field_id( 'show_date' ); ?>"><?php _e( 'Show post date:',    'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_date' ); ?>" name="<?php echo $this->get_field_name( 'show_date' ); ?>" <?php checked( true, $settings['show_date'] ); ?> value="1" /></label></p>
    897                 <p><label for="<?php echo $this->get_field_id( 'show_user' ); ?>"><?php _e( 'Show topic author:', 'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_user' ); ?>" name="<?php echo $this->get_field_name( 'show_user' ); ?>" <?php checked( true, $settings['show_user'] ); ?> value="1" /></label></p>
     949                <p><label for="<?php echo $this->get_field_id( 'show_date' ); ?>"><?php esc_html_e( 'Show post date:',    'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_date' ); ?>" name="<?php echo $this->get_field_name( 'show_date' ); ?>" <?php checked( true, $settings['show_date'] ); ?> value="1" /></label></p>
     950                <p><label for="<?php echo $this->get_field_id( 'show_user' ); ?>"><?php esc_html_e( 'Show topic author:', 'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_user' ); ?>" name="<?php echo $this->get_field_name( 'show_user' ); ?>" <?php checked( true, $settings['show_user'] ); ?> value="1" /></label></p>
    898951
    899952                <p>
    900                         <label for="<?php echo $this->get_field_id( 'order_by' ); ?>"><?php _e( 'Order By:', 'bbpress' ); ?></label>
     953                        <label for="<?php echo $this->get_field_id( 'order_by' ); ?>"><?php esc_html_e( 'Order By:', 'bbpress' ); ?></label>
    901954                        <select name="<?php echo $this->get_field_name( 'order_by' ); ?>" id="<?php echo $this->get_field_id( 'order_by' ); ?>">
    902                                 <option <?php selected( $settings['order_by'], 'newness' );   ?> value="newness"><?php _e( 'Newest Topics',                'bbpress' ); ?></option>
    903                                 <option <?php selected( $settings['order_by'], 'popular' );   ?> value="popular"><?php _e( 'Popular Topics',               'bbpress' ); ?></option>
    904                                 <option <?php selected( $settings['order_by'], 'freshness' ); ?> value="freshness"><?php _e( 'Topics With Recent Replies', 'bbpress' ); ?></option>
     955                                <option <?php selected( $settings['order_by'], 'newness'   ); ?> value="newness"><?php esc_html_e( 'Newest Topics',                'bbpress' ); ?></option>
     956                                <option <?php selected( $settings['order_by'], 'popular'   ); ?> value="popular"><?php esc_html_e( 'Popular Topics',               'bbpress' ); ?></option>
     957                                <option <?php selected( $settings['order_by'], 'freshness' ); ?> value="freshness"><?php esc_html_e( 'Topics With Recent Replies', 'bbpress' ); ?></option>
    905958                        </select>
    906959                </p>
    … …  
    919972        public function parse_settings( $instance = array() ) {
    920973                return bbp_parse_args( $instance, array(
    921                         'title'        => __( 'Recent Topics', 'bbpress' ),
     974                        'title'        => esc_html__( 'Recent Topics', 'bbpress' ),
    922975                        'max_shown'    => 5,
    923976                        'show_date'    => false,
    … …  
    930983
    931984/**
    932  * bbPress Stats Widget
     985 * bbPress Statistics Widget
    933986 *
    934987 * Adds a widget which displays the forum statistics
    … …  
    941994
    942995        /**
    943          * bbPress Stats Widget
    944          *
    945          * Registers the stats widget
     996         * bbPress Statistics Widget
     997         *
     998         * Registers the statistics widget
    946999         *
    9471000         * @since 2.3.0 bbPress (r4509)
    … …  
    9531006                $widget_ops = apply_filters( 'bbp_stats_widget_options', array(
    9541007                        'classname'   => 'widget_display_stats',
    955                         'description' => __( 'Some statistics from your forum.', 'bbpress' )
     1008                        'description' => esc_html__( 'Some statistics from your forum.', 'bbpress' )
    9561009                ) );
    9571010
    958                 parent::__construct( false, __( '(bbPress) Statistics', 'bbpress' ), $widget_ops );
     1011                parent::__construct( false, esc_html__( '(bbPress) Statistics', 'bbpress' ), $widget_ops );
    9591012        }
    9601013
    … …  
    10041057
    10051058        /**
    1006          * Update the stats widget options
     1059         * Update the statistics widget options
    10071060         *
    10081061         * @since 2.3.0 bbPress (r4509)
    … …  
    10211074
    10221075        /**
    1023          * Output the stats widget options form
     1076         * Output the statistics widget options form
    10241077         *
    10251078         * @since 2.3.0 bbPress (r4509)
    … …  
    10351088
    10361089                <p>
    1037                         <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'bbpress' ); ?>
     1090                        <label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'bbpress' ); ?>
    10381091                                <input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo esc_attr( $settings['title'] ); ?>"/>
    10391092                        </label>
    … …  
    10531106        public function parse_settings( $instance = array() ) {
    10541107                return bbp_parse_args( $instance, array(
    1055                         'title' => __( 'Forum Statistics', 'bbpress' )
    1056                 ),
    1057                 'stats_widget_settings' );
     1108                        'title' => esc_html__( 'Forum Statistics', 'bbpress' )
     1109                ), 'stats_widget_settings' );
    10581110        }
    10591111}
    … …  
    10831135                $widget_ops = apply_filters( 'bbp_replies_widget_options', array(
    10841136                        'classname'   => 'widget_display_replies',
    1085                         'description' => __( 'A list of the most recent replies.', 'bbpress' )
     1137                        'description' => esc_html__( 'A list of the most recent replies.', 'bbpress' )
    10861138                ) );
    10871139
    1088                 parent::__construct( false, __( '(bbPress) Recent Replies', 'bbpress' ), $widget_ops );
     1140                parent::__construct( false, esc_html__( '(bbPress) Recent Replies', 'bbpress' ), $widget_ops );
    10891141        }
    10901142
    … …  
    11301182                // bbp_pre_get_posts_normalize_forum_visibility action and function.
    11311183                $widget_query = new WP_Query( array(
    1132                         'post_type'           => bbp_get_reply_post_type(),
    1133                         'post_status'         => bbp_get_public_topic_statuses(),
    1134                         'posts_per_page'      => (int) $settings['max_shown'],
    1135                         'ignore_sticky_posts' => true,
    1136                         'no_found_rows'       => true,
     1184
     1185                        // What and when
     1186                        'post_type'      => bbp_get_reply_post_type(),
     1187                        'post_status'    => bbp_get_public_topic_statuses(),
     1188                        'posts_per_page' => (int) $settings['max_shown'],
     1189
     1190                        // Performance
     1191                        'ignore_sticky_posts'    => true,
     1192                        'no_found_rows'          => true,
     1193                        'nopaging'               => true,
     1194                        'update_post_term_cache' => false,
     1195                        'update_post_meta_cache' => false
    11371196                ) );
    11381197
    … …  
    11421201                }
    11431202
     1203                // Start an output buffer
     1204                ob_start();
     1205
    11441206                echo $args['before_widget'];
    11451207
    … …  
    11481210                } ?>
    11491211
    1150                 <ul>
     1212                <ul class="bbp-replies-widget">
    11511213
    11521214                        <?php while ( $widget_query->have_posts() ) : $widget_query->the_post(); ?>
    … …  
    11711233
    11721234                                                // translators: 1: reply author, 2: reply link, 3: reply timestamp
    1173                                                 printf( _x( '%1$s on %2$s %3$s', 'widgets', 'bbpress' ), $author_link, $reply_link, '<div>' . bbp_get_time_since( get_the_time( 'U' ) ) . '</div>' );
     1235                                                printf( esc_html_x( '%1$s on %2$s %3$s', 'widgets', 'bbpress' ), $author_link, $reply_link, '<div>' . bbp_get_time_since( get_the_time( 'U' ) ) . '</div>' );
    11741236
    11751237                                        // Reply link and timestamp
    … …  
    11771239
    11781240                                                // translators: 1: reply link, 2: reply timestamp
    1179                                                 printf( _x( '%1$s %2$s',         'widgets', 'bbpress' ), $reply_link,  '<div>' . bbp_get_time_since( get_the_time( 'U' ) ) . '</div>'              );
     1241                                                printf( esc_html_x( '%1$s %2$s',         'widgets', 'bbpress' ), $reply_link,  '<div>' . bbp_get_time_since( get_the_time( 'U' ) ) . '</div>'              );
    11801242
    11811243                                        // Reply author and title
    … …  
    11831245
    11841246                                                // translators: 1: reply author, 2: reply link
    1185                                                 printf( _x( '%1$s on %2$s',      'widgets', 'bbpress' ), $author_link, $reply_link                                                                 );
     1247                                                printf( esc_html_x( '%1$s on %2$s',      'widgets', 'bbpress' ), $author_link, $reply_link                                                                 );
    11861248
    11871249                                        // Only the reply title
    … …  
    11891251
    11901252                                                // translators: 1: reply link
    1191                                                 printf( _x( '%1$s',              'widgets', 'bbpress' ), $reply_link                                                                               );
     1253                                                printf( esc_html_x( '%1$s',              'widgets', 'bbpress' ), $reply_link                                                                               );
    11921254
    11931255                                        endif;
    … …  
    12051267                // Reset the $post global
    12061268                wp_reset_postdata();
     1269
     1270                // Output the current buffer
     1271                echo ob_get_clean();
    12071272        }
    12081273
    … …  
    12391304                $settings = $this->parse_settings( $instance ); ?>
    12401305
    1241                 <p><label for="<?php echo $this->get_field_id( 'title'     ); ?>"><?php _e( 'Title:',                   'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'title'     ); ?>" name="<?php echo $this->get_field_name( 'title'     ); ?>" type="text" value="<?php echo esc_attr( $settings['title']     ); ?>" /></label></p>
    1242                 <p><label for="<?php echo $this->get_field_id( 'max_shown' ); ?>"><?php _e( 'Maximum replies to show:', 'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'max_shown' ); ?>" name="<?php echo $this->get_field_name( 'max_shown' ); ?>" type="text" value="<?php echo esc_attr( $settings['max_shown'] ); ?>" /></label></p>
    1243                 <p><label for="<?php echo $this->get_field_id( 'show_date' ); ?>"><?php _e( 'Show post date:',          'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_date' ); ?>" name="<?php echo $this->get_field_name( 'show_date' ); ?>" <?php checked( true, $settings['show_date'] ); ?> value="1" /></label></p>
    1244                 <p><label for="<?php echo $this->get_field_id( 'show_user' ); ?>"><?php _e( 'Show reply author:',       'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_user' ); ?>" name="<?php echo $this->get_field_name( 'show_user' ); ?>" <?php checked( true, $settings['show_user'] ); ?> value="1" /></label></p>
     1306                <p><label for="<?php echo $this->get_field_id( 'title'     ); ?>"><?php esc_html_e( 'Title:',                   'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'title'     ); ?>" name="<?php echo $this->get_field_name( 'title'     ); ?>" type="text" value="<?php echo esc_attr( $settings['title']     ); ?>" /></label></p>
     1307                <p><label for="<?php echo $this->get_field_id( 'max_shown' ); ?>"><?php esc_html_e( 'Maximum replies to show:', 'bbpress' ); ?> <input class="widefat" id="<?php echo $this->get_field_id( 'max_shown' ); ?>" name="<?php echo $this->get_field_name( 'max_shown' ); ?>" type="text" value="<?php echo esc_attr( $settings['max_shown'] ); ?>" /></label></p>
     1308                <p><label for="<?php echo $this->get_field_id( 'show_date' ); ?>"><?php esc_html_e( 'Show post date:',          'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_date' ); ?>" name="<?php echo $this->get_field_name( 'show_date' ); ?>" <?php checked( true, $settings['show_date'] ); ?> value="1" /></label></p>
     1309                <p><label for="<?php echo $this->get_field_id( 'show_user' ); ?>"><?php esc_html_e( 'Show reply author:',       'bbpress' ); ?> <input type="checkbox" id="<?php echo $this->get_field_id( 'show_user' ); ?>" name="<?php echo $this->get_field_name( 'show_user' ); ?>" <?php checked( true, $settings['show_user'] ); ?> value="1" /></label></p>
    12451310
    12461311                <?php
    … …  
    12571322        public function parse_settings( $instance = array() ) {
    12581323                return bbp_parse_args( $instance, array(
    1259                         'title'     => __( 'Recent Replies', 'bbpress' ),
     1324                        'title'     => esc_html__( 'Recent Replies', 'bbpress' ),
    12601325                        'max_shown' => 5,
    12611326                        'show_date' => false,
    12621327                        'show_user' => false
    1263                 ),
    1264                 'replies_widget_settings' );
     1328                ), 'replies_widget_settings' );
    12651329        }
    12661330}
Note: See TracChangeset for help on using the changeset viewer.