Clicking message Reply button multiple times produces duplicate messages

[jdgrimes]

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:

1. Send a test message to yourself or somebody else.
2. Enter a reply message.
3. 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).

[jdgrimes]

I have tested 7718.diff​, and it fixes the issue.

[r-a-y]

In 11911:

bp-legacy: Disable "Send Reply" button onclick when replying to a private message thread via AJAX.

Previously, it was possible for a user to click on the "Send Reply" button
multiple times, causing the same reply to be duplicated several times over.

This commit addreses this by disabling the "Send Reply" button when it is
clicked and re-enables it after the AJAX request is completed.

Props jdgrimes.

Fixes #7718.

[r-a-y]

Thanks for the report and the patch, @jdgrimes.

I've committed the fix for v3.0 since this is a rather (embarassingly) old bug and not a regression from v2.9.