openjverein / jverein

Open JVerein - Open Source Vereinsverwaltung
https://openjverein.github.io
GNU General Public License v3.0
45 stars 17 forks source link

Neu Button für weitere Views #405

Closed JohannMaierhofer closed 4 weeks ago

JohannMaierhofer commented 1 month ago

Ich habe jetzt die Neu Buttons auch in den Views für Lehrgäge, Arbeitseinsätze, Wiedervorlage und Zusatzbeiträge eingebaut. Bei den Lesefeldern habe ich den MitgliedInput eingebaut. Zusätzlich habe ich bei den Views die setMandatory(true) eingebaut für Inputs die im plausi check als mandatory überprüft werden. Da bei Betrag als double die implementierte Prüfung auf null nicht funktioniert (getter liefert 0) habe ich diese in Double geändert. Einige Checks musste ich korrigieren bzw. neu einbauen.

JohannMaierhofer commented 1 month ago

PS: Das mit dem Mitglied Input bei den Vollzahler schaue ich mir noch an. Dann per eigenem PR.

JohannMaierhofer commented 1 month ago

@lenilsas Ich bin momentan nicht mehr so glücklich mit der Änderung, dass im Suche Input per Default das erste Mitglied erscheint. Ich möchte ja die Aktion für ein bestimmtes Mitglied machen und dieses auch direkt eintippen können. Jetzt muss ich immer erst den Default Namen löschen. Das finde ich umständlich. Es macht eigentlich nur bei den Lesefeldern wegen der automatischen Auswertung Sinn. Wäre es für dich ok wenn ich das wieder aus dem Suchen Input entferne und bei Lesefeldern das Query in der Action mache und dann das erste Mitglied an den Input übergebe.

lenilsas commented 1 month ago

Ja, so meinte ich es eigentlich auch. Nur bei den Lesefeldern macht der Default Eintrag Sinn.

JohannMaierhofer commented 1 month ago

Ja, so meinte ich es eigentlich auch. Nur bei den Lesefeldern macht der Default Eintrag Sinn.

Ok, ich ändere das.

JohannMaierhofer commented 1 month ago

Habe es geändert