Feature gegenbuchung #138
Feature gegenbuchung #138
Conversation
dippeal
added
documentation
src/de/jost_net/JVerein/gui/action/BuchungGegenbuchungAction.java
Outdated
Show resolved
Hide resolved
Habe auch die anderen Stellen aus Feature openjverein#104 vereinfacht um doppelte Einträge bei der Datenbank Abfrage zu verhindern.
|
Ich hab mal das Review von @NicoB77 und/oder @MSchmalzl angefordert, da ich mich fachlich mit dem Thema zu wenig auskenne, um beurteilen zu können, ob das gemerged werden kann. |
| b.setAuszugsnummer(null); | ||
| b.setBlattnummer(null); | ||
| b.setMitgliedskonto(null); | ||
| GUI.startView(new BuchungView(), b); |
There was a problem hiding this comment.
Was ist mit den anderen Attributen (Abrechnungslauf, Projekt, Spendenbescheinigung, ...)? Wäre es nicht sinnvoller, eine neue Buchung zu erstellen und nur die gewünschten Attribute zu füllen?
There was a problem hiding this comment.
Ja, wahrscheinlich ist das besser. Ich werde das ändern.
|
Habe im letzten Commit ddie Eingabe der Buchungsart im Konto auf einen .setPleaseChoose("Bitte wählen...") abgeändert. Das ist ja anscheinen üblich so. Der Text ist wie beim Hibiscus Konto. |
| bu.setDatum(b.getDatum()); | ||
| if (b.getBuchungsart() != null) | ||
| bu.setBuchungsart(b.getBuchungsartId()); | ||
| if (b.getProjekt() != null) |
There was a problem hiding this comment.
Führt das Setzen des Projekts nicht dazu, dass sich die beiden Buchungen im Projektsaldo annullieren? Mir fällt gerade kein Szenario ein, in dem man das will.
There was a problem hiding this comment.
Also bei mir heben sich Umbuchungen auch im normalen Buchungsklassen Saldo gegenseitig auf. Dafür sind es ja Umbuchungen und keine Einnahmen oder Ausgaben. Und ich erlaube ja die Aktion nur bei Umbuchungen.
Z. B. Eine Umbuchung von Geld vom Girokonto auf ein Anlagekonto muss sich aufheben weil das weder Einnahmen noch Ausgaben sind. Der Geldbestand bleibt ja gleich.
Oder wenn ich eine Einzahlung für eine Kaution habe ist das auch keine Einnahme sondern ich mache eine Gegenbuchung im Kautionskonto. Ich habe mehr Geld auf dem Girokonto aber gleichzeitig mehr Schulden.
Wenn die Buchungen jetzt einem Projekt zugeordnet sind sollte das doch innerhalb des Projektes genauso sein nehme ich mal an.
There was a problem hiding this comment.
Hier stimme ich @JohannMaierhofer zu.
There was a problem hiding this comment.
Warum sollte die Umbuchung dann überhaupt einem Projekt zugeordnet werden?
There was a problem hiding this comment.
Ich denke, das hängt dann davon ab ob man auch an anderen Stellen die Buchungen als Teil des Projekts sehen möchte z.B. in den Buchungslisten.
Für das Feature ist das aber egal.
Entweder ist eine Buchung nicht Teil eines Projektes und dann wird auch nichts mit kopiert.
Oder es ist Teil eines Projektes und dann wird es halt mit kopiert. Sollte dann aus irgendeinem Grund das Projekt nicht in der Gegenbuchung gewünscht sein, dann kann man es ja vor dem abspeichern immer noch rauslöschen. Aber das Behalten dürfte in dem Fall wohl der Standard sein.
There was a problem hiding this comment.
Ich ordne Umbuchungen auch immer einem Projekt zu -zumindest wenn sie zu einem Projekt gehören.
Es ist aber wie schon richtig beschrieben jedem selbst überlassen ob er das macht oder nicht.
|
Nach dem Kommentar oben, beim Anlegen eine neue Buchungen zu erzeugen frage ich mich ob das nicht auch bei der bestehenden copy Funktion so gemacht werden sollte. Da werden ja dann auch versteckte Attribute kopiert die man dann im View nicht sieht. |
|
Ich habe jetzt die Dateien die nicht direkt zum Feature gehören entfernt und werde sie separat übergeben. Das denke ich ist besser, dann kann man sie separat entscheiden. Die anderen Fixes sind in der Diskussion zu diesem Feature gefunden worden. Ich hatte sie halt gleich geändert. |
Das Feature soll das Erstellen einer Gegenbuchung bei Umbuchungen in ein offline Konto vereinfachen.
Bei einer Umbuchung in ein offline Konto erhält man den Buchungseintrag nicht über Hibiscus und muss ihn deshalb manuell erzeugen.
Bisher benutze ich die Copy Funktion, lösche dann Kontoauszugsdaten, invertiere den Betrag und setzte das andere Konto. Das geht aber nicht immer weil die Copy Funktion nicht bei Elementen einer Splitbuchung existiert. Das ist aber nötig, z.B. Abbuchung eines Darlehens. Diese splitte ich in Zins und Tilgung. Die Tilgung braucht dann eine Gegenbuchung im offline Darlehenskonto.
Das neue Feature schaut jetzt so aus:
Ich hoffe, das können auch noch Andere brauchen.