#7718 closed defect (bug) (fixed)
Clicking message Reply button multiple times produces duplicate messages
Reported by: | jdgrimes | Owned by: | r-a-y |
---|---|---|---|
Milestone: | 3.0 | Priority: | normal |
Severity: | normal | Version: | 1.7 |
Component: | Messages | Keywords: | has-patch |
Cc: |
Description
I originally started investigating this due to a complaint from a client that users were posting complaining about duplicate messages being posted. I was able to reproduce this:
- Send a test message to yourself or somebody else.
- Enter a reply message.
- Hit the Send Reply button multiple times in quick succession.
Result: your reply will be sent multiple times. Tested on both latest stable and trunk, using 2015 and 2017 themes.
Suggested fix: the Reply button should be disabled while the Ajax request is being sent, to avoid the event callback being triggered multiple times.
Patch forthcoming.
Related: #7457. However, that ticket is about messages specifically, and not replies (which I haven't been able to reproduce).
I have tested 7718.diff, and it fixes the issue.