$bookingDate parsing in MT940 about phpfints

[na-oma]

phpFinTS/lib/Fhp/Parser/MT940.php

Line 125 in e8b18a7

// :61:1605110509D198,02NMSCNONREF

Da irgendein Schlaufuchs im Standard in einer 500 Zeichen Nachricht 2 Zeichen sparen wollte, muss an dieser Stelle von einer einzigen Jahreszahl auf das Jahr von zwei verschiedenen Daten geschlossen werden.
Wenn ich den Code richtig interpretiere, nimmt er an, dass laut MT940-Standard ein Valuta-Date niemals vor einem Booking-Date liegen kann. M.a.W. Valuta-Datum kommt immer nach/ist gleich Booking-Date.
Wenn dann der Monat des Valuta-Dates tatsächlich vor dem Monat des Booking-Dates liegt, geht der Code davon aus, dass das Booking-Date im Jahr davor liegt, während das Valuta-Date im aktuellen Jahr liegt, deshalb wird ein Jahr abgezogen.

61:1906290701
29.06.19 und 01.07.18 = falsch

Ist laut Standard überhaupt erlaubt, das als 29.06.19 (valuta) und 01.07.19 (booking) zu interpretieren?

61:1901011228
01.01.19 und 28.12.17 = falsch

Da scheint definitiv was faul zu sein, Standard hin oder her. Zwei Jahre sollten das nicht sein...er zieht nochmal ein Jahr zusätzlich ab...

Ich kenne leider den Standard nicht auswendig, also weiss ich nicht, ob das Valuta-Datum auch vor dem Booking-Datum liegen darf.

[djm03]

Es gibt mittlerweile die Echtzeit-Gutschrift, zumindest sehe ich dort immer wieder den Fall. Aktuell zb:

61:1909280930
61:1909290930
61:1908310902

[na-oma]

Hmm, dann scheint das wohl möglich zu sein, dass Valuta-Date vor Booking-Date.

Dann ist die einzige Frage, ob beide Daten um ca. 1 Jahr auseinanderliegen können? Dann könnte man nicht mehr unterscheiden ob

1909280930

als

Valuta: 28.09.19 und Booking: 30.09.19

oder

Valuta: 28.09.19 und Booking: 30.09.18 (also ein Jahr eher)

geparst werden soll.

Man könnte natürlich einfach definieren, dass Booking und Valuta maximal ein halbes Jahr auseinanderliegen, und dann immer das Booking-Date genommen wird, was am nächsten am Valuta-Date liegt.

Vielleicht reicht ja dein Fix die Zeile zu löschen und deckt alle Fälle ab? Ich steh grad auf dem Schlauch^^
Mach doch mal n Pullrequest.

[djm03]

Ich wüßte jetzt nicht warum Booking und Valuta länger als ein paar Tage auseinander liegen sollten. Ob das entfernen der einen Zeile alles abdeckt, weiß ich jedoch auch nicht. Zumindest würde es die mit "falsch" angegebenen Ergebnisse korrigieren.

[nemiah]

Schick einfach mal einen PR, mir scheint es ein Fortschritt zu sein 😉

[Magellanth]

Ist durch #202 behoben, kann auch zu.