[wp-trac] Re: [WordPress Trac] #7435: Reply to comments from admin
WordPress Trac
wp-trac at lists.automattic.com
Fri Sep 5 22:58:49 GMT 2008
#7435: Reply to comments from admin
----------------------------+-----------------------------------------------
Reporter: ryan | Owner: azaozz
Type: enhancement | Status: new
Priority: normal | Milestone: 2.7
Component: Administration | Version:
Severity: normal | Resolution:
Keywords: blessed |
----------------------------+-----------------------------------------------
Comment (by caesarsgrunt):
There's a bug which causes the delete function to be called repeatedly for
any already existing comment after one has been added using this ajax form
and the page hasn't been reloaded.
I haven't managed to find the cause of this bug yet, but I'd guess it must
be in the code to do with the reply feature rather than the delete code.
I described this bug on [http://trac.wordpress.org/ticket/4529#comment:8
#4529], but I have found the following extra details since :[[BR]]
It's ''not'' just the second comment. After a comment or comments have
been added using the ajax 'reply' form, any time an ''already existing''
comment is deleted the delete function is called ''the number of comments
added + 1'' times. So if three comments are added, any already existing
comments which are deleted will cause the delete comment handler to be
called four times with the same comment ID.
Whilst this bug doesn't currently matter except for causing the deleted
comment to flash repeatedly if enough comments have been added, if my
patch on #4529 is committed the bug will cause it to malfunction. Whilst I
can work around it, it makes more sense to fix it.
--
Ticket URL: <http://trac.wordpress.org/ticket/7435#comment:24>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list