compare both email and name before skipping vcard import

[johndoh]

for #9592

compare both the email address and the name of a contact (when available) before declaring it a duplicate

[pabzm]

Thanks for the contribution! But is #9513 really the correct reference? I don't understand how this related to that.

[johndoh]

sorry. I have corrected the reference.

[pabzm]

Looks good to me, thanks! 👍

(The CI errors are not your concern.)

[pabzm]

@alecpl FYI: I'll merge this in a few days unless you object.

[alecpl]

I don't have a strong opinion, you can merge. Maybe something like "import/merge strategy" feature, where user would pick how to handle potential duplicates (before or after upload), would solve this better.

[pabzm]

Maybe something like "import/merge strategy" feature, where user would pick how to handle potential duplicates (before or after upload), would solve this better.

@alecpl I agree that that would be another improvement, but I think this current change is one step forward already and thus merge it now. Thank you for the feedback!