public/mailtrain
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fixes #278

Browse source 
Although the bounced counter is correctly increased, the status of
failed messages has been set to 'unsubscribed' instead of 'bounced‘.
This commit is contained in:
witzig 2017-07-14 23:48:18 +02:00
parent b57615543b
commit 4cede37e9d
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
services/sender.js
View file

@ -523,7 +523,7 @@ let sendLoop = () => {
} }
} }
let status = err ? 2 : 1; let status = err ? 3 : 1;
let response = err && (err.response || err.message) || info.response || info.messageId; let response = err && (err.response || err.message) || info.response || info.messageId;
let responseId = response.split(/\s+/).pop(); let responseId = response.split(/\s+/).pop();
@ -533,7 +533,7 @@ let sendLoop = () => {
return; return;
} }
let query = 'UPDATE `campaigns` SET `delivered`=`delivered`+1 ' + (status === 2 ? ', `bounced`=`bounced`+1 ' : '') + ' WHERE id=? LIMIT 1'; let query = 'UPDATE `campaigns` SET `delivered`=`delivered`+1 ' + (status === 3 ? ', `bounced`=`bounced`+1 ' : '') + ' WHERE id=? LIMIT 1';
connection.query(query, [message.campaignId], err => { connection.query(query, [message.campaignId], err => {
if (err) { if (err) {