Ignore case of OTP keywords when deciding whether to hide them

This commit is contained in:
Danny Moesch 2018-12-02 23:07:41 +01:00 committed by Bob Sun
parent cfb50276bc
commit 3cc7510b29

View file

@ -108,11 +108,12 @@ public class Password {
public func getFilteredAdditions() -> [AdditionField] { public func getFilteredAdditions() -> [AdditionField] {
return additions.filter { field in return additions.filter { field in
field.title.lowercased() != Constants.USERNAME_KEYWORD let title = field.title.lowercased()
&& field.title.lowercased() != Constants.LOGIN_KEYWORD return title != Constants.USERNAME_KEYWORD
&& field.title.lowercased() != Constants.PASSWORD_KEYWORD && title != Constants.LOGIN_KEYWORD
&& (!field.title.hasPrefix(Constants.UNKNOWN) || !SharedDefaults[.isHideUnknownOn]) && title != Constants.PASSWORD_KEYWORD
&& (!Constants.OTP_KEYWORDS.contains(field.title) || !SharedDefaults[.isHideOTPOn]) && (!field.title.hasPrefix(Constants.UNKNOWN) || !SharedDefaults[.isHideUnknownOn])
&& (!Constants.OTP_KEYWORDS.contains(title) || !SharedDefaults[.isHideOTPOn])
} }
} }