ParceVcf fix on failed vCard

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

1 files changed, 6 insertions, 4 deletions

diff --git a/src/services/parseVcf.js b/src/services/parseVcf.js
index e74b083f..65d5f95c 100644
--- a/src/services/parseVcf.js
+++ b/src/services/parseVcf.js
@@ -35,17 +35,19 @@ export default function parseVcf(data = '', addressbook) {
 
 	importState.total = vCards.length
 
-	return vCards.map(vCard => {
+	// Not using map because we want to only push valid contacts
+	// map force to return at least undefined
+	return vCards.reduce((contacts, vCard) => {
 		try {
 			// console.log(vCards.indexOf(vCard))
 			let contact = new Contact(vCard, addressbook)
 			importState.accepted++
-			return contact
+			contacts.push(contact)
 		} catch (e) {
 			// Parse error! Do not stop here...
 			importState.denied++
-			// eslint-disable-next-line
 			console.error(e)
 		}
-	})
+		return contacts
+	}, [])
 }