[wp-trac] [WordPress Trac] #15500: Ajax comment delete display bug

WordPress Trac wp-trac at lists.automattic.com
Mon Dec 13 15:49:50 UTC 2010


#15500: Ajax comment delete display bug
--------------------------+-------------------------------------------------
 Reporter:  hew           |       Owner:  scribu                 
     Type:  defect (bug)  |      Status:  assigned               
 Priority:  high          |   Milestone:  3.1                    
Component:  Comments      |     Version:  3.1                    
 Severity:  major         |    Keywords:  has-patch needs-testing
--------------------------+-------------------------------------------------

Comment(by garyc40):

 Thanks! That's just a way to check whether refillTheExtraList() was called
 when an event trigger. Is there a better way to do that?

 By the way, I noticed that some variables were not set correctly at the
 beginning of the file edit-comments.dev.js (line 7, 8, 9):

 {{{
 totalInput = $('.tablenav input[name="_total"]', '#comments-form');
 perPageInput = $('.tablenav input[name="_per_page"]', '#comments-form');
 pageInput = $('.tablenav input[name="_page"]', '#comments-form');
 }}}

 These all yield empty arrays. Removing the ".tablenav" part of the query
 would return the correct element. I tried that, but there would be weird
 behaviors. Did some debugging and it seemed like total pages count,
 pagination links and comment count are not properly updated when comments
 are being deleted (see #15530 as well).

 If you try removing .tablenav, and have 61 comments. Number of comments
 per page is 20, number of pages is 4. If you trash one comment, then
 suddenly the existing pagination link elements are appended with a list of
 [1] [2] [3] pagination links.

 I don't know if anyone notices this, and if a fix is planned for this as
 well?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/15500#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list