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

Buchung geprüft Markieren. Buchungsliste teilweise aktualisieren #637

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

lenilsas
Copy link

@lenilsas lenilsas commented Feb 2, 2025

Ich habe die Möglichkeit Buchungen als Geprüft zu markieren hinzugefügt.
In der Buchungsliste kann auf ungeprüfte Buchungen gefiltert werden.
Wenn in den Einstellungen aktiviert, wird der Status automatisch mit Hibiscus synchronisiert.

In dem Zuge habe ich eine AutoUpdateTablePart erstellt, die beim Speichern eines Elements automatisch dieses neu in der Tabelle lädt. Dadurch ist die BuchungMessage überflüssig geworden. Lediglich bei Splitbuchung auflösen muss die Liste noch ganz neu geladen werden.
Die SplitbuchungWiederherstelungAction habe ich aus dem Menü extrahiert und eine eigene Datei erstellt.

@lenilsas lenilsas linked an issue Feb 2, 2025 that may be closed by this pull request
@JohannMaierhofer
Copy link

Ich bekomme diese Exception wenn ich geprüft setze. Die Buchung ist in keinem Hibiscus Konto.

[Tue Feb 04 14:43:44 CET 2025][ERROR][main][de.willuhn.jameica.gui.GUI.loop] main loop crashed, retry
java.lang.NullPointerException: Cannot invoke "java.lang.Integer.toString()" because the return value of "de.jost_net.JVerein.rmi.Buchung.getUmsatzid()" is null
at de.jost_net.JVerein.gui.action.BuchungGeprueftAction.handleAction(BuchungGeprueftAction.java:72)
at de.willuhn.jameica.gui.parts.ContextMenu$1.handleEvent(ContextMenu.java:184)

@JohannMaierhofer
Copy link

Bei mir kommt es zu einer Fehlermeldung wenn ich eine neue Splittbuchungen speichern will. Damit kann man keine mehr erzeugen.

@lenilsas
Copy link
Author

lenilsas commented Feb 5, 2025

Ich hab die Fehler behoben und den Code vereinfacht

@@ -111,7 +104,6 @@ public void handleAction(Object context) throws ApplicationException
}
}
}
control.getBuchungsList();
if (ba == null)
{
GUI.getStatusBar().setSuccessText("Buchungsarten gel�scht");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich frage mich echt, warum hier das 'ö' in "gelöscht" nicht richtig dargestellt wird, die Sonderumlaute in BuchungMenu.java jedoch schon. Na egal.

Copy link
Member

@tolot27 tolot27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht vom Code her schon mal gut aus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Buchungsliste nur partiell aktualisieren
3 participants