Skip to:
Content

bbPress.org

Changeset 3825


Ignore:
Timestamp:
03/25/2012 09:37:42 PM (13 years ago)
Author:
johnjamesjacoby
Message:

Audit widgets for untranslated or incorrectly translated strings.

  • Remove PHP4 compat from widgets
  • Add missing translation to recent replies widget
  • Add brackets to titles to better group widgets together
  • Fixes #1797
File:
1 edited

Legend:

Unmodified
Added
Removed

  • branches/plugin/bbp-includes/bbp-core-widgets.php

    r3620 r3825  
    2525
    2626    /**
    27      * Register the widget
    28      *
    29      * @since bbPress (r3389)
    30      *
    31      * @uses register_widget()
    32      */
    33     function register_widget() {
    34         register_widget( 'BBP_Login_Widget' );
    35     }
    36 
    37     /**
    3827     * bbPress Login Widget
    3928     *
    … …  
    4534     *                        widget options
    4635     */
    47     function BBP_Login_Widget() {
     36    public function __construct() {
    4837        $widget_ops = apply_filters( 'bbp_login_widget_options', array(
    4938            'classname'   => 'bbp_widget_login',
    50             'description' => __( 'The login widget.', 'bbpress' )
     39            'description' => __( 'A simple login form with optional links to sign-up and lost password pages.', 'bbpress' )
    5140        ) );
    5241
    53         parent::WP_Widget( false, __( 'bbPress Login Widget', 'bbpress' ), $widget_ops );
     42        parent::__construct( false, __( '(bbPress) Login Widget', 'bbpress' ), $widget_ops );
     43    }
     44
     45    /**
     46     * Register the widget
     47     *
     48     * @since bbPress (r3389)
     49     *
     50     * @uses register_widget()
     51     */
     52    public function register_widget() {
     53        register_widget( 'BBP_Login_Widget' );
    5454    }
    5555
    … …  
    6464     * @uses get_template_part() To get the login/logged in form
    6565     */
    66     function widget( $args, $instance ) {
     66    public function widget( $args, $instance ) {
    6767        extract( $args );
    6868
    … …  
    152152     * @param array $old_instance The old instance options
    153153     */
    154     function update( $new_instance, $old_instance ) {
     154    public function update( $new_instance, $old_instance ) {
    155155        $instance             = $old_instance;
    156156        $instance['title']    = strip_tags( $new_instance['title'] );
    … …  
    170170     * @uses BBP_Login_Widget::get_field_name() To output the field name
    171171     */
    172     function form( $instance ) {
     172    public function form( $instance ) {
    173173
    174174        // Form values
    … …  
    210210
    211211    /**
    212      * Register the widget
    213      *
    214      * @since bbPress (r3389)
    215      *
    216      * @uses register_widget()
    217      */
    218     function register_widget() {
    219         register_widget( 'BBP_Views_Widget' );
    220     }
    221 
    222     /**
    223212     * bbPress View Widget
    224213     *
    … …  
    230219     *                        widget options
    231220     */
    232     function BBP_Views_Widget() {
     221    public function __construct() {
    233222        $widget_ops = apply_filters( 'bbp_views_widget_options', array(
    234223            'classname'   => 'widget_display_views',
    235             'description' => __( 'A list of views.', 'bbpress' )
     224            'description' => __( 'A list of registered optional topic views.', 'bbpress' )
    236225        ) );
    237226
    238         parent::WP_Widget( false, __( 'bbPress View List', 'bbpress' ), $widget_ops );
     227        parent::__construct( false, __( '(bbPress) Topic Views List', 'bbpress' ), $widget_ops );
     228    }
     229
     230    /**
     231     * Register the widget
     232     *
     233     * @since bbPress (r3389)
     234     *
     235     * @uses register_widget()
     236     */
     237    public function register_widget() {
     238        register_widget( 'BBP_Views_Widget' );
    239239    }
    240240
    … …  
    251251     * @uses bbp_view_title() To output the view title
    252252     */
    253     function widget( $args, $instance ) {
     253    public function widget( $args, $instance ) {
    254254
    255255        // Only output widget contents if views exist
    … …  
    286286     * @param array $old_instance The old instance options
    287287     */
    288     function update( $new_instance, $old_instance ) {
     288    public function update( $new_instance, $old_instance ) {
    289289        $instance          = $old_instance;
    290290        $instance['title'] = strip_tags( $new_instance['title'] );
    … …  
    302302     * @uses BBP_Views_Widget::get_field_name() To output the field name
    303303     */
    304     function form( $instance ) {
     304    public function form( $instance ) {
    305305        $title = !empty( $instance['title'] ) ? esc_attr( $instance['title'] ) : ''; ?>
    306306
    … …  
    327327
    328328    /**
    329      * Register the widget
    330      *
    331      * @since bbPress (r3389)
    332      *
    333      * @uses register_widget()
    334      */
    335     function register_widget() {
    336         register_widget( 'BBP_Forums_Widget' );
    337     }
    338 
    339     /**
    340329     * bbPress Forum Widget
    341330     *
    … …  
    347336     *                        widget options
    348337     */
    349     function BBP_Forums_Widget() {
     338    public function __construct() {
    350339        $widget_ops = apply_filters( 'bbp_forums_widget_options', array(
    351340            'classname'   => 'widget_display_forums',
    352             'description' => __( 'A list of forums.', 'bbpress' )
     341            'description' => __( 'A list of forums with an option to set the parent.', 'bbpress' )
    353342        ) );
    354343
    355         parent::WP_Widget( false, __( 'bbPress Forum List', 'bbpress' ), $widget_ops );
     344        parent::__construct( false, __( '(bbPress) Forums List', 'bbpress' ), $widget_ops );
     345    }
     346
     347    /**
     348     * Register the widget
     349     *
     350     * @since bbPress (r3389)
     351     *
     352     * @uses register_widget()
     353     */
     354    public function register_widget() {
     355        register_widget( 'BBP_Forums_Widget' );
    356356    }
    357357
    … …  
    376376     * @uses bbp_forum_title() To display the forum title
    377377     */
    378     function widget( $args, $instance ) {
     378    public function widget( $args, $instance ) {
    379379        extract( $args );
    380380
    … …  
    421421     * @param array $old_instance The old instance options
    422422     */
    423     function update( $new_instance, $old_instance ) {
     423    public function update( $new_instance, $old_instance ) {
    424424        $instance                 = $old_instance;
    425425        $instance['title']        = strip_tags( $new_instance['title'] );
    … …  
    443443     * @uses BBP_Forums_Widget::get_field_name() To output the field name
    444444     */
    445     function form( $instance ) {
     445    public function form( $instance ) {
    446446        $title        = !empty( $instance['title']        ) ? esc_attr( $instance['title']        ) : '';
    447447        $parent_forum = !empty( $instance['parent_forum'] ) ? esc_attr( $instance['parent_forum'] ) : '0'; ?>
    … …  
    479479
    480480    /**
    481      * Register the widget
    482      *
    483      * @since bbPress (r3389)
    484      *
    485      * @uses register_widget()
    486      */
    487     function register_widget() {
    488         register_widget( 'BBP_Topics_Widget' );
    489     }
    490 
    491     /**
    492481     * bbPress Topic Widget
    493482     *
    … …  
    499488     *                        widget options
    500489     */
    501     function BBP_Topics_Widget() {
     490    public function __construct() {
    502491        $widget_ops = apply_filters( 'bbp_topics_widget_options', array(
    503492            'classname'   => 'widget_display_topics',
    … …  
    505494        ) );
    506495
    507         parent::WP_Widget( false, __( 'bbPress Topics List', 'bbpress' ), $widget_ops );
     496        parent::__construct( false, __( '(bbPress) Recent Topics', 'bbpress' ), $widget_ops );
     497    }
     498
     499    /**
     500     * Register the widget
     501     *
     502     * @since bbPress (r3389)
     503     *
     504     * @uses register_widget()
     505     */
     506    public function register_widget() {
     507        register_widget( 'BBP_Topics_Widget' );
    508508    }
    509509
    … …  
    529529     * @uses bbp_get_topic_reply_count() To get the topic reply count
    530530     */
    531     function widget( $args, $instance ) {
     531    public function widget( $args, $instance ) {
    532532
    533533        extract( $args );
    … …  
    624624     * @param array $old_instance The old instance options
    625625     */
    626     function update( $new_instance, $old_instance ) {
     626    public function update( $new_instance, $old_instance ) {
    627627        $instance              = $old_instance;
    628628        $instance['title']     = strip_tags( $new_instance['title']     );
    … …  
    643643     * @uses BBP_Topics_Widget::get_field_name() To output the field name
    644644     */
    645     function form( $instance ) {
     645    public function form( $instance ) {
    646646        $title     = !empty( $instance['title']     ) ? esc_attr( $instance['title']     ) : '';
    647647        $max_shown = !empty( $instance['max_shown'] ) ? esc_attr( $instance['max_shown'] ) : '';
    … …  
    673673
    674674    /**
    675      * Register the widget
    676      *
    677      * @since bbPress (r3389)
    678      *
    679      * @uses register_widget()
    680      */
    681     function register_widget() {
    682         register_widget( 'BBP_Replies_Widget' );
    683     }
    684 
    685     /**
    686675     * bbPress Replies Widget
    687676     *
    … …  
    693682     *                        widget options
    694683     */
    695     function BBP_Replies_Widget() {
     684    public function __construct() {
    696685        $widget_ops = apply_filters( 'bbp_replies_widget_options', array(
    697686            'classname'   => 'widget_display_replies',
    698             'description' => __( 'A list of bbPress recent replies.', 'bbpress' )
     687            'description' => __( 'A list of the most recent replies.', 'bbpress' )
    699688        ) );
    700689
    701         parent::WP_Widget( false, 'bbPress Reply List', $widget_ops );
     690        parent::__construct( false, __( '(bbPress) Recent Replies', 'bbpress' ), $widget_ops );
     691    }
     692
     693    /**
     694     * Register the widget
     695     *
     696     * @since bbPress (r3389)
     697     *
     698     * @uses register_widget()
     699     */
     700    public function register_widget() {
     701        register_widget( 'BBP_Replies_Widget' );
    702702    }
    703703
    … …  
    725725     * @uses get_the_time() To get the time of the reply
    726726     */
    727     function widget( $args, $instance ) {
     727    public function widget( $args, $instance ) {
    728728
    729729        extract( $args );
    … …  
    760760
    761761                        /* translators: bbpress replies widget: 1: reply author, 2: reply link, 3: reply date, 4: reply time */
    762                         printf( _x( $show_date == 'on' ? '%1$s on %2$s, %3$s, %4$s' : '%1$s on %2$s', 'widgets', 'bbpress' ), $author_link, $reply_link, get_the_date(), get_the_time() );
     762                        if ( $show_date == 'on' ) {
     763                            printf( _x( '%1$s on %2$s, %3$s, %4$s', 'widgets', 'bbpress' ), $author_link, $reply_link, get_the_date(), get_the_time() );
     764                        } else {
     765                            printf( _x( '%1$s on %2$s',             'widgets', 'bbpress' ), $author_link, $reply_link );
     766                        }
     767
    763768                        ?>
    764769
    … …  
    784789     * @param array $old_instance The old instance options
    785790     */
    786     function update( $new_instance, $old_instance ) {
     791    public function update( $new_instance, $old_instance ) {
    787792        $instance              = $old_instance;
    788793        $instance['title']     = strip_tags( $new_instance['title']     );
    … …  
    802807     * @uses BBP_Replies_Widget::get_field_name() To output the field name
    803808     */
    804     function form( $instance ) {
     809    public function form( $instance ) {
    805810        $title     = !empty( $instance['title']     ) ? esc_attr( $instance['title']     ) : '';
    806811        $max_shown = !empty( $instance['max_shown'] ) ? esc_attr( $instance['max_shown'] ) : '';
Note: See TracChangeset for help on using the changeset viewer.