From 68eb5e5483fde15519add114c0f3f2c06ed12f0c Mon Sep 17 00:00:00 2001 From: Laurynas Karvelis Date: Mon, 24 Dec 2018 13:56:33 +0200 Subject: [PATCH] fix: default param value to true to avoid breaking people's existing code; small return value fix --- src/types/Base.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/types/Base.js b/src/types/Base.js index 6e62dd6..1f3b80e 100644 --- a/src/types/Base.js +++ b/src/types/Base.js @@ -28,7 +28,7 @@ export default class Base extends EventEmitter { * @param {boolean} expectAnswer whether a sent message expects an answer from a contact(s) * @return {Promise} returns a promise, resolved with message:answer */ - send(bot, expectAnswer = false) { + send(bot, expectAnswer = true) { if (this._keyboard) { const replyMarkup = JSON.stringify(this._keyboard.getProperties()); this.properties.reply_markup = replyMarkup; @@ -43,7 +43,8 @@ export default class Base extends EventEmitter { .then(messageId => { if (!expectAnswer) { // no need to add more event callbacks for the messages that don't need expect an answer - return resolve(); + resolve(); + return; } const chat = this.properties.chat_id; @@ -73,8 +74,6 @@ export default class Base extends EventEmitter { bot.removeListener('update', listener); } }); - - return void 0; }) .catch(reject) .finally(() => {