Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2025-01-27 02:19:46 +08:00
Code Issues Projects Releases Packages Wiki Activity

[client-sync] Fix contact ranking error

Browse source 
Summary:
This was showing up in Sentry. Check to see if the recipient has an email
before calling toLowerCase.

Test Plan: yolo

Reviewers: halla, juan

Reviewed By: juan

Differential Revision: https://phab.nylas.com/D4423
This commit is contained in:
Mark Hahnenberg 2017-04-13 16:06:48 -07:00
parent 91f5ca4de9
commit b1123433d0
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
packages/client-sync/src/local-api/routes/contacts.js
View file

@ -16,6 +16,9 @@ const calculateContactScores = (messages, result) => {
for (const message of messages) { for (const message of messages) {
const weight = getMessageWeight(message, now); const weight = getMessageWeight(message, now);
for (const recipient of message.getRecipients()) { for (const recipient of message.getRecipients()) {
if (!recipient.email) {
continue;
}
const email = recipient.email.toLowerCase(); const email = recipient.email.toLowerCase();
result[email] = result[email] ? (result[email] + weight) : weight; result[email] = result[email] ? (result[email] + weight) : weight;
} }