From 7e0bd82bd39b3c11355158481eafdacf88635877 Mon Sep 17 00:00:00 2001 From: Scott Nonnenberg Date: Fri, 4 Aug 2017 11:50:49 -0700 Subject: [PATCH] handleDataMessage: Remove if(confirm) checks, add try/catch FREEBIE --- js/models/messages.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/js/models/messages.js b/js/models/messages.js index 386dc1f98..c8558f3b2 100644 --- a/js/models/messages.js +++ b/js/models/messages.js @@ -510,22 +510,22 @@ console.log('done with handleDataMessage', message.idForLogging()); - if (confirm) { - confirm(); - } + confirm(); return resolve(); } catch (e) { handleError(e); } }, function(error) { - console.log('handleDataMessage: Message', message.idForLogging(), 'was deleted'); + try { + console.log('handleDataMessage: Message', message.idForLogging(), 'was deleted'); - if (confirm) { confirm(); + return resolve(); + } + catch (e) { + handleError(e); } - - return resolve(); }); }, handleError); }, handleError);