From 3cc7510b294809924fcbcc7fb552f69d42afb5b9 Mon Sep 17 00:00:00 2001 From: Danny Moesch Date: Sun, 2 Dec 2018 23:07:41 +0100 Subject: [PATCH] Ignore case of OTP keywords when deciding whether to hide them --- passKit/Models/Password.swift | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/passKit/Models/Password.swift b/passKit/Models/Password.swift index a4b0527..e885978 100644 --- a/passKit/Models/Password.swift +++ b/passKit/Models/Password.swift @@ -108,11 +108,12 @@ public class Password { public func getFilteredAdditions() -> [AdditionField] { return additions.filter { field in - field.title.lowercased() != Constants.USERNAME_KEYWORD - && field.title.lowercased() != Constants.LOGIN_KEYWORD - && field.title.lowercased() != Constants.PASSWORD_KEYWORD - && (!field.title.hasPrefix(Constants.UNKNOWN) || !SharedDefaults[.isHideUnknownOn]) - && (!Constants.OTP_KEYWORDS.contains(field.title) || !SharedDefaults[.isHideOTPOn]) + let title = field.title.lowercased() + return title != Constants.USERNAME_KEYWORD + && title != Constants.LOGIN_KEYWORD + && title != Constants.PASSWORD_KEYWORD + && (!field.title.hasPrefix(Constants.UNKNOWN) || !SharedDefaults[.isHideUnknownOn]) + && (!Constants.OTP_KEYWORDS.contains(title) || !SharedDefaults[.isHideOTPOn]) } }