feature/belegnummer #58

dippeal · 2023-10-29T23:26:54Z

In dem PR geht es um die neue Funktion Belegnummern zu verwenden und die Möglichkeit diese zu importieren. Original von VinRud im jverein#62 und Forum Titel neues Feld bei Buchungen - Belegnummer.

Im PR ist die angepasste Version von kpatzwald um den DB Fehler com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException in buchungscontrol.java Zeile 238 zu beheben.

Fix BuchungsControl.java
Code Formatierung
Neue Update0419 für belegnummer

…tfunktion aus anderen Buchungsfeldern) + neue Einstellmöglichkeit

…erwendeten Einstellung)

…die Belegnummer (anstatt der Buchungsnummer) anzeigen

…nt automatisch pro Konto und pro Geschäftsjahr (de)aktiviert werden.

… erstellten Felder in den Einstellungen wieder gelöscht werden

…r für Iterator

Hier nun die im Forum angemerkte, notwendige Korrektur (https://jverein-forum.de/viewtopic.php?p=19526#p19526) in Zeile 238.

…ummer

willuhn · 2023-11-01T08:51:23Z

Die Umlaute in BuchungsControl sind nicht kaputt. Du hast sie lediglich mit dem falschen Encoding geöffnet. JVerein verwendet ISO-8859-1 und nicht UTF-8 (kann man sicher irgendwann mal umstellen, soll aber nicht Thema dieses PR sein).

Generell: Bitte kleinere(!) Pull-Requests, die jeweils nur genau eine isolierte Änderung enthalten. Das Review ist sonst echt schwer.

src/de/jost_net/JVerein/Einstellungen.java

willuhn · 2023-11-01T11:57:35Z

Aber der PR #24 ist doch noch offen. Warum hast du die selbe Änderung dann nochmal hier als neuen PR eingereicht? Ich würde es sinnvoller finden, das Thema dann dort in #24 zu behandeln.

dippeal · 2023-11-01T12:05:56Z

In meinem Branch ist die fertige Lösung mit aktuellem Stand von kpatzwald und angepasster Update Datei (welche in #24 zum Konflikt führt).

NicoB77 · 2023-11-02T11:15:49Z

Es ist ein bisschen unglücklich, ausgerechnet in einem so großen PR die Formatierung zu ändern. Z.B. scheinen die Änderungen in BuchungQuery.java (fast?) nur aus Umformatierungen zu bestehen. Es ist sehr mühsam, hier zu kontrollieren, ob es überhaupt inhaltliche Änderungen gibt. Das mach das Review unnötig langwierig.

dippeal · 2023-11-02T11:24:11Z

Ich bin der Meinung, dass ich es vereinfacht habe, da in dem f1c553a mindestens die Formatierung (wohl auch das encoding) umgestellt wurden was zu noch mehr geänderten Zeilen im PR geführt hätte.

src/de/jost_net/JVerein/server/EinstellungImpl.java

src/de/jost_net/JVerein/gui/control/EinstellungControl.java

dippeal

Review und Funktionstest okay

…ummer - Add Update0421 database update file

…ummer

src/de/jost_net/JVerein/gui/control/EinstellungControl.java

src/de/jost_net/JVerein/gui/control/BuchungsControl.java

NicoB77 · 2023-11-26T12:57:16Z

src/de/jost_net/JVerein/gui/control/BuchungsControl.java

+  public static void setNewLastBelegnummer(Integer NewLastBelegnummer,
+      Date NewDate_, String NewKontoId) throws RemoteException
+  {
+    getLastBelegnummer(NewDate_, NewKontoId);


Wird hier nur für side effects aufgerufen, das ist nicht gut. Generell wäre der Code deutlich einfacher, wenn der aktuelle Status (Konto, Geschäftsjahr, ...) nicht gespeichert würde.

Gerne einen Code Vorschlag machen. Es ist nicht meine Implementierung, ich würde die Funktion dem openjverein aber gerne hinzufügen.

NicoB77 · 2023-11-26T13:15:18Z

src/de/jost_net/JVerein/gui/control/BuchungsControl.java


      if (b.getSpeicherung())
      {
        b.store();
        getID().setValue(b.getID());
+        setNewLastBelegnummer(b.getBelegnummer(), b.getDatum(),


Wenn ich den Code richtig lese, bekommt eine neue Buchung die Nachfolgenummer der zuletzt bearbeiteten Buchung. Wenn das eine alte Buchung war, die nur bearbeitet wurde, gibt es doppelte Nummern. Ich würde den Cache entfernen (und damit auch die Funktion setNewLastBelegnummer), dann gibt es das Probem nicht.

Kann ich nicht nachvollziehen. Die setNewLastBelegnummer holt sich doch über getLastBelegnummer die nächste/korrekte Nummer. Hast du deine Überlegung mal getestet?

Nein, die Funktion holt sich nichts. getLastBelegnummer wird nur aufgerufen, um die Cache-Variablen zu setzen.

Mach bitte einen Code-Vorschlag (change request / PR).
setNewLastBelegnummer wird insgesamt an drei Stellen verwendet:

src/de/jost_net/JVerein/gui/control/BuchungsControl.java

src/de/jost_net/JVerein/io/SplitbuchungsContainer.java

src/de/jost_net/JVerein/io/Buchungsuebernahme.java

NicoB77 · 2023-11-26T13:29:27Z

src/de/jost_net/JVerein/server/BuchungImpl.java

+  public Integer getBelegnummer() throws RemoteException
+  {
+    Integer belegnummer = (Integer) getAttribute("belegnummer");
+    if (belegnummer == null)


Der Fall sollte doch gar nicht vorkommen. Warum nicht return (Integer) ... wie bei den anderen Attributen? Genauso bei setBelegnummer.

Hier scheint der default Wert -1 von Bedeutung zu sein. In der Update Datei ist der default auch mit -1 gesetzt. Würde ich so lassen.

Ich sehe aber keinen Fall, für den belegnummer == null eintritt. Für die alten Zeilen wird das Feld beim Datenbankupdate gefüllt, für die danach angelegten Buchungen beim Speichern. Und wenn das Feature deaktiviert ist, wird immer 0 gespeichert, nicht -1.

NicoB77 · 2023-11-26T13:46:57Z

src/de/jost_net/JVerein/server/DDLTool/Updates/Update0422.java

+      execute(addColumn("einstellung",
+          new Column("belegnummer_pro_jahr", COLTYPE.BOOLEAN, 0, "TRUE", false, false)), false);
+      execute(
+          addColumn("buchung", new Column("belegnummer", COLTYPE.INTEGER, 19, "-1", false, false)),


Warum -1, wenn neue Buchungen bei deaktiviertem Feature 0 bekommen?

Steht in Zeile 100. Um zu prüfen, dass die hinterlegten Belegnummern nicht konvertiert werden können.

NicoB77 · 2023-11-26T13:46:59Z

src/de/jost_net/JVerein/server/DDLTool/Updates/Update0422.java

+        "In der neueste jVerein Version besteht die M�glichkeit manuelle Belegnummern anstatt "
+            + "der Buchungsnummer aus der Datenbank zu verwenden.\n\nFalls du bisher schon Belegnummern verwendet hast, "
+            + "kannst du im folgenden Dialog ausw�hlen, aus welchem bisherigen Buchungsfeld diese importiert werden sollen. "
+            + "Falls du dieses Feature nicht nutzen m�chtest, w�hle \"Buchungsnummer\" aus.\n\n");


Wenn das Feature nicht genutzt wird, bekommen neue Buchungen Belegnummer 0. Beim Import nicht, das kommt mir nicht vernünftig vor. Außerdem ist das Datenbankupdate kein guter Ort für den Import, weil dann der Auswahldialog auch angezeigt wird, wenn JVerein neu aufgesetzt wird. Der Auswahldialog sollte angezeigt werden, wenn die Einstellungen verändert werden. Mit der zusätzlichen Option, gar keinen Import vorzunehmen.

Das hatte sich @VinRud so gedacht. Ich finde es nicht schlimm. Kannst gerne einen Code-Vorschlag machen.

…e/belegnummer

…ummer

FullHD23 · 2024-01-31T10:29:10Z

Hallo zusammen,
ich finde es super wie viel Aktivität hier dank euch in der Weiterentwicklung von JVerein ist. Mit den Belegnummern ist es leider weiterhin sehr schwierig für mich, weil ich diese für die Erstellung des Kassenberichts (letztlich manuelle Sortierung von mehreren Konten und einer Barkasse) benötige.

Scheinbar ist ja weiterhin das größte Problem hier die unterschiedliche Formatierung. Leider weiß ich nicht genau wie man die Konflikte hier auflöst und somit kann das Feature aktuell nicht übernommen werden...

VinRud and others added 10 commits April 13, 2020 15:07

Belegnummern können verwendet werden: Update der Datenbank (mit Impor…

2e848db

…tfunktion aus anderen Buchungsfeldern) + neue Einstellmöglichkeit

Belegnummer wird in der Auswertung berücksichtigt (entsprechend der v…

f1c553a

…erwendeten Einstellung)

die Views "Splitbuchung" und "Spendenbescheinigung" sollen auch ggf. …

15055f4

…die Belegnummer (anstatt der Buchungsnummer) anzeigen

Belegnummern werden automatisch inkrementiert. Dabei kann der Inkreme…

35feda9

…nt automatisch pro Konto und pro Geschäftsjahr (de)aktiviert werden.

in DB Update: Falls Fehler beim Import auftritt, sollen die zuvor neu…

6fa6779

… erstellten Felder in den Einstellungen wieder gelöscht werden

auslesen der letzten Belegnummer direkt über SQL Befehl anstatt Filte…

14e4b38

…r für Iterator

Merge branch 'feature/belegnummer'

f221a44

Update BuchungsControl.java

bcba0ab

Hier nun die im Forum angemerkte, notwendige Korrektur (https://jverein-forum.de/viewtopic.php?p=19526#p19526) in Zeile 238.

Merge remote-tracking branch 'openjverein/master' into feature/belegn…

6687896

…ummer

Fix: Umlaut in BuchungsControl.java & formatting

bcd74a6

dippeal mentioned this pull request Oct 31, 2023

feature/automatische verbuchung von steuern #65

Closed

willuhn reviewed Nov 1, 2023

View reviewed changes

src/de/jost_net/JVerein/Einstellungen.java Show resolved Hide resolved

dippeal commented Nov 3, 2023

View reviewed changes

src/de/jost_net/JVerein/server/EinstellungImpl.java Show resolved Hide resolved

dippeal commented Nov 3, 2023

View reviewed changes

src/de/jost_net/JVerein/gui/control/EinstellungControl.java Show resolved Hide resolved

dippeal commented Nov 3, 2023

View reviewed changes

This was referenced Nov 6, 2023

Formularfeld Zähler & Formularverknüpfungen #72

Merged

Feature/belegnummer #24

Closed

Alexander Dippe added 2 commits November 12, 2023 11:55

Merge remote-tracking branch 'openjverein/master' into feature/belegn…

990f827

…ummer - Add Update0421 database update file

Merge remote-tracking branch 'openjverein/master' into feature/belegn…

d71f4c2

…ummer

NicoB77 requested changes Nov 26, 2023

View reviewed changes

Alexander Dippe added 5 commits November 26, 2023 19:14

Fix: setUnterdrueckungOhneBuchung doppelt

af46cea

Fix. Encoding text

ddc5c85

Fix: encoding issues

17df4c9

Merge remote-tracking branch 'origin/feature/belegnummer' into featur…

6013e91

…e/belegnummer

Merge remote-tracking branch 'openjverein/master' into feature/belegn…

b7f57cf

…ummer

Merge remote-tracking branch 'openjverein/master' into feature/belegn…

4b06048

…ummer

dippeal added the enhancement New feature or request label Dec 14, 2023

Merge remote-tracking branch 'openjverein/master' into feature/belegn…

76dfce4

…ummer

dippeal added the help wanted Extra attention is needed label Mar 10, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feature/belegnummer #58

feature/belegnummer #58

dippeal commented Oct 29, 2023 •

edited

willuhn commented Nov 1, 2023

willuhn commented Nov 1, 2023

dippeal commented Nov 1, 2023

NicoB77 commented Nov 2, 2023

dippeal commented Nov 2, 2023

dippeal left a comment

NicoB77 Nov 26, 2023

dippeal Nov 26, 2023

NicoB77 Nov 26, 2023

dippeal Nov 26, 2023

NicoB77 Nov 26, 2023

dippeal Nov 26, 2023

NicoB77 Nov 26, 2023

dippeal Nov 26, 2023

NicoB77 Nov 26, 2023

NicoB77 Nov 26, 2023

dippeal Nov 26, 2023

NicoB77 Nov 26, 2023

dippeal Nov 26, 2023

FullHD23 commented Jan 31, 2024

feature/belegnummer #58

Are you sure you want to change the base?

feature/belegnummer #58

Conversation

dippeal commented Oct 29, 2023 • edited

willuhn commented Nov 1, 2023

willuhn commented Nov 1, 2023

dippeal commented Nov 1, 2023

NicoB77 commented Nov 2, 2023

dippeal commented Nov 2, 2023

dippeal left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

FullHD23 commented Jan 31, 2024

dippeal commented Oct 29, 2023 •

edited