codinux
/
fints4k
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Implemented logging only if it's really an exception and not a known wrong implementation of FinTS standard

This commit is contained in:
dankito 2020-09-18 18:45:23 +02:00
parent 347b3e59e7
commit 75399e0664
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
fints4k/src/commonMain/kotlin/net/dankito/banking/fints/response/ResponseParser.kt
View File

@ -456,8 +456,10 @@ open class ResponseParser(
try { try {
return parseCodeEnum(smsAbbuchungskontoErforderlichString, SmsAbbuchungskontoErforderlich.values()) return parseCodeEnum(smsAbbuchungskontoErforderlichString, SmsAbbuchungskontoErforderlich.values())
} catch (e: Exception) { } catch (e: Exception) {
if (isEncodedBooleanValue(smsAbbuchungskontoErforderlichString) == false) {
log.error(e) { "Could not parse '$smsAbbuchungskontoErforderlichString' to SmsAbbuchungskontoErforderlich" } log.error(e) { "Could not parse '$smsAbbuchungskontoErforderlichString' to SmsAbbuchungskontoErforderlich" }
} }
}
// Bankhaus Neelmeyer and Oldenburgische Landesbank encode SmsAbbuchungskontoErforderlich with boolean values (with is wrong according to FinTS standard) // Bankhaus Neelmeyer and Oldenburgische Landesbank encode SmsAbbuchungskontoErforderlich with boolean values (with is wrong according to FinTS standard)
return tryToParseEnumAsBoolean(smsAbbuchungskontoErforderlichString, return tryToParseEnumAsBoolean(smsAbbuchungskontoErforderlichString,
@ -469,8 +471,10 @@ open class ResponseParser(
try { try {
return parseCodeEnum(auftraggeberkontoErforderlichString, AuftraggeberkontoErforderlich.values()) return parseCodeEnum(auftraggeberkontoErforderlichString, AuftraggeberkontoErforderlich.values())
} catch (e: Exception) { } catch (e: Exception) {
if (isEncodedBooleanValue(auftraggeberkontoErforderlichString) == false) {
log.error(e) { "Could not parse '$auftraggeberkontoErforderlichString' to AuftraggeberkontoErforderlich" } log.error(e) { "Could not parse '$auftraggeberkontoErforderlichString' to AuftraggeberkontoErforderlich" }
} }
}
// Bankhaus Neelmeyer and Oldenburgische Landesbank encode AuftraggeberkontoErforderlich with boolean values (with is wrong according to FinTS standard) // Bankhaus Neelmeyer and Oldenburgische Landesbank encode AuftraggeberkontoErforderlich with boolean values (with is wrong according to FinTS standard)
return tryToParseEnumAsBoolean(auftraggeberkontoErforderlichString, return tryToParseEnumAsBoolean(auftraggeberkontoErforderlichString,
@ -478,6 +482,10 @@ open class ResponseParser(
AuftraggeberkontoErforderlich.AuftraggeberkontoDarfNichtAngegebenWerden) AuftraggeberkontoErforderlich.AuftraggeberkontoDarfNichtAngegebenWerden)
} }
protected open fun isEncodedBooleanValue(value: String): Boolean {
return value == "N" || value == "J"
}
protected open fun <T : Enum<T>> tryToParseEnumAsBoolean(enumString: String, valueForTrue: T, valueForFalse: T): T { protected open fun <T : Enum<T>> tryToParseEnumAsBoolean(enumString: String, valueForTrue: T, valueForFalse: T): T {
try { try {
val bool = parseBoolean(enumString) val bool = parseBoolean(enumString)