paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-03 05:23:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

compose: Don't allow same message to be sent twice.

Browse Source 
It is possible to send the message twice by clicking send button
very quickly twice or by pressing enter and send button simultaneously.

This can be easily reproduced for large messages sent in a stream
narrow. Hard to reproduce for small messages or in PM narrows.

I was not able to reproduce locally, but was able to reproduce on
chat.zulip.org. So, this is an untested bug fix.

Fixes #22562
This commit is contained in:
Aman Agrawal
2022-07-28 07:31:57 +00:00
committed by Tim Abbott
parent 5fd4d3f053
commit a13058223d
3 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
frontend_tests/node_tests/compose.js
View File

@@ -43,6 +43,7 @@ const server_events = mock_esm("../../static/js/server_events");
const transmit = mock_esm("../../static/js/transmit");
const upload = mock_esm("../../static/js/upload");
const compose_ui = zrequire("compose_ui");
const compose_closed_ui = zrequire("compose_closed_ui");
const compose_state = zrequire("compose_state");
const compose = zrequire("compose");
@@ -361,6 +362,7 @@ test_ui("finish", ({override, override_rewire, mock_template}) => {
$("#compose-send-button .loader").hide();
$("#compose-textarea").off("select");
$("#compose-textarea").val("");
compose_ui.compose_spinner_visible = false;
const res = compose.finish();
assert.equal(res, false);
assert.ok(!$("#compose_banners .recipient_not_subscribed").visible());
@@ -374,6 +376,7 @@ test_ui("finish", ({override, override_rewire, mock_template}) => {
$("#compose .preview_message_area").show();
$("#compose .markdown_preview").hide();
$("#compose-textarea").val("foobarfoobar");
compose_ui.compose_spinner_visible = false;
compose_state.set_message_type("private");
override(compose_pm_pill, "get_emails", () => "bob@example.com");
override(compose_pm_pill, "get_user_ids", () => []);
@@ -399,6 +402,7 @@ test_ui("finish", ({override, override_rewire, mock_template}) => {
$("#compose .preview_message_area").show();
$("#compose .markdown_preview").hide();
$("#compose-textarea").val("foobarfoobar");
compose_ui.compose_spinner_visible = false;
compose_state.set_message_type("stream");
compose_state.set_stream_name("social");
override_rewire(people, "get_by_user_id", () => []);