-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature: Unterdrücke Buchungsarten in Dialogen #104
Feature: Unterdrücke Buchungsarten in Dialogen #104
Conversation
Das Feature unterdrückt in Auswahldialogen Buchungsarten die seit mehr als 2 Jahren nicht benutzt wurden. Das Feature ist in den Einstellungen auswählbar.
Vorschlag: Anstatt 2 Jahre fest zu definieren lege in den Einstellungen ein Zahlenfeld an in dem man den Wert setzen kann. 0/null/leer String wäre dann "alle Jahre anzeigen" und eine natürliche Zahl entsprechend das Limit. |
Das wäre sicherlich eine Verbesserung des Features. Ich versuche den Code entsprechend zu ändern, evtl. mit Monaten statt Jahren. |
Im Feature zur Unterdrückung ungenutzter Buchungsarten ist jetzt der Zeitraum in Monaten einstellbar.
Ich habe noch eine Frage. |
Exception entfernt in BuchungZuordnungDialog wenn abbrechen gewählt wird. Deprecated Long ersetzt.
Abbruch gibt keine Meldung mehr aus so wie es auch bei vielen anderen Dialogen ist. Funktioniert analog bei ESC und dem Schließen Icon oben rechts. Icons zu den Buttons hinzugefügt analog Mitglieder Konto Auswahl Dialog.
Vorschlag von Willuhn implementiert
Habe den Kommentar bezüglich der catch Implementierung aus Request openjverein#104 implementiert. Analog zu Buchungsart Zuordnung kann man einstellen ob überschrieben werden soll. Analog zu Mitgliedskonto Zuordnung kann das Projekt auch gelöscht werden
Habe den Kommentar bezüglich der catch Implementierung aus Request openjverein#104 implementiert. Analog zu Mitgliedskonto Zuordnung kann der Kontoauszug auch per Button forced gelöscht werden
Analog zu den anderen Dialogen möchte ich noch löschen implementieren. |
* Verbesserung ProjektAuswahlDialog Update Button Text "weiter" zu "übernehmen". Weiter suggeriert das noch was folgt. Übernehmen wie bei anderen Dialogen. Korrektur falsche ä Icons für die Buttons Korrektes Handling von Abbruch über Button, Icon und ESC ohne Meldung * replace Boolean with boolean * Weiter Verbesserung Habe den Kommentar bezüglich der catch Implementierung aus Request #104 implementiert. Analog zu Buchungsart Zuordnung kann man einstellen ob überschrieben werden soll. Analog zu Mitgliedskonto Zuordnung kann das Projekt auch gelöscht werden * remove obsolete import java.rmi.RemoteException * Bessere Dialoghöhe 175
* Verbesserung KontoauszugZuordnungDialog Icons für die Buttons hinzugefügt, analog zu den anderen Dialogen. Korrektes Handling von Abbruch über Button, Icon und ESC ohne Ausgabe einer Meldung. ESC und Abbruch Button meldeten "Fehler bei der Zuordnung des Projektes". Abbruch Icon rechts oben meldete "Kontoauszug zugeordnet" obwohl Abbruch!. * Weitere Verbesserung und fix catch Habe den Kommentar bezüglich der catch Implementierung aus Request #104 implementiert. Analog zu Mitgliedskonto Zuordnung kann der Kontoauszug auch per Button forced gelöscht werden * Delete obsolete import java.rmi.RemoteException * Dialoghöhe analog zu Projekt Auswahl
Try/Catch Handling entsprechend dem Kommentar von willuhn aus Pull request openjverein#104 commit d8cebbf angepasst
Jetzt könnte man das Feature mergen |
Könnte jemand dieses Feature reviewen, bitte. Aus historischen Gründen habe ich alte Buchungsarten die benutzt sind und ich sie darum nicht löschen kann. Sie werden aber nicht mehr für neue Buchungen verwendet. |
Ich habe oben einen Code-Kommentar zu deiner Implementierung geschrieben. Erst alle Daten aus der DB zu ziehen und dann zu verarbeiten ist nicht performant. Es sollten nur die benötigten Daten aus der DB geladen werden. Einen nicht getesteten Code-Vorschlag habe ich im review hinterlassen. |
Wo finde ich den Code-Kommentar? |
Mach mal bitte einen Issue auf, dass du als Mitglied in der OpenJVerein github Organisation aufgenommen werden willst. Ich kann mir vorstellen, dass nur Mitglieder die reviews sehen können. Ansonsten https://github.com/openjverein/jverein/pull/104/files#diff-222eb26935ffd571a1c0aa018d3157ecda51de7c7428fc669db611a49051eda5 Zeile 150. |
Seltsam, jetzt bin ich Mitglied, aber ich kann trotzdem keinen Kommentar finden. Auch bei dem Link sehe ich nichts, auch keine Zeile 150. |
mhm dann noch mal hier - Formatierung geht leider verloren. Statt im Java das Filtern durchzuführen soll doch lieber eine gefilterte Liste aus der DB geliefert werden (analog bei den anderen Filtern im Code). Irgendwie sowas (ungetestet !): |
Der Vorschlag funktioniert. Allerdings sind jetzt Buchungsklassen so oft im Menü wie es entsprechende Buchungen gibt. Ich muss noch filtern damit ein Eintrag nur einmal vorkommt. |
Habe den Kommentar von dippeal implementiert |
Habe auch die anderen Stellen aus Feature openjverein#104 vereinfacht um doppelte Einträge bei der Datenbank Abfrage zu verhindern.
* Zwischenstand * Feature Gegenbuchung * kleine Optimierung * Auch Mitgliedskonto entfernen * Resolve comments * Use DISTINCT * Kommentare gelöst Habe auch die anderen Stellen aus Feature #104 vereinfacht um doppelte Einträge bei der Datenbank Abfrage zu verhindern. * Remove Sub-Select * Change Update Version 429 to 430 * konto aus query entfernt * Fertige Buchungsartabfrage * Fix Sort und Formatter * Fix Filter in Combo Box * Neue Buchung bei Action * Verbessere Buchungsart Eingabe * Update Schleife * Update Schleife in KontoKontrol * Entferne Dateien die nicht zum Feature gehören
Das Feature unterdrückt in Auswahldialogen Buchungsarten die seit mehr als 2 Jahren nicht benutzt wurden. Das Feature ist in den Einstellungen auswählbar.