Skip to content
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

Fix Zusatzfelder #294

Merged
merged 1 commit into from
Sep 7, 2024
Merged

Conversation

JohannMaierhofer
Copy link

Ich habe einige Fehler bezüglich der Zusatzfelder gefixed die mir schon früher aufgefallen sind. Dabei habe ich gesehen, dass ich in #290 den IntegerNullInput nicht gebraucht hätte. Man kann mit -1 in den Settings auch das IntegerInput Feld auf null setzen. Allerdings brauche ich es hier für den Fix.
Es gab mehrere Fehler zu korrigieren:

  • Im Mitglieder Tab für Zusatzfelder wurde bei Integer Inputs beim Speichern eine 0 gespeichert wenn nichts gezetzt war (null war). Man konnte die 0 auch nicht löschen. Hatte man schon Mitglieder bevor man das Integer Zusatzfeld erzeugt hat, dann hatten die den Wert null und es wurde nichts angezeigt in der Tabelle. Nur wenn man das Mitglied editiert hat wurde der Wert auf 0 gesetzt. Darum gab es eine Mix aus null und 0. Ich lasse jetzt beim Speichern den Wert null.
  • MitgliedQuery hat geprüft ob die Anzahl der selektierten Einträge größer ist als die Anzahl der Zusatzfelder Definitionen. Dann wird die Anzahl auf 0 gesetzt. Das ist aber falsch. Bei Integer, Datum etc. sind im Dialog je zwei Einträge für von und bis. Also kann die Anzahl der selektierten Einträge auch großer sein. Ich habe den Check entfernt.
  • In der Parameter Liste der PDF Reports bei Auswertung Mitglieder werden alle gesetzten Parameter ausgegeben und dabei auf null und false geprüft. Nicht gesetzte Inter Werte wurden mit -1 in den Settings gespeichert. Das ist nicht null und darum werden sie mit dem Wert -1 in den Parametern aufgeführt obwohl sie da nicht hingehören. Ich dachte schon ich ignoriere auch die "-1" allerdings könnte man ja in einem Textfeld -1 eintragen. Das würde dann auch ignoriert. Darum habe ich hier nun den IntegerNullInput genommen und setzte auch bei Integer bei null den Wert in den Settings auf "" so wie auch bei Datum etc.
  • In der Parameter Liste wurde bei den Parameter Namen der name ausgegeben und nicht das Label wie am GUI. Das war unschön, schon wegen der Kleinschrift. Ich nehme jetzt das Label.
  • Dann gab es das Problem, dass die von und bis Einträge den gleichen Namen hatten. Das führte in der params Map dazu, dass der bis Eintrag den von Eintrag überschrieben hat. Es war dann im Report nur ein Eintrag und man wusste auch nicht ob es der von oder bis Eintrag ist. Ich speichere in den Settings jetzt nicht nur das Label aus der Felddefinition sondern das GUI Label mit dem von und bis. Damit ist der Key unterschiedlich und man sieht auch ob der angezeigte Wert der von oder bis Wert ist.

@JohannMaierhofer JohannMaierhofer added the fix This will fix a bug label Aug 12, 2024
@lenilsas
Copy link

Beim Schreiben der Einstellungen (SetAttribut) hast du die namen geändert, hast du sie auch beim lesen geändert, das konnte ich jetzt nicht finden?

@JohannMaierhofer
Copy link
Author

Die werden nicht für das GUI gelesen. Für den Dialog kommen die Label aus der Datenbank von der Tabelle felddefinition.
Gelesen werden die Setting Namen beim Generieren des PDF Reports und so wie gelesen in den Report geschrieben.

@lenilsas
Copy link

ok, so genau habe ich das nicht angeschaut.

@dippeal dippeal merged commit c7dbcd5 into openjverein:master Sep 7, 2024
@JohannMaierhofer JohannMaierhofer deleted the fix_zusatzfelder branch October 20, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix This will fix a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants