Umbenennung von Klassen

[mbmueller]

          Vielleicht sollte man die Klasse wirklich mal umbenennen in Mitgliedstyp. Man kann ja die Datenbank Tabelle beim alten Namen lassen. Für neue JVerein Entwickler ist halt die Historie nicht immer bekannt.

Das gleiche gilt dann auch für Mitgliedskonto welches man in Sollbuchung umbenennen sollte.

Wenn wir mal einen Stand haben wo nicht zuviele gleichzeitig arbeiten könnte ich das mal machen. Es sind dann halt viele Files betroffen.

Originally posted by @JohannMaierhofer in #446 (comment)

[lenilsas]

Ich fände auch eine Einheitliche Benennung der Klassen gut. Insbesondere bei den Views ist das sehr unterschiedlich, mal ...SuchView mal ...UebersichtView mal ListeView etc.
Außerdem sind im package io viele Klassen die nach util gehören, auch hier sollte mal für Ordnung gesorgt werden.
Das könnte man, wenn mal wenige PRs offen sind machen, in dem Zuge könnte mal alle Dateien formatieren.

[JohannMaierhofer]

Ich glaube, das muss sehr gut geplant werde. Wir könnten hier das Vorgehen besprechen.

Ein Weg wäre z.B.

• ab jetzt keine neuen Feature PRs, nur Bug fixes
• die bestehenden PRs abschließen
• das Thema mit der Docu implementieren
• dann die v2. 9.0 rausgeben

Anschließend

• schrittweise Klassen umbenennen incl. der getter bei anderen Klassen
• Diese einzeln reviewen falls was schief geht
• dann Klassen verschieben

Als letztes dann die Formatierung aller Klassen. Das sollte dann keine Fehler erzeugen.

Anschließend von hier mit neuen Features starten.

Die Frage wäre dann noch ob wir auch die Tabellen und Spalten in der DB mit umbenennen wollen. Das wäre aufwendig weil man dann alle sql strings anschauen muss und korrigieren.
Auch kann man dann nicht mehr auf eine alte Version zurück. Die nächste Version müsste dann eine 3.0.0 sein.
Da könnte man Dann auch auf einen Umstieg auf eine aktuelle H2db nachdenken.

[lenilsas]

• schrittweise Klassen umbenennen incl. der getter bei anderen Klassen

Dass ist mit der Refactoring Funktion von Eclipse ohne großen Aufwand möglich, es wird an allen Orten richtig ersetzt.
Ich denke die größte Schwierigkeit sind die Merg Konflikte bei anderen PRs. Ich denke auch, dass bald wieder ein Release passend wäre. Ich würde gerne die Issues mit den Änderungen bei den Rechnungen noch vorher umsetzen, so dass hier eine einmalige Umstellung ist und nicht beim nächsten Release wieder Änderungen daran sind.

[JohannMaierhofer]

Ich denke die größte Schwierigkeit sind die Merg Konflikte bei anderen PRs. Ich denke auch, dass bald wieder ein Release passend wäre. Ich würde gerne die Issues mit den Änderungen bei den Rechnungen noch vorher umsetzen, so dass hier eine einmalige Umstellung ist und nicht beim nächsten Release wieder Änderungen daran sind.

Ja, das mit den Rechnungen sollte komplett in die nächste Release.

[mbmueller]

• ab jetzt keine neuen Feature PRs, nur Bug fixes

Gilt das dann ab jetzt, oder wollen wir einen Termin festlegen, ab dem dann keine PRs zugelassen werden? Bspw. der 01.01.?

[JohannMaierhofer]

Das von mir war nur ein Vorschlag, wir sollten das noch genauer besprechen.
Eine neue Version noch im Dezember denke ich macht Sinn. Es gibt ja schon wieder einige neu Features und dann wäre auch die E-Rechnung für alle bis zum 1. Januar 2025 verfügbar.
Neue Features könnten auch noch Sinn machen. Wie z.B. das Rechnung Feature vervollständigen.
Für weitere Features wäre dann die Frage bis wann sie da sein müssten damit sie auch noch gereviewt werden können. Ein Review dauert ja auch ein paar Wochen und wir haben auch noch einige offen.

Wir müssen aber auch die Entwicklung nicht bremsen. Wenn du also schon an Features arbeitest, können wir die Umbenennung auch weiter verschieben. Sie eilt ja nicht.

[mbmueller]

Wir müssen aber auch die Entwicklung nicht bremsen. Wenn du also schon an Features arbeitest, können wir die Umbenennung auch weiter verschieben. Sie eilt ja nicht.

Arbeite im Moment eben noch an #434.

[JohannMaierhofer]

Wir müssen aber auch die Entwicklung nicht bremsen. Wenn du also schon an Features arbeitest, können wir die Umbenennung auch weiter verschieben. Sie eilt ja nicht.

Arbeite im Moment eben noch an #434.

Dann sollten wir bis danach warten mit der Umbenennung.