[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