Remove the `messageSend` event

CHANGES.md

@@ -33,6 +33,7 @@ Removed events:
 * `chatBoxInsertedIntoDOM`
 * `bookmarkViewsInitialized`
 * `rosterGroupsFetched`
+* `messageSend` (use `sendMessage` instead)
 
 The `chatBoxMaximized` and `chatBoxMinimized` events now have the `model` as
 payload and not the `view` since it might not be exist at that time.

spec/chatbox.js

@@ -940,8 +940,7 @@ describe("Chatboxes", function () {
             });
             expect(window.confirm).toHaveBeenCalledWith('Are you sure you want to clear the messages from this conversation?');
             await u.waitUntil(() => view.model.messages.length === 0);
-            expect(_converse.api.trigger.calls.count(), 1);
-            expect(_converse.api.trigger.calls.mostRecent().args, ['messageSend', message]);
+            await u.waitUntil(() => !view.querySelectorAll('.chat-msg__body').length);
             done();
         }));
     });

spec/messages.js

@@ -936,7 +936,6 @@ describe("A Chat Message", function () {
             await mock.sendMessage(view, message);
             expect(view.model.sendMessage).toHaveBeenCalled();
             expect(view.model.messages.length, 2);
-            expect(_converse.api.trigger.calls.mostRecent().args, ['messageSend', message]);
             expect(sizzle('.chat-content .chat-msg:last .chat-msg__text', view).pop().textContent).toEqual(message);
             done();
         }));

spec/muc.js

@@ -1903,7 +1903,7 @@ describe("Groupchats", function () {
             });
             await u.waitUntil(() => view.querySelectorAll('.chat-msg__text').length);
 
-            expect(_converse.api.trigger).toHaveBeenCalledWith('messageSend', jasmine.any(_converse.Message));
+            expect(_converse.api.trigger).toHaveBeenCalledWith('sendMessage', jasmine.any(Object));
             expect(view.querySelectorAll('.chat-msg').length).toBe(1);
 
             // Let's check that if we receive the same message again, it's

src/plugins/chatview/bottom-panel.js

@@ -186,15 +186,6 @@ export default class ChatBottomPanel extends ElementView {
             textarea.style.height = 'auto';
             this.updateCharCounter(textarea.value);
         }
-        if (message) {
-            /**
-             * Triggered whenever a message is sent by the user
-             * @event _converse#messageSend
-             * @type { _converse.Message }
-             * @example _converse.api.listen.on('messageSend', message => { ... });
-             */
-            api.trigger('messageSend', message);
-        }
         if (api.settings.get('view_mode') === 'overlayed') {
             // XXX: Chrome flexbug workaround. The .chat-content area
             // doesn't resize when the textarea is resized to its original size.