Skip to:
Content

BuddyPress.org

Changeset 3624


Ignore:
Timestamp:
12/30/2010 09:29:17 PM (14 years ago)
Author:
djpaul
Message:

Use comment_form() to comply with WP theme standards and improves code style in comments.php. Fixes #2978. Props denis.ogun (via Google Code In)

Location:
trunk/bp-themes/bp-default
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/bp-themes/bp-default/_inc/css/default.css

    r3606 r3624  
    873873}
    874874
    875 form.standard-form#signup_form div div.error {
     875.standard-form#signup_form div div.error {
    876876    color: #fff;
    877877    background: #e41717;
    … …  
    968968-------------------------------------------------------------- */
    969969
    970 form.standard-form textarea,
    971 form.standard-form input[type=text],
    972 form.standard-form select,
    973 form.standard-form input[type=password],
     970.standard-form textarea,
     971.standard-form input[type=text],
     972.standard-form select,
     973.standard-form input[type=password],
    974974.dir-search input[type=text] {
    975975    border: 1px inset #ccc;
    … …  
    983983}
    984984
    985 form.standard-form select {
     985.standard-form select {
    986986    padding: 3px;
    987987}
    988988
    989 form.standard-form input[type=password] {
     989.standard-form input[type=password] {
    990990    margin-bottom: 5px;
    991991}
    992992
    993 form.standard-form label,
    994 form.standard-form span.label {
     993.standard-form label,
     994.standard-form span.label {
    995995    display: block;
    996996    font-weight: bold;
    … …  
    998998}
    999999
    1000 form.standard-form div.checkbox label,
    1001 form.standard-form div.radio label {
     1000.standard-form div.checkbox label,
     1001.standard-form div.radio label {
    10021002    font-weight: normal;
    10031003    margin: 5px 0 0 0;
    … …  
    10061006}
    10071007
    1008 form.standard-form#sidebar-login-form label {
     1008.standard-form#sidebar-login-form label {
    10091009    margin-top: 5px;
    10101010}
    10111011
    1012 form.standard-form input[type=text] {
     1012.standard-form input[type=text] {
    10131013    width: 75%;
    10141014}
    10151015
    1016 form.standard-form#sidebar-login-form input[type=text],
    1017 form.standard-form#sidebar-login-form input[type=password] {
     1016.standard-form#sidebar-login-form input[type=text],
     1017.standard-form#sidebar-login-form input[type=password] {
    10181018    padding: 4px;
    10191019    width: 95%;
    10201020}
    10211021
    1022 form.standard-form #basic-details-section input[type=password],
    1023 form.standard-form #blog-details-section input#signup_blog_url {
     1022.standard-form #basic-details-section input[type=password],
     1023.standard-form #blog-details-section input#signup_blog_url {
    10241024    width: 35%;
    10251025}
    10261026
    1027 form.standard-form#signup_form input[type=text],
    1028 form.standard-form#signup_form textarea {
     1027.standard-form#signup_form input[type=text],
     1028.standard-form#signup_form textarea {
    10291029    width: 90%;
    10301030}
    10311031
    1032 form.standard-form#signup_form div.submit {
     1032.standard-form#signup_form div.submit {
    10331033    float: right;
    10341034}
    … …  
    10381038}
    10391039
    1040 form.standard-form textarea {
     1040.standard-form textarea {
    10411041    width: 75%;
    10421042    height: 120px;
    10431043}
    10441044
    1045 form.standard-form textarea#message_content {
     1045.standard-form textarea#message_content {
    10461046    height: 200px;
    10471047}
    10481048
    1049 form.standard-form#send-reply textarea {
     1049.standard-form#send-reply textarea {
    10501050    width: 97.5%;
    10511051}
    10521052
    1053 form.standard-form p.description {
     1053.standard-form p.description {
    10541054    font-size: 11px;
    10551055    color: #888;
    … …  
    10571057}
    10581058
    1059 form.standard-form div.submit {
     1059.standard-form div.submit {
    10601060    padding: 15px 0;
    10611061    clear: both;
    10621062}
    10631063
    1064 form.standard-form div.submit input {
     1064.standard-form div.submit input {
    10651065    margin-right: 15px;
    10661066}
    10671067
    1068 form.standard-form div.radio ul {
     1068.standard-form div.radio ul {
    10691069    margin: 10px 0 15px 38px;
    10701070    list-style: disc;
    10711071}
    10721072
    1073 form.standard-form div.radio ul li {
     1073.standard-form div.radio ul li {
    10741074    margin-bottom: 5px;
    10751075}
    10761076
    1077 form.standard-form a.clear-value {
     1077.standard-form a.clear-value {
    10781078    display: block;
    10791079    margin-top: 5px;
    … …  
    10811081}
    10821082
    1083 form.standard-form #basic-details-section,
    1084 form.standard-form #blog-details-section,
    1085 form.standard-form #profile-details-section {
     1083.standard-form #basic-details-section,
     1084.standard-form #blog-details-section,
     1085.standard-form #profile-details-section {
    10861086    float: left;
    10871087    width: 48%;
    10881088}
    10891089
    1090 form.standard-form #profile-details-section {
     1090.standard-form #profile-details-section {
    10911091    float: right;
    10921092}
    10931093
    1094 form.standard-form #blog-details-section {
     1094.standard-form #blog-details-section {
    10951095    clear: left;
    10961096}
    10971097
    1098 form.standard-form input:focus,
    1099 form.standard-form textarea:focus,
    1100 form.standard-form select:focus {
     1098.standard-form input:focus,
     1099.standard-form textarea:focus,
     1100.standard-form select:focus {
    11011101    background: #fafafa;
    11021102    color: #555;

  • trunk/bp-themes/bp-default/comments.php

    r3592 r3624  
    1     <?php
    2         if ( post_password_required() ) :
    3             echo '<h3 class="comments-header">' . __('Password Protected', 'buddypress') . '</h3>';
    4             echo '<p class="alert password-protected">' . __('Enter the password to view comments.', 'buddypress') . '</p>';
    5             return;
    6         endif;
     1<?php
     2    if ( post_password_required() ) {
     3        echo '<h3 class="comments-header">' . __( 'Password Protected', 'buddypress' ) . '</h3>';
     4        echo '<p class="alert password-protected">' . __( 'Enter the password to view comments.', 'buddypress' ) . '</p>';
     5        return;
     6    }
    77
    8         if ( is_page() && !have_comments() && !comments_open() && !pings_open() )
    9             return;
    10     ?>
     8    if ( is_page() && !have_comments() && !comments_open() && !pings_open() )
     9        return;
    1110
    12     <?php if ( have_comments() ) : ?>
     11    if ( have_comments() ) :
     12        $num_comments = 0;
     13        $num_trackbacks = 0;
     14        foreach ( (array)$comments as $comment ) {
     15            if ( 'comment' != get_comment_type() )
     16                $num_trackbacks++;
     17            else
     18                $num_comments++;
     19        }
     20?>
     21    <div id="comments">
    1322
    14         <div id="comments">
     23        <h3>
     24            <?php printf( _n( 'One Response to %2$s', '%1$s Responses to %2$s', $num_comments, 'buddypress' ), number_format_i18n( $num_comments ), '<em>' . get_the_title() . '</em>' ) ?>
     25        </h3>
    1526
    16             <?php
    17                 // Only include comments
    18                 $numTrackBacks = 0; $numComments = 0;
    19                 foreach ( (array)$comments as $comment )
    20                     if ( 'comment' != get_comment_type() )
    21                         $numTrackBacks++;
    22                     else
    23                         $numComments++;
    24             ?>
     27        <?php do_action( 'bp_before_blog_comment_list' ) ?>
    2528
    26             <h3 id="comments">
    27                 <?php
    28                     printf( _n( 'One Response to %2$s', '%1$s Responses to %2$s', $numComments, 'buddypress' ),
    29                     number_format_i18n( $numComments ), '<em>' . get_the_title() . '</em>' );
    30                 ?>
    31             </h3>
     29        <ol class="commentlist">
     30            <?php wp_list_comments( array( 'callback' => 'bp_dtheme_blog_comments' ) ) ?>
     31        </ol><!-- .comment-list -->
    3232
    33             <?php do_action( 'bp_before_blog_comment_list' ) ?>
     33        <?php do_action( 'bp_after_blog_comment_list' ) ?>
    3434
    35             <ol class="commentlist">
    36                 <?php wp_list_comments( array( 'callback' => 'bp_dtheme_blog_comments' ) ); ?>
    37             </ol><!-- .comment-list -->
    38 
    39             <?php do_action( 'bp_after_blog_comment_list' ) ?>
    40 
    41             <?php if ( get_option( 'page_comments' ) ) : ?>
    42 
    43                 <div class="comment-navigation paged-navigation">
    44 
    45                     <?php paginate_comments_links(); ?>
    46 
    47                 </div>
    48 
    49             <?php endif; ?>
    50 
    51         </div><!-- #comments -->
    52 
    53     <?php else : ?>
    54 
    55         <?php if ( pings_open() && !comments_open() && is_single() ) : ?>
    56 
    57             <p class="comments-closed pings-open">
    58                 <?php printf( __('Comments are closed, but <a href="%1$s" title="Trackback URL for this post">trackbacks</a> and pingbacks are open.', 'buddypress'), trackback_url( '0' ) ); ?>
    59             </p>
    60 
    61         <?php elseif ( !comments_open() && is_single() ) : ?>
    62 
    63             <p class="comments-closed">
    64                 <?php _e('Comments are closed.', 'buddypress'); ?>
    65             </p>
    66 
     35        <?php if ( get_option( 'page_comments' ) ) : ?>
     36            <div class="comment-navigation paged-navigation">
     37                <?php paginate_comments_links() ?>
     38            </div>
    6739        <?php endif; ?>
    6840
     41    </div><!-- #comments -->
     42<?php else : ?>
     43
     44    <?php if ( pings_open() && !comments_open() && is_single() ) : ?>
     45        <p class="comments-closed pings-open">
     46            <?php printf( __( 'Comments are closed, but <a href="%1$s" title="Trackback URL for this post">trackbacks</a> and pingbacks are open.', 'buddypress' ), trackback_url( '0' ) ) ?>
     47        </p>
     48    <?php elseif ( !comments_open() && is_single() ) : ?>
     49        <p class="comments-closed">
     50            <?php _e( 'Comments are closed.', 'buddypress' ) ?>
     51        </p>
    6952    <?php endif; ?>
    7053
    71         <?php if ( comments_open() ) : ?>
     54<?php endif; ?>
    7255
    73         <div id="respond">
     56<?php if ( comments_open() ) : ?>
     57    <div id="respond">
    7458
    75             <div class="comment-avatar-box">
    76                 <div class="avb">
    77                     <?php if ( bp_loggedin_user_id() ) : ?>
    78                         <a href="<?php echo bp_loggedin_user_domain() ?>">
    79                             <?php echo get_avatar( bp_loggedin_user_id(), 50 ); ?>
    80                         </a>
    81                     <?php else : ?>
    82                         <?php echo get_avatar( 0, 50 ); ?>
    83                     <?php endif; ?>
    84                 </div>
     59        <div class="comment-avatar-box">
     60            <div class="avb">
     61                <?php if ( bp_loggedin_user_id() ) : ?>
     62                    <a href="<?php echo bp_loggedin_user_domain() ?>">
     63                        <?php echo get_avatar( bp_loggedin_user_id(), 50 ) ?>
     64                    </a>
     65                <?php else : ?>
     66                    <?php echo get_avatar( 0, 50 ) ?>
     67                <?php endif; ?>
    8568            </div>
     69        </div>
    8670
    87             <div class="comment-content">
     71        <div class="comment-content standard-form">
     72            <?php comment_form() ?>
     73        </div><!-- .comment-content -->
    8874
    89                 <h3 id="reply" class="comments-header">
    90                     <?php comment_form_title( __( 'Leave a Reply', 'buddypress' ), __( 'Leave a Reply to %s', 'buddypress' ), true ); ?>
    91                 </h3>
     75    </div><!-- #respond -->
     76<?php endif; ?>
    9277
    93                 <p id="cancel-comment-reply">
    94                     <?php cancel_comment_reply_link( __( 'Click here to cancel reply.', 'buddypress' ) ); ?>
    95                 </p>
     78<?php if ( !empty( $num_trackbacks ) ) : ?>
     79    <div id="trackbacks">
     80        <span class="title"><?php the_title() ?></span>
    9681
    97                 <?php if ( get_option( 'comment_registration' ) && !$user_ID ) : ?>
     82        <h3>
     83            <?php printf( _n( '%d Trackback', '%d Trackbacks', $num_trackbacks, 'buddypress' ), number_format_i18n( $num_trackbacks ) ) ?>
     84        </h3>
    9885
    99                     <p class="alert">
    100                         <?php printf( __('You must be <a href="%1$s" title="Log in">logged in</a> to post a comment.', 'buddypress'), wp_login_url( get_permalink() ) ); ?>
    101                     </p>
     86        <ul id="trackbacklist">
     87            <?php foreach ( (array)$comments as $comment ) : ?>
    10288
    103                 <?php else : ?>
     89                <?php if ( 'comment' != get_comment_type() ) : ?>
     90                    <li>
     91                        <h5><?php comment_author_link() ?></h5>
     92                        <em>on <?php comment_date() ?></em>
     93                    </li>
     94                <?php endif; ?>
    10495
    105                     <?php do_action( 'bp_before_blog_comment_form' ) ?>
     96            <?php endforeach; ?>
     97        </ul>
    10698
    107                     <form action="<?php echo site_url( 'wp-comments-post.php' ) ?>" method="post" id="commentform" class="standard-form">
    108 
    109                         <?php if ( $user_ID ) : ?>
    110 
    111                             <p class="log-in-out">
    112                                 <?php printf( __('Logged in as <a href="%1$s" title="%2$s">%2$s</a>.', 'buddypress'), bp_loggedin_user_domain(), $user_identity ); ?> <a href="<?php echo wp_logout_url( get_permalink() ); ?>" title="<?php _e('Log out of this account', 'buddypress'); ?>"><?php _e('Log out &rarr;', 'buddypress'); ?></a>
    113                             </p>
    114 
    115                         <?php else : ?>
    116 
    117                             <?php $req = get_option( 'require_name_email' ); ?>
    118 
    119                             <p class="form-author">
    120                                 <label for="author"><?php _e('Name', 'buddypress'); ?> <?php if ( $req ) : ?><span class="required"><?php _e('*', 'buddypress'); ?></span><?php endif; ?></label>
    121                                 <input type="text" class="text-input" name="author" id="author" value="<?php echo $comment_author; ?>" size="40" tabindex="1" />
    122                             </p>
    123 
    124                             <p class="form-email">
    125                                 <label for="email"><?php _e('Email', 'buddypress'); ?>  <?php if ( $req ) : ?><span class="required"><?php _e('*', 'buddypress'); ?></span><?php endif; ?></label>
    126                                 <input type="text" class="text-input" name="email" id="email" value="<?php echo $comment_author_email; ?>" size="40" tabindex="2" />
    127                             </p>
    128 
    129                             <p class="form-url">
    130                                 <label for="url"><?php _e('Website', 'buddypress'); ?></label>
    131                                 <input type="text" class="text-input" name="url" id="url" value="<?php echo $comment_author_url; ?>" size="40" tabindex="3" />
    132                             </p>
    133 
    134                         <?php endif; ?>
    135 
    136                         <p class="form-textarea">
    137                             <label for="comment"><?php _e('Comment', 'buddypress'); ?></label>
    138                             <textarea name="comment" id="comment" cols="60" rows="10" tabindex="4"></textarea>
    139                         </p>
    140 
    141                         <?php do_action( 'bp_blog_comment_form' ) ?>
    142 
    143                         <p class="form-submit">
    144                             <input class="submit-comment button" name="submit" type="submit" id="submit" tabindex="5" value="<?php _e('Submit', 'buddypress'); ?>" />
    145                             <?php comment_id_fields(); ?>
    146                         </p>
    147 
    148                         <div class="comment-action">
    149                             <?php do_action( 'comment_form', $post->ID ); ?>
    150                         </div>
    151 
    152                     </form>
    153 
    154                     <?php do_action( 'bp_after_blog_comment_form' ) ?>
    155 
    156                 <?php endif; ?>
    157 
    158             </div><!-- .comment-content -->
    159         </div><!-- #respond -->
    160 
    161         <?php endif; ?>
    162 
    163         <?php if ( isset( $numTrackBacks ) && $numTrackBacks ) : ?>
    164             <div id="trackbacks">
    165 
    166                 <span class="title"><?php the_title() ?></span>
    167 
    168                 <?php if ( 1 == $numTrackBacks ) : ?>
    169                     <h3><?php printf( __( '%d Trackback', 'buddypress' ), $numTrackBacks ) ?></h3>
    170                 <?php else : ?>
    171                     <h3><?php printf( __( '%d Trackbacks', 'buddypress' ), $numTrackBacks ) ?></h3>
    172                 <?php endif; ?>
    173 
    174                 <ul id="trackbacklist">
    175                     <?php foreach ( (array)$comments as $comment ) : ?>
    176 
    177                         <?php if ( get_comment_type() != 'comment' ) : ?>
    178                             <li><h5><?php comment_author_link() ?></h5><em>on <?php comment_date() ?></em></li>
    179                         <?php endif; ?>
    180                     <?php endforeach; ?>
    181                 </ul>
    182             </div>
    183         <?php endif; ?>
     99    </div>
     100<?php endif; ?>

  • trunk/bp-themes/bp-default/functions.php

    r3614 r3624  
    478478 * @since 1.3
    479479 */
    480 function bp_dtheme_main_nav( $args ) {
    481 ?>
     480function bp_dtheme_main_nav( $args ) { ?>
    482481    <ul id="nav">
    483482        <li<?php if ( is_front_page() ) : ?> class="selected"<?php endif; ?>>
    … …  
    490489<?php
    491490}
     491
     492/**
     493 * Applies BuddyPress customisations to the post comment form.
     494 *
     495 * @global string $user_identity The display name of the user
     496 * @param array $default_labels The default options for strings, fields etc in the form
     497 * @see comment_form()
     498 * @since 1.3
     499 */
     500function bp_dtheme_comment_form( $default_labels ) {
     501    global $user_identity;
     502
     503    $commenter = wp_get_current_commenter();
     504    $req = get_option( 'require_name_email' );
     505    $aria_req = ( $req ? " aria-required='true'" : '' );
     506    $fields =  array(
     507        'author' => '<p class="comment-form-author">' . '<label for="author">' . __( 'Name' ) . ( $req ? '<span class="required"> *</span>' : '' ) . '</label> ' .
     508                    '<input id="author" name="author" type="text" value="' . esc_attr( $commenter['comment_author'] ) . '" size="30"' . $aria_req . ' /></p>',
     509        'email'  => '<p class="comment-form-email"><label for="email">' . __( 'Email' ) . ( $req ? '<span class="required"> *</span>' : '' ) . '</label> ' .
     510                    '<input id="email" name="email" type="text" value="' . esc_attr(  $commenter['comment_author_email'] ) . '" size="30"' . $aria_req . ' /></p>',
     511        'url'    => '<p class="comment-form-url"><label for="url">' . __( 'Website' ) . '</label>' .
     512                    '<input id="url" name="url" type="text" value="' . esc_attr( $commenter['comment_author_url'] ) . '" size="30" /></p>',
     513    );
     514
     515    $new_labels = array(
     516        'cancel_reply_link'    => '<p id="cancel-comment-reply">' . __( 'Cancel reply', 'buddypress' ) . '</p>',
     517        'comment_field'        => '<p class="form-textarea"><label for="comment">' . __( 'Comment', 'buddypress' ) . '</label><textarea name="comment" id="comment" cols="60" rows="10" aria-required="true"></textarea></p>',
     518        'comment_notes_after'  => '',
     519        'comment_notes_before' => '',
     520        'fields'               => apply_filters( 'comment_form_default_fields', $fields ),
     521        'logged_in_as'         => '<p class="log-in-out">' . sprintf( __( 'Logged in as <a href="%1$s">%2$s</a>. <a href="%3$s">Log out?</a>', 'buddypress' ), bp_loggedin_user_domain(), $user_identity, wp_logout_url( get_permalink() ) ) . '</p>',
     522        'must_log_in'          => '<p class="alert">' . sprintf( __( 'You must be <a href="%1$s">logged in</a> to post a comment.', 'buddypress' ), wp_login_url( get_permalink() ) )   . '</p>',
     523        'title_reply'          => '<h3 id="reply" class="comments-header">' . __( 'Leave a reply', 'buddypress' ) . '</h3>',
     524        'title_reply_to'       => '<h3 id="reply" class="comments-header">' . __( 'Leave a reply to %s', 'buddypress' ) . '</h3>'
     525    );
     526
     527    return apply_filters( 'bp_dtheme_comment_form', array_merge( $default_labels, $new_labels ) );
     528}
     529add_filter( 'comment_form_defaults', 'bp_dtheme_comment_form', 10 );
     530
    492531
    493532// Everything beyond this point is deprecated as of BuddyPress 1.3. This will be removed in a future version.

  • trunk/bp-themes/bp-default/rtl.css

    r3606 r3624  
    345345-------------------------------------------------------------- */
    346346
    347 form.standard-form#signup_form div.submit {
    348     float: left;
    349 }
    350 
    351 form.standard-form#signup_form div.signup-avatar {
     347.standard-form#signup_form div.submit {
     348    float: left;
     349}
     350
     351.standard-form#signup_form div.signup-avatar {
    352352    margin-left: 15px;
    353353    margin-right: 0;
    354354}
    355355
    356 form.standard-form div.submit input {
     356.standard-form div.submit input {
    357357    margin-left: 15px;
    358358    margin-right: 0;
    359359}
    360360
    361 form.standard-form div.radio ul {
     361.standard-form div.radio ul {
    362362    margin: 10px 38px 15px 0;
    363363}
    364364
    365 form.standard-form #basic-details-section,
    366 form.standard-form #blog-details-section,
    367 form.standard-form #profile-details-section {
    368     float: right;
    369 }
    370 
    371 form.standard-form #profile-details-section {
    372     float: left;
    373 }
    374 
    375 form.standard-form #blog-details-section {
     365.standard-form #basic-details-section,
     366.standard-form #blog-details-section,
     367.standard-form #profile-details-section {
     368    float: right;
     369}
     370
     371.standard-form #profile-details-section {
     372    float: left;
     373}
     374
     375.standard-form #blog-details-section {
    376376    clear: right;
    377377}
Note: See TracChangeset for help on using the changeset viewer.