Fix string detection, put delete in last and fix properties scoping

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

1 files changed, 15 insertions, 6 deletions

diff --git a/src/services/checks/missingFN.js b/src/services/checks/missingFN.js
index 449f4a43..86a92c7a 100644
--- a/src/services/checks/missingFN.js
+++ b/src/services/checks/missingFN.js
@@ -30,8 +30,9 @@ export default {
 		return !contact.vCard.hasProperty('fn')						// No FN
 			|| contact.vCard.getFirstPropertyValue('fn') === ''		// Empty FN
 			|| ( // we don't want to fix newly created contacts
-				contact.dav																		// Existing contact
-				&& contact.vCard.getFirstPropertyValue('fn') === t('contacts', 'New contact')	// AND Unchanged FN
+				contact.dav															// Existing contact
+				&& contact.vCard.getFirstPropertyValue('fn')
+					.toLowerCase() === t('contacts', 'New contact').toLowerCase()	// AND Unchanged FN
 			)
 	},
 	fix: contact => {
@@ -39,14 +40,22 @@ export default {
 			// Stevenson;John;Philip,Paul;Dr.;Jr.,M.D.,A.C.P.
 			// -> John Stevenson
 			const n = contact.vCard.getFirstPropertyValue('n')
-			contact.fullName = n.slice(0, 2).reverse().join(' ')
-			return true
+			const fullName = n.slice(0, 2).reverse().join(' ')
+			if (fullName.trim() !== '') {
+				contact.fullName = fullName
+				return true
+			}
+			return false
 		} else if (contact.vCard.hasProperty('org')) {
 			const org = contact.vCard.getFirstPropertyValue('org')
 			// ABC, Inc.;North American Division;Marketing
 			// -> ABC, Inc.
-			contact.fullName = org[0]
-			return true
+			const fullName = org[0]
+			if (fullName.trim() !== '') {
+				contact.fullName = fullName
+				return true
+			}
+			return false
 		}
 		return false
 	}