From 0b7faedb8df6580b1926ff153456edbbdd08fd0a Mon Sep 17 00:00:00 2001 From: Robert Jakubowicz Date: Mon, 25 Sep 2017 14:39:27 -0400 Subject: [PATCH] Fix: check attachments is defined before accessing first element --- .../rocketchat-lib/server/lib/sendNotificationsOnMessage.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/rocketchat-lib/server/lib/sendNotificationsOnMessage.js b/packages/rocketchat-lib/server/lib/sendNotificationsOnMessage.js index 1083d18d76e..ddda8b880de 100644 --- a/packages/rocketchat-lib/server/lib/sendNotificationsOnMessage.js +++ b/packages/rocketchat-lib/server/lib/sendNotificationsOnMessage.js @@ -31,7 +31,7 @@ function parseMessageText(message, userId) { const user = RocketChat.models.Users.findOneById(userId); const lng = user && user.language || RocketChat.settings.get('language') || 'en'; - if (!message.msg && message.attachments[0]) { + if (!message.msg && message.attachments && message.attachments[0]) { message.msg = message.attachments[0].image_type ? TAPi18n.__('User_uploaded_image', {lng}) : TAPi18n.__('User_uploaded_file', {lng}); } message.msg = RocketChat.callbacks.run('beforeNotifyUser', message.msg);