WordPress.org

Make WordPress Core

Ticket #7769: 7769.2.diff

File 7769.2.diff, 14.9 KB (added by ryan, 10 years ago)

Implement previous_comments_link() and add options for paging and threading

  • wp-includes/comment-template.php

     
    850850
    851851        $args = wp_parse_args($args, $defaults);
    852852
    853         if ( 0 == $args['depth'] || $args['max_depth'] < $args['depth'] )
     853        if ( 0 == $args['depth'] || $args['max_depth'] <= $args['depth'] )
    854854                return;
    855855
    856856        extract($args, EXTR_SKIP);
     
    973973 * @uses Walker_Comment
    974974 *
    975975 * @param $args string|array Formatting options
    976         * @param $comments array Optional array of comment objects.  Defaults to $wp_query->comments
     976 * @param $comments array Optional array of comment objects.  Defaults to $wp_query->comments
    977977 */
    978978function wp_list_comments($args = array(), $comments = null ) {
    979979        global $wp_query;
    980980
    981         $defaults = array('walker' => null, 'depth' => 3, 'style' => 'ul', 'callback' => null, 'end-callback' => null, 'type' => 'all');
     981        $defaults = array('walker' => null, 'depth' => '', 'style' => 'ul', 'callback' => null, 'end-callback' => null, 'type' => 'all',
     982                'page' => get_query_var('cpage'), 'per_page' => '');
    982983
    983984        $r = wp_parse_args( $args, $defaults );
    984985
     986        if ( '' === $r['per_page'] && get_option('page_comments') )
     987                $r['per_page'] = get_query_var('comments_per_page');
     988
     989        if ( empty($r['per_page']) ) {
     990                $r['page'] = 0;
     991        } else {
     992                $r['page'] = intval($r['page']);
     993                if ( empty($r['page']) )
     994                        $r['page'] = 1;
     995        }
     996
     997        if ( '' === $r['depth'] ) {
     998                if ( get_option('thread_comments') )
     999                        $r['depth'] = get_option('thread_comments_depth');
     1000                else
     1001                        $r['depth'] = -1;
     1002        }
     1003
    9851004        extract( $r, EXTR_SKIP );
    9861005
    9871006        if ( empty($walker) )
     
    9951014                                $wp_query->comments_by_type = &separate_comments($wp_query->comments);
    9961015                        if ( empty($wp_query->comments_by_type[$type]) )
    9971016                                return;
    998                         return $walker->walk($wp_query->comments_by_type[$type], $depth, $r);
     1017                        $walker->paged_walk($wp_query->comments_by_type[$type], $depth, $page, $per_page, $r);
     1018                        $wp_query->max_num_comment_pages = $walker->max_pages;
     1019                        return;
    9991020                }
    1000                 $walker->walk($wp_query->comments, $depth, $r);
     1021                $walker->paged_walk($wp_query->comments, $depth, $page, $per_page, $r);
     1022                $wp_query->max_num_comment_pages = $walker->max_pages;
    10011023        } else {
    10021024                if ( empty($comments) )
    10031025                        return;
    10041026                if ( 'all' != $type ) {
    1005                         $comments_by_type = separate_comments($comments);
     1027                        $comments_by_type = &separate_comments($comments);
    10061028                        if ( empty($comments_by_type[$type]) )
    10071029                                return;
    1008                         return $walker->walk($comments_by_type[$type], $depth, $r);
     1030                        $walker->paged_walk($comments_by_type[$type], $depth, $page, $per_page, $r);
     1031                        $wp_query->max_num_comment_pages = $walker->max_pages;
     1032                        return;
    10091033                }
    1010                 $walker->walk($comments, $depth, $r);
     1034                $walker->paged_walk($comments, $depth, $page, $per_page, $r);
     1035                $wp_query->max_num_comment_pages = $walker->max_pages;
    10111036        }
    10121037}
    10131038
  • wp-includes/query.php

     
    844844        var $max_num_pages = 0;
    845845
    846846        /**
     847         * The amount of comment pages.
     848         *
     849         * @since 2.7.0
     850         * @access public
     851         * @var int
     852         */
     853        var $max_num_comment_pages = 0;
     854
     855        /**
    847856         * Set if query is single post.
    848857         *
    849858         * @since 1.5.0
     
    16121621                else if ( $q['posts_per_page'] == 0 )
    16131622                        $q['posts_per_page'] = 1;
    16141623
     1624                if ( !isset($q['comments_per_page']) || $q['comments_per_page'] == 0 )
     1625                        $q['comments_per_page'] = get_option('comments_per_page');
     1626
    16151627                if ( $this->is_home && (empty($this->query) || $q['preview'] == 'true') && ( 'page' == get_option('show_on_front') ) && get_option('page_on_front') ) {
    16161628                        $this->is_page = true;
    16171629                        $this->is_home = false;
  • wp-includes/link-template.php

     
    793793        }
    794794}
    795795
     796function get_comments_pagenum_link($pagenum = 1) {
     797        global $wp_rewrite;
     798
     799        $pagenum = (int) $pagenum;
     800
     801        $request = remove_query_arg( 'cpage' );
     802
     803        $home_root = parse_url(get_option('home'));
     804        $home_root = ( isset($home_root['path']) ) ? $home_root['path'] : '';
     805        $home_root = preg_quote( trailingslashit( $home_root ), '|' );
     806
     807        $request = preg_replace('|^'. $home_root . '|', '', $request);
     808        $request = preg_replace('|^/+|', '', $request);
     809
     810        $base = trailingslashit( get_bloginfo( 'home' ) );
     811
     812        if ( $pagenum > 1 ) {
     813                $result = add_query_arg( 'cpage', $pagenum, $base . $request );
     814        } else {
     815                $result = $base . $request;
     816        }
     817
     818        $result = apply_filters('get_comments_pagenum_link', $result);
     819
     820        return $result;
     821}
     822
     823function next_comments_link($label='', $max_page = 0) {
     824        global $wp_query;
     825
     826        if ( !is_singular() )
     827                return;
     828
     829        $page = get_query_var('cpage');
     830       
     831        if ( !$page )
     832                $page = 1;
     833
     834        if ( !$page )
     835                $page = 1;
     836
     837        $nextpage = intval($page) + 1;
     838
     839        if ( empty($max_page) )
     840                $max_page = $wp_query->max_num_comment_pages;
     841
     842        if ( $nextpage > $max_page )
     843                return;
     844
     845        if ( empty($label) )
     846                $label = __('&raquo; Newer Comments');
     847
     848        echo '<a href="' . clean_url(get_comments_pagenum_link($nextpage));
     849        $attr = apply_filters( 'next_comments_link_attributes', '' );
     850        echo "\" $attr>". preg_replace('/&([^#])(?![a-z]{1,8};)/', '&#038;$1', $label) .'</a>';
     851}
     852
     853function previous_comments_link($label='') {
     854        global $wp_query;
     855
     856        if ( !is_singular() )
     857                return;
     858
     859        $page = get_query_var('cpage');
     860
     861        if ( $page <= 1 )
     862                return;
     863
     864        $nextpage = intval($page) - 1;
     865
     866        if ( empty($label) )
     867                $label = __('&laquo; Older Comments');
     868
     869        echo '<a href="' . clean_url(get_comments_pagenum_link($nextpage));
     870        $attr = apply_filters( 'previous_comments_link_attributes', '' );
     871        echo "\" $attr>". preg_replace('/&([^#])(?![a-z]{1,8};)/', '&#038;$1', $label) .'</a>';
     872}
     873
    796874function get_shortcut_link() {
    797875        $link = "javascript:
    798876                        var d=document,
  • wp-includes/formatting.php

     
    18201820                case 'default_category':
    18211821                case 'default_email_category':
    18221822                case 'default_link_category':
     1823                case 'close_comments_days_old':
     1824                case 'comments_per_page':
     1825                case 'thread_comments_depth':
    18231826                        $value = abs((int) $value);
    18241827                        break;
    18251828
  • wp-includes/classes.php

     
    2626         * @access public
    2727         * @var array
    2828         */
    29         var $public_query_vars = array('m', 'p', 'posts', 'w', 'cat', 'withcomments', 'withoutcomments', 's', 'search', 'exact', 'sentence', 'debug', 'calendar', 'page', 'paged', 'more', 'tb', 'pb', 'author', 'order', 'orderby', 'year', 'monthnum', 'day', 'hour', 'minute', 'second', 'name', 'category_name', 'tag', 'feed', 'author_name', 'static', 'pagename', 'page_id', 'error', 'comments_popup', 'attachment', 'attachment_id', 'subpost', 'subpost_id', 'preview', 'robots', 'taxonomy', 'term');
     29        var $public_query_vars = array('m', 'p', 'posts', 'w', 'cat', 'withcomments', 'withoutcomments', 's', 'search', 'exact', 'sentence', 'debug', 'calendar', 'page', 'paged', 'more', 'tb', 'pb', 'author', 'order', 'orderby', 'year', 'monthnum', 'day', 'hour', 'minute', 'second', 'name', 'category_name', 'tag', 'feed', 'author_name', 'static', 'pagename', 'page_id', 'error', 'comments_popup', 'attachment', 'attachment_id', 'subpost', 'subpost_id', 'preview', 'robots', 'taxonomy', 'term', 'cpage');
    3030
    3131        /**
    3232         * Private query variables.
     
    3636         * @since 2.0.0
    3737         * @var array
    3838         */
    39         var $private_query_vars = array('offset', 'posts_per_page', 'posts_per_archive_page', 'what_to_show', 'showposts', 'nopaging', 'post_type', 'post_status', 'category__in', 'category__not_in', 'category__and', 'tag__in', 'tag__not_in', 'tag__and', 'tag_slug__in', 'tag_slug__and', 'tag_id', 'post_mime_type', 'perm');
     39        var $private_query_vars = array('offset', 'posts_per_page', 'posts_per_archive_page', 'what_to_show', 'showposts', 'nopaging', 'post_type', 'post_status', 'category__in', 'category__not_in', 'category__and', 'tag__in', 'tag__not_in', 'tag__and', 'tag_slug__in', 'tag_slug__and', 'tag_id', 'post_mime_type', 'perm', 'comments_per_page');
    4040
    4141        /**
    4242         * Extra query variables set by the user.
     
    731731        var $db_fields;
    732732
    733733        /**
     734         * Max number of pages walked by the paged walker
     735         *
     736         * @since 2.7.0
     737         * @var int
     738         * @access protected
     739         */
     740        var $max_pages = 1;
     741
     742        /**
    734743         * Starts the list before the elements are added.
    735744         *
    736745         * Additional parameters are used in child classes. The args parameter holds
     
    947956        function paged_walk( $elements, $max_depth, $page_num, $per_page ) {
    948957
    949958                /* sanity check */
    950                 if ( empty($elements) || $max_depth < 0 )
     959                if ( empty($elements) || $max_depth < -1 )
    951960                        return '';
    952961
    953962                $args = array_slice( func_get_args(), 4 );
     
    956965                $id_field = $this->db_fields['id'];
    957966                $parent_field = $this->db_fields['parent'];
    958967
     968                $count = -1;
     969                if ( -1 == $max_depth )
     970                        $total_top = count( $elements );
     971                if ( $page_num < 1 || $per_page < 0  ) {
     972                        // No paging
     973                        $paging = false;
     974                        $start = 0;
     975                        if ( -1 == $max_depth )
     976                                $end = $total_top;
     977                        $this->max_pages = 1;
     978                } else {
     979                        $paging = true;
     980                        $start = ( (int)$page_num - 1 ) * (int)$per_page;
     981                        $end   = $start + $per_page;
     982                        if ( -1 == $max_depth )
     983                                $this->max_pages = ceil($total_top / $per_page);
     984                }
     985
     986                // flat display
     987                if ( -1 == $max_depth ) {
     988                        $empty_array = array();
     989                        foreach ( $elements as $e ) {
     990                                $count++;
     991                                if ( $count < $start )
     992                                        continue;
     993                                if ( $count >= $end )
     994                                        break;
     995                                $this->display_element( $e, $empty_array, 1, 0, $args, $output );
     996                        }
     997                        return $output;
     998                }
     999
    9591000                /*
    9601001                 * seperate elements into two buckets: top level and children elements
    9611002                 * children_elements is two dimensional array, eg.
     
    9701011                                $children_elements[ $e->$parent_field ][] = $e;
    9711012                }
    9721013
    973                 $count = -1;
    9741014                $total_top = count( $top_level_elements );
    975                 if ( $page_num < 1 || $per_page < 0  ) {
    976                         $start = 0;
    977                         $end = $total_top;
    978                 } else {
    979                         $start = ( (int)$page_num - 1 ) * (int)$per_page;
    980                         $end   = $start + $per_page;
    981                 }
    9821015
     1016                if ( $paging )
     1017                        $this->max_pages = ceil($total_top / $per_page);
     1018
    9831019                foreach( $top_level_elements as $e ){
    9841020                        $count++;
    9851021
     
    9961032                        $this->display_element( $e, $children_elements, $max_depth, 0, $args, $output );
    9971033                }
    9981034
    999                 if ( $end >= $total_top && count( $children_elements ) > 0 ){
     1035                if ( $end >= $total_top && count( $children_elements ) > 0 ) {
    10001036                        $empty_array = array();
    10011037                        foreach ( $children_elements as $orphans )
    10021038                                foreach( $orphans as $op )
     
    10301066                foreach ( (array)$children_elements[$id] as $child )
    10311067                        $this->unset_children( $child, $children_elements );
    10321068
    1033                 unset( $children_elements[$id] );
     1069                if ( isset($children_elements[$id]) )
     1070                        unset( $children_elements[$id] );
    10341071
    10351072        }
    10361073}
  • wp-content/themes/default/comments.php

     
    1515        <h3 id="comments"><?php comments_number('No Responses', 'One Response', '% Responses' );?> to &#8220;<?php the_title(); ?>&#8221;</h3>
    1616
    1717        <ol class="commentlist">
    18         <?php wp_list_comments($comments); ?>
     18        <?php wp_list_comments(); ?>
    1919        </ol>
    20 
     20        <div class="navigation">
     21                <div class="alignleft"><?php previous_comments_link() ?></div>
     22                <div class="alignright"><?php next_comments_link() ?></div>
     23        </div>
    2124 <?php else : // this is displayed if there are no comments so far ?>
    2225
    2326        <?php if ('open' == $post->comment_status) : ?>
  • wp-admin/options-discussion.php

     
    4040<input name="close_comments_for_old_posts" type="checkbox" id="close_comments_for_old_posts" value="1" <?php checked('1', get_option('close_comments_for_old_posts')); ?> />
    4141<?php _e('Close comments on articles older than') ?></label> <?php printf(__('%s days'), '<input name="close_comments_days_old" type="text" id="close_comments_days_old" value="' . attribute_escape(get_option('close_comments_days_old')) . '" size="3" />') ?>
    4242<br />
     43<label for="thread_comments">
     44<input name="thread_comments" type="checkbox" id="thread_comments" value="1" <?php checked('1', get_option('thread_comments')); ?> />
     45<?php _e('Group replies into threads') ?></label> <?php printf(__('%s levels deep'), '<input name="thread_comments_depth" type="text" id="thread_comments_depth" value="' . attribute_escape(get_option('thread_comments_depth')) . '" size="3" />') ?>
     46<br />
     47<label for="page_comments">
     48<input name="page_comments" type="checkbox" id="page_comments" value="1" <?php checked('1', get_option('page_comments')); ?> />
     49<?php _e('Break comments into pages with') ?></label> <?php printf(__('%s comments per page'), '<input name="comments_per_page" type="text" id="comments_per_page" value="' . attribute_escape(get_option('comments_per_page')) . '" size="3" />') ?>
     50<br />
    4351<small><em><?php echo '(' . __('These settings may be overridden for individual articles.') . ')'; ?></em></small>
    4452</fieldset></td>
    4553</tr>
  • wp-admin/options.php

     
    2323
    2424$whitelist_options = array(
    2525        'general' => array('blogname', 'blogdescription', 'admin_email', 'users_can_register', 'gmt_offset', 'date_format', 'time_format', 'start_of_week', 'comment_registration', 'default_role' ),
    26         'discussion' => array( 'default_pingback_flag', 'default_ping_status', 'default_comment_status', 'comments_notify', 'moderation_notify', 'comment_moderation', 'require_name_email', 'comment_whitelist', 'comment_max_links', 'moderation_keys', 'blacklist_keys', 'show_avatars', 'avatar_rating', 'close_comments_for_old_posts', 'close_comments_days_old' ),
     26        'discussion' => array( 'default_pingback_flag', 'default_ping_status', 'default_comment_status', 'comments_notify', 'moderation_notify', 'comment_moderation', 'require_name_email', 'comment_whitelist', 'comment_max_links', 'moderation_keys', 'blacklist_keys', 'show_avatars', 'avatar_rating', 'close_comments_for_old_posts', 'close_comments_days_old', 'thread_comments', 'thread_comments_depth', 'page_comments', 'comments_per_page' ),
    2727        'misc' => array( 'hack_file', 'use_linksupdate', 'uploads_use_yearmonth_folders', 'upload_path' ),
    2828        'media' => array( 'thumbnail_size_w', 'thumbnail_size_h', 'thumbnail_crop', 'medium_size_w', 'medium_size_h', 'large_size_w', 'large_size_h', 'image_default_size', 'image_default_align', 'image_default_link_type' ),
    2929        'privacy' => array( 'blog_public' ),