WordPress.org

Make WordPress Core

Ticket #20088: 20088.4.diff

File 20088.4.diff, 5.2 KB (added by lancewillett, 6 years ago)

Use esc_url properly, props kobenland

  • wp-includes/comment-template.php

     
    12451245                        case 'div':
    12461246                                break;
    12471247                        case 'ol':
    1248                                 echo "<ol class='children'>\n";
     1248                                echo '<ol class="children">' . "\n";
    12491249                                break;
    12501250                        default:
    12511251                        case 'ul':
    1252                                 echo "<ul class='children'>\n";
     1252                                echo '<ul class="children">' . "\n";
    12531253                                break;
    12541254                }
    12551255        }
     
    12691269                        case 'div':
    12701270                                break;
    12711271                        case 'ol':
    1272                                 echo "</ol>\n";
     1272                                echo "</ol><!-- .children -->\n";
    12731273                                break;
    12741274                        default:
    12751275                        case 'ul':
    1276                                 echo "</ul>\n";
     1276                                echo "</ul><!-- .children -->\n";
    12771277                                break;
    12781278                }
    12791279        }
     
    13301330                $depth++;
    13311331                $GLOBALS['comment_depth'] = $depth;
    13321332
    1333                 if ( !empty($args['callback']) ) {
    1334                         call_user_func($args['callback'], $comment, $args, $depth);
     1333                if ( !empty( $args['callback'] ) ) {
     1334                        call_user_func( $args['callback'], $comment, $args, $depth );
    13351335                        return;
    13361336                }
    13371337
    13381338                $GLOBALS['comment'] = $comment;
    1339                 extract($args, EXTR_SKIP);
     1339                extract( $args, EXTR_SKIP );
    13401340
    13411341                if ( 'div' == $args['style'] ) {
    13421342                        $tag = 'div';
     
    13451345                        $tag = 'li';
    13461346                        $add_below = 'div-comment';
    13471347                }
    1348 ?>
    1349                 <<?php echo $tag ?> <?php comment_class(empty( $args['has_children'] ) ? '' : 'parent') ?> id="comment-<?php comment_ID() ?>">
    1350                 <?php if ( 'div' != $args['style'] ) : ?>
    1351                 <div id="div-comment-<?php comment_ID() ?>" class="comment-body">
    1352                 <?php endif; ?>
    1353                 <div class="comment-author vcard">
    1354                 <?php if ($args['avatar_size'] != 0) echo get_avatar( $comment, $args['avatar_size'] ); ?>
    1355                 <?php printf(__('<cite class="fn">%s</cite> <span class="says">says:</span>'), get_comment_author_link()) ?>
    1356                 </div>
    1357 <?php if ($comment->comment_approved == '0') : ?>
    1358                 <em class="comment-awaiting-moderation"><?php _e('Your comment is awaiting moderation.') ?></em>
    1359                 <br />
    1360 <?php endif; ?>
     1348               
     1349                switch ( $comment->comment_type ) :
     1350                        case 'pingback' :
     1351                        case 'trackback' :
     1352                        // Display trackbacks differently than normal comments.
     1353                ?>
     1354                <<?php echo $tag ?> <?php comment_class(); ?> id="comment-<?php comment_ID(); ?>">
     1355                        <div class="comment-body"><?php _e( 'Pingback:' ); ?> <?php comment_author_link(); ?> <?php edit_comment_link( __( '(Edit)' ), '<span class="edit-link">', '</span>' ); ?></div>
     1356                <?php
     1357                                break;
     1358                        default :
     1359                        // Proceed with normal comments.
     1360                ?>
     1361                <<?php echo $tag ?> <?php comment_class( empty( $args['has_children'] ) ? '' : 'parent' ); ?> id="comment-<?php comment_ID(); ?>">
     1362                        <?php if ( 'div' != $args['style'] ) : ?>
     1363                        <div id="div-comment-<?php comment_ID(); ?>" class="comment-body">
     1364                        <?php endif; ?>
     1365                        <div class="comment-author vcard">
     1366                                <?php
     1367                                        if ( 0 != $args['avatar_size'] )
     1368                                                echo get_avatar( $comment, $args['avatar_size'] );
     1369                                ?>
     1370                                <?php printf( __( '<cite class="fn">%s</cite> <span class="says">says:</span>' ), get_comment_author_link() ); ?>
     1371                        </div><!-- .comment-author -->
    13611372
    1362                 <div class="comment-meta commentmetadata"><a href="<?php echo htmlspecialchars( get_comment_link( $comment->comment_ID ) ) ?>">
    1363                         <?php
    1364                                 /* translators: 1: date, 2: time */
    1365                                 printf( __('%1$s at %2$s'), get_comment_date(),  get_comment_time()) ?></a><?php edit_comment_link(__('(Edit)'),'&nbsp;&nbsp;','' );
    1366                         ?>
    1367                 </div>
     1373                        <div class="comment-meta commentmetadata">
     1374                                <a href="<?php echo esc_url( get_comment_link( $comment->comment_ID ) ); ?>">
     1375                                <?php printf( __( '%1$s at %2$s' ), get_comment_date(),  get_comment_time() ); /* translators: 1: date, 2: time */ ?></a>
     1376                                <?php edit_comment_link( __( '(Edit)' ), '<span class="edit-link">', '</span>' ); ?>
     1377                        </div><!-- .comment-meta -->
    13681378
    1369                 <?php comment_text() ?>
     1379                        <?php if ( '0' == $comment->comment_approved ) : ?>
     1380                                <p class="comment-awaiting-moderation"><em><?php _e( 'Your comment is awaiting moderation.' ); ?></em><p>
     1381                        <?php endif; ?>
    13701382
    1371                 <div class="reply">
    1372                 <?php comment_reply_link(array_merge( $args, array('add_below' => $add_below, 'depth' => $depth, 'max_depth' => $args['max_depth']))) ?>
    1373                 </div>
    1374                 <?php if ( 'div' != $args['style'] ) : ?>
    1375                 </div>
    1376                 <?php endif; ?>
    1377 <?php
     1383                        <div class="comment-text">
     1384                                <?php comment_text(); ?>
     1385                        </div><!-- .comment_text -->
     1386
     1387                        <div class="reply">
     1388                                <?php comment_reply_link( array_merge( $args, array( 'add_below' => $add_below, 'depth' => $depth, 'max_depth' => $args['max_depth'] ) ) ); ?>
     1389                        </div><!-- .reply -->
     1390                        <?php if ( 'div' != $args['style'] ) : ?>
     1391                </div><!-- div-comment-## -->
     1392                        <?php endif;
     1393
     1394                        break;
     1395                endswitch; // end comment_type check
    13781396        }
    13791397
    13801398        /**
     
    13861404         * @param int $depth Depth of comment.
    13871405         * @param array $args
    13881406         */
    1389         function end_el(&$output, $comment, $depth = 0, $args = array() ) {
    1390                 if ( !empty($args['end-callback']) ) {
    1391                         call_user_func($args['end-callback'], $comment, $args, $depth);
     1407        function end_el( &$output, $comment, $depth = 0, $args = array() ) {
     1408                if ( !empty( $args['end-callback'] ) ) {
     1409                        call_user_func( $args['end-callback'], $comment, $args, $depth );
    13921410                        return;
    13931411                }
    13941412                if ( 'div' == $args['style'] )
    1395                         echo "</div>\n";
     1413                        echo "</div><!-- #comment-## -->\n";
    13961414                else
    1397                         echo "</li>\n";
     1415                        echo "</li><!-- #comment-## -->\n";
    13981416        }
    1399 
    14001417}
    14011418
    14021419/**