WordPress.org

Make WordPress Core

Ticket #15530: 15530.7.diff

File 15530.7.diff, 7.2 KB (added by nacin, 7 years ago)
  • wp-admin/admin-ajax.php

     
    189189 * @param int $comment_id
    190190 * @return die
    191191 */
    192 function _wp_ajax_delete_comment_response( $comment_id ) {
     192function _wp_ajax_delete_comment_response( $comment_id, $delta = -1 ) {
    193193        $total = (int) @$_POST['_total'];
    194194        $per_page = (int) @$_POST['_per_page'];
    195195        $page = (int) @$_POST['_page'];
     
    198198        if ( !$total || !$per_page || !$page || !$url )
    199199                die( (string) time() );
    200200
    201         if ( --$total < 0 ) // Take the total from POST and decrement it (since we just deleted one)
     201        $total += $delta;
     202        if ( $total < 0 )
    202203                $total = 0;
    203204
    204         if ( 0 != $total % $per_page && 1 != mt_rand( 1, $per_page ) ) // Only do the expensive stuff on a page-break, and about 1 other time per page
    205                 die( (string) time() );
     205        // Only do the expensive stuff on a page-break, and about 1 other time per page
     206        if ( 0 == $total % $per_page || 1 == mt_rand( 1, $per_page ) ) {
     207                $post_id = 0;
     208                $status = 'total_comments'; // What type of comment count are we looking for?
     209                $parsed = parse_url( $url );
     210                if ( isset( $parsed['query'] ) ) {
     211                        parse_str( $parsed['query'], $query_vars );
     212                        if ( !empty( $query_vars['comment_status'] ) )
     213                                $status = $query_vars['comment_status'];
     214                        if ( !empty( $query_vars['p'] ) )
     215                                $post_id = (int) $query_vars['p'];
     216                }
    206217
    207         $post_id = 0;
    208         $status = 'total_comments'; // What type of comment count are we looking for?
    209         $parsed = parse_url( $url );
    210         if ( isset( $parsed['query'] ) ) {
    211                 parse_str( $parsed['query'], $query_vars );
    212                 if ( !empty( $query_vars['comment_status'] ) )
    213                         $status = $query_vars['comment_status'];
    214                 if ( !empty( $query_vars['p'] ) )
    215                         $post_id = (int) $query_vars['p'];
     218                $comment_count = wp_count_comments($post_id);
     219
     220                if ( isset( $comment_count->$status ) ) // We're looking for a known type of comment count
     221                        $total = $comment_count->$status;
     222                        // else use the decremented value from above
    216223        }
    217224
    218         $comment_count = wp_count_comments($post_id);
    219225        $time = time(); // The time since the last comment count
    220226
    221         if ( isset( $comment_count->$status ) ) // We're looking for a known type of comment count
    222                 $total = $comment_count->$status;
    223         // else use the decremented value from above
    224 
    225         $page_links = paginate_links( array(
    226                 'base' => add_query_arg( 'apage', '%#%', $url ),
    227                 'format' => '',
    228                 'prev_text' => __('&laquo;'),
    229                 'next_text' => __('&raquo;'),
    230                 'total' => ceil($total / $per_page),
    231                 'current' => $page
    232         ) );
    233227        $x = new WP_Ajax_Response( array(
    234228                'what' => 'comment',
    235229                'id' => $comment_id, // here for completeness - not used
    236230                'supplemental' => array(
    237                         'pageLinks' => $page_links,
     231                        'total_items_i18n' => sprintf( _n( '1 item', '%s items', $total ), number_format_i18n( $total ) ),
     232                        'total_pages' => ceil( $total / $per_page ),
     233                        'total_pages_i18n' => number_format_i18n( ceil( $total / $per_page ) ),
    238234                        'total' => $total,
    239235                        'time' => $time
    240236                )
     
    331327        check_ajax_referer( "delete-comment_$id" );
    332328        $status = wp_get_comment_status( $comment->comment_ID );
    333329
     330        $delta = -1;
    334331        if ( isset($_POST['trash']) && 1 == $_POST['trash'] ) {
    335332                if ( 'trash' == $status )
    336333                        die( (string) time() );
     
    339336                if ( 'trash' != $status )
    340337                        die( (string) time() );
    341338                $r = wp_untrash_comment( $comment->comment_ID );
     339                if ( ! isset( $_POST['comment_status'] ) || $_POST['comment_status'] != 'trash' ) // undo trash, not in trash
     340                        $delta = 1;
    342341        } elseif ( isset($_POST['spam']) && 1 == $_POST['spam'] ) {
    343342                if ( 'spam' == $status )
    344343                        die( (string) time() );
     
    347346                if ( 'spam' != $status )
    348347                        die( (string) time() );
    349348                $r = wp_unspam_comment( $comment->comment_ID );
     349                if ( ! isset( $_POST['comment_status'] ) || $_POST['comment_status'] != 'spam' ) // undo spam, not in spam
     350                        $delta = 1;
    350351        } elseif ( isset($_POST['delete']) && 1 == $_POST['delete'] ) {
    351352                $r = wp_delete_comment( $comment->comment_ID );
    352353        } else {
     
    354355        }
    355356
    356357        if ( $r ) // Decide if we need to send back '1' or a more complicated response including page links and comment counts
    357                 _wp_ajax_delete_comment_response( $comment->comment_ID );
     358                _wp_ajax_delete_comment_response( $comment->comment_ID, $delta );
    358359        die( '0' );
    359360        break;
    360361case 'delete-tag' :
  • wp-admin/includes/class-wp-comments-list-table.php

     
    4848                        $comment_status = 'all';
    4949
    5050                $comment_type = !empty( $_REQUEST['comment_type'] ) ? $_REQUEST['comment_type'] : '';
     51                error_log( var_export( $comment_type, true ) );
    5152
    5253                $search = ( isset( $_REQUEST['s'] ) ) ? $_REQUEST['s'] : '';
    5354
  • wp-admin/js/edit-comments.dev.js

     
    44setCommentsList = function() {
    55        var totalInput, perPageInput, pageInput, lastConfidentTime = 0, dimAfter, delBefore, updateTotalCount, delAfter;
    66
    7         totalInput = $('.tablenav input[name="_total"]', '#comments-form');
    8         perPageInput = $('.tablenav input[name="_per_page"]', '#comments-form');
    9         pageInput = $('.tablenav input[name="_page"]', '#comments-form');
     7        totalInput = $('input[name="_total"]', '#comments-form');
     8        perPageInput = $('input[name="_per_page"]', '#comments-form');
     9        pageInput = $('input[name="_page"]', '#comments-form');
    1010
    1111        dimAfter = function( r, settings ) {
    1212                var c = $('#' + settings.element);
     
    3838                settings.data._per_page = perPageInput.val() || 0;
    3939                settings.data._page = pageInput.val() || 0;
    4040                settings.data._url = document.location.href;
     41                settings.data.comment_status = $('input[name=comment_status]', '#comments-form').val();
    4142
    4243                if ( cl.indexOf(':trash=1') != -1 )
    4344                        action = 'trash';
     
    192193                                total = 0;
    193194
    194195                        if ( ( 'object' == typeof r ) && lastConfidentTime < settings.parsed.responses[0].supplemental.time ) {
    195                                 pageLinks = settings.parsed.responses[0].supplemental.pageLinks || '';
    196                                 if ( $.trim( pageLinks ) )
    197                                         $('.tablenav-pages').find( '.page-numbers' ).remove().end().append( $( pageLinks ) );
    198                                 else
    199                                         $('.tablenav-pages').find( '.page-numbers' ).remove();
    200 
     196                                total_items_i18n = settings.parsed.responses[0].supplemental.total_items_i18n || '';
     197                                if ( total_items_i18n ) {
     198                                        $('.displaying-num').text( total_items_i18n );
     199                                        $('.total-pages').text( settings.parsed.responses[0].supplemental.total_pages_i18n );
     200                                        $('.tablenav-pages').find('.next-page, .last-page').toggleClass('disabled', settings.parsed.responses[0].supplemental.total_pages == $('.current-page').val());
     201                                }
    201202                                updateTotalCount( total, settings.parsed.responses[0].supplemental.time, true );
    202203                        } else {
    203204                                updateTotalCount( total, r, false );
     
    236237                args.no_placeholder = true;
    237238
    238239                args.paged ++;
    239                
     240
     241                // $.query.get() needs some correction to be sent into an ajax request
     242                if ( true === args.comment_type )
     243                        args.comment_type = '';
     244
    240245                args = $.extend(args, {
    241246                        'action': 'fetch-list',
    242247                        'list_args': list_args,