Umbenennung von Klassen

openjverein / jverein

Open JVerein - Open Source Vereinsverwaltung

https://openjverein.github.io

GNU General Public License v3.0

45 stars 17 forks source link

Umbenennung von Klassen #480

Open mbmueller opened 1 week ago

mbmueller commented 1 week ago

          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 https://github.com/openjverein/jverein/pull/446#discussion_r1850882194_

lenilsas commented 2 days ago

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 commented 1 day ago

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 commented 1 day ago

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 commented 1 day ago

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.