Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SEPA Import: Überprüfe PAIN Typ #11

Closed

Conversation

@michael-dev
Copy link

commented Dec 17, 2015

Bislang konnten SEPA Sammel-Lastschriften als SEPA Sammel-Überweisung importiert werden.
Dies konnte dazu führen, dass versehentlich die falsche Transaktionsart ausgeführt wurde.

Diese Änderung überprüft den PAIN Typ und erlaubt für den Import bei Überweisungen nur Überweisungen und bei Lastschriften nur Lastschriften.

Bislang konnten SEPA Sammel-Lastschriften als SEPA Sammel-Überweisung importiert werden.
Dies konnte dazu führen, dass versehentlich die falsche Transaktionsart ausgeführt wurde.

Diese Änderung überprüft den PAIN Typ und erlaubt für den Import bei Überweisungen nur Überweisungen und bei Lastschriften nur Lastschriften.
@michael-dev michael-dev force-pushed the michael-dev:feature/check-pain-version branch from 96792b4 to 31e9017 Dec 17, 2015
@willuhn

This comment has been minimized.

Copy link
Owner

commented Dec 17, 2015

Es ist eigentlich Absicht, dass das nicht geprueft wird. Damit der User die Auftraege auch dann importieren kann, wenn die SEPA-Datei eigentlich falsch war, der User aber keine Moeglichkeit hat, das zu korrigieren, weil er keinen Zugriff auf das Programm hat, aus dem die Daten stammen.

Mit dem Patch haette der User u.U. keine Chance mehr, die Daten zu verarbeiten.

@michael-dev

This comment has been minimized.

Copy link
Author

commented Dec 17, 2015

Könnte man vielleicht wenigstens eine Warnung anzeigen? Weil hier habe ich gerade einen Nutzer, dem genau o.g. passiert ist.

@willuhn

This comment has been minimized.

Copy link
Owner

commented Dec 17, 2015

Ja, eine Warnung ist eine gute Idee.

@michael-dev michael-dev force-pushed the michael-dev:feature/check-pain-version branch 2 times, most recently from 3bafcdb to 2076128 Dec 17, 2015
@michael-dev

This comment has been minimized.

Copy link
Author

commented Dec 17, 2015

Mir ist nicht ganz klar, wie ich ein Warnungs-Popup in de.willuhn.jameica.hbci.io.AbstractSepaImporter erzeugen soll. Daher habe ich stattdessen dem Import-Dialog eine "Import erzwingen" Checkbox ergänzt, mit welcher der Import erzwungen werden kann, wenn er sonst wegen PAIN Typ nicht akzeptiert würde.

… dies überschreiben

Durch die neue Checkbox "Import erzwingen" kann die PAIN Typ Überprüfung deaktiviert werden.
@michael-dev michael-dev force-pushed the michael-dev:feature/check-pain-version branch from 2076128 to 8ec6d0f Dec 17, 2015
@willuhn

This comment has been minimized.

Copy link
Owner

commented Dec 17, 2015

Du kannst Rueckfragen von ueberall aus per "Application.getCallback().askUser(" foo bar?")" machen. Wenn die Anwendung ohne UI laeuft, erscheint die Frage dann auf der Konsole.

@willuhn

This comment has been minimized.

Copy link
Owner

commented Dec 18, 2015

Habs mit 2ce2868 eingebaut. Rueckfrage erfolgt direkt im Importer per Application.getCallback().askUser()

@willuhn willuhn closed this Dec 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.