Fix Buchungskorrektur

[JohannMaierhofer]

Dieser Request korrigiert das Feature Buchungskorrektur wie in #122 diskutiert.
Die Änderungen sind wie folgt:

• Der implementierte Suchalgorithmus wurde durch den Aufruf "VerwendungszweckUtil.getTag(t, Tag.SVWZ)" aus Hibiscus ersetzt. Da die Methode eine Klasse vom Typ Transfer braucht habe ich eine Dummy Klasse erzeugt die das Interface Transfer implementiert, allerdings nur für den Zweck. Damit funktioniert die Ersetzung wie gewünscht.
• Beim Query der Buchungen für den Buchungskorrektur View wurde bisher nur nach SVWZ gesucht. Es gibt aber Buchungen die das nicht enthalten aber einige der anderen Keys. Ich schaue also jetzt ob irgend ein SEPA Tag enthalten ist. Da IBAN bei mir auch manchmal im Zweck vorkommt suche ich hier direkt nach IBAN+ oder IBAN:
• Die Tabelle hat auch Buchungen die schon abgeschlossen waren angezeigt. Beim speichern wurden sie dann ignoriert. Ich zeige jetzt nur noch Buchungen an die noch nicht abgeschlossen sind.
• BuchungsListTablePart habe ich durch TablePart ersetzt. Damit wird nicht mehr im Tabellen Summary der Gesamtbetrag ausgegeben. Der macht in diesem View keinen Sinn.

[JohannMaierhofer]

@willuhn ich hätte eine Frage an dich.
Eine Bank fügt im Verwendungszweck die benutze TAN hinzu. Hier zwei Beispiele
Verpflegung TAN: 253631 IBAN: DE816....
Biertasting TAN: Auftrag nicht TAN-pflichtig, da Kleinbetragszahlung IBAN: DE32.....

Den TAN Text hätte ich auch gerne entfernt. Würde es Sinn machen diesen als Tag in VerwendungszweckUtil aufzunehmen obwohl es kein offizieller SEPA Key ist?
Ich könnte das natürlich auch in JVerein nachträglich herausfiltern.

[willuhn]

@lenilsas hat in willuhn/obantoo#2 gerade sowas eingebaut - allerdings für "TAN1" und nicht für "TAN".

[lenilsas]

es ist nicht in willuhn/obantoo#2 sondern in willuhn/hibiscus#142

[JohannMaierhofer]

Könnt ihr dann auch TAN ohne 1 einbauen?

[lenilsas]

TAN habe ich in den Pullrequest willuhn/hibiscus#142 mit hinein genommen

[willuhn]

• Der implementierte Suchalgorithmus wurde durch den Aufruf "VerwendungszweckUtil.getTag(t, Tag.SVWZ)" aus Hibiscus ersetzt. Da die Methode eine Klasse vom Typ Transfer braucht habe ich eine Dummy Klasse erzeugt die das Interface Transfer implementiert, allerdings nur für den Zweck. Damit funktioniert die Ersetzung wie gewünscht.

Es gibt auch eine Methode "Map<Tag,String> VerwendungszweckUtil.parse(String... lines)". Das liefert dir eine Map mit allen Tags und benötigt nicht das "Transfer"-Interface.

[JohannMaierhofer]

Das hatte ich mir auch angeschaut, aber getTag macht noch etwas mehr. Das würde ich sonst verlieren.

[willuhn]

Das hatte ich mir auch angeschaut, aber getTag macht noch etwas mehr. Das würde ich sonst verlieren.

Stimmt, hast Recht.