iamtorsten / libfintx

Moved to https://github.com/libfintx/libfintx - please use the new repository!
https://github.com/libfintx/libfintx
GNU Lesser General Public License v3.0
115 stars 40 forks source link

Umsätze abholen ergibt Fehler ! #112

Closed SongPing closed 1 year ago

SongPing commented 1 year ago

Nachdem ich libfintx upgedated habe, funktioniert das Abholen der Umsätze nicht mehr. Der Codestand von 3.8.2021 und 8.10.2021 funktionierte einwandfrei. Aber bei so ziemlich jedem neueren Codestand funktioniert das Abholen der Umsätze nicht! Z.B. beim aktuellen Codestand von 13.3.2023 funktioniert zwar die Synchronisation und die Kontoabfrage, aber nicht mehr das Abholen der Umsätze.

Code: 3060 | Typ: Warning | Nachricht: Bitte beachten Sie die enthaltenen Warnungen/Hinweise.
Code: 0020 | Typ: Success | Nachricht: Auftrag ausgeführt.
Code: 3050 | Typ: Warning | Nachricht: BPD nicht mehr aktuell, aktuelle Version enthalten.
Code: 3920 | Typ: Warning | Nachricht: Zugelassene TAN-Verfahren für den Benutzer:962:972:982
Code: 0901 | Typ: Success | Nachricht: *PIN gültig.
Code: 0020 | Typ: Success | Nachricht: *Dialoginitialisierung erfolgreich
Code: 9050 | Typ: Error | Nachricht: Die Nachricht enthält Fehler.
Code: 9800 | Typ: Error | Nachricht: Dialog abgebrochen
Code: 9110 | Typ: Error | Nachricht: Ungültige Auftragsnachricht: Unbekannter Aufbau.
Code: 9010 | Typ: Error | Nachricht: Auftrag wegen genereller Fehler in Auftragsnachricht nicht verarbeitet.
SongPing commented 1 year ago

Der Fehler hat sich im Commit "Update Segmentbuilder" vom 27.11.2021 in Segments/HKKAZ.cs eingeschlichen. Interessanterweise ist die funktionierende Version noch vorhanden, nur ist sie auskommentiert,

...
StringBuilder sb = new StringBuilder();
sb.Append(activeAccount.AccountNumber);
sb.Append(DEG.Separator);
sb.Append(activeAccount.SubAccountFeature);
sb.Append(DEG.Separator);
sb.Append(SEG_COUNTRY.Germany);
sb.Append(DEG.Separator);
sb.Append(activeAccount.AccountBankCode);
sb.Append(sEG.Delimiter);
sb.Append(DEG.DeAdd);
sb.Append(sEG.Delimiter);
sb.Append(FromDate);
sb.Append(sEG.Delimiter);
sb.Append(ToDate);
sb.Append(sEG.Terminator);
string rawData = sb.ToString();
segments = sEG.toSEG(new SEG_DATA { Header = "HKKAZ", Num = client.SEGNUM,
    Version = client.HIKAZS, RefNum = 0, RawData = rawData });

//Remove the comment, so that segments is overriden => work again!
segments = "HKKAZ:" + client.SEGNUM + ":" + client.HIKAZS + "+" + activeAccount.AccountIban + ":" + activeAccount.AccountBic + ":" + activeAccount.AccountNumber + "::280:" + activeAccount.AccountBankCode + "+N+" + FromDate + "+" + ToDate + "'";
...
iamtorsten commented 1 year ago

@SongPing can you please retest.

SongPing commented 1 year ago

@iamtorsten yes, it works now. Thanks, for the bugfix!

iamtorsten commented 1 year ago

Welcome.