Auswahlbox für Geschlecht jetzt auch als Pflichtfeld markiert (#265)

openjverein · Jul 23, 2024 · 6aa99c7 · 6aa99c7
1 parent dcf2d7e
commit 6aa99c7
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/src/de/jost_net/JVerein/gui/control/KursteilnehmerControl.java b/src/de/jost_net/JVerein/gui/control/KursteilnehmerControl.java
@@ -344,7 +344,7 @@ public DateInput getGeburtsdatum() throws RemoteException
     Date d = getKursteilnehmer().getGeburtsdatum();
     this.geburtsdatum = new DateInput(d, new JVDateFormatTTMMJJJJ());
     this.geburtsdatum.setTitle("Geburtsdatum");
-    this.geburtsdatum.setText(("Bitte Geburtsdatum wählen"));
+    this.geburtsdatum.setText("Bitte Geburtsdatum wählen");
     this.geburtsdatum.setMandatory(true);
     return geburtsdatum;
   }

diff --git a/src/de/jost_net/JVerein/gui/input/GeschlechtInput.java b/src/de/jost_net/JVerein/gui/input/GeschlechtInput.java
@@ -60,11 +60,14 @@ private static ArrayList<GeschlechtObject> init() throws RemoteException
   public Object getValue()
   {
     GeschlechtObject o = (GeschlechtObject) super.getValue();
-    if (o == null)
+    if (null == o)
     {
-      return MAENNLICH;
+      return null;
+    } 
+    else
+    {
+      return o.geschlecht;
     }
-    return o.geschlecht;
   }
 
   /**

diff --git a/src/de/jost_net/JVerein/server/KursteilnehmerImpl.java b/src/de/jost_net/JVerein/server/KursteilnehmerImpl.java
@@ -97,6 +97,10 @@ private void plausi() throws RemoteException, ApplicationException
       {
         throw new ApplicationException("Bitte Geburtsdatum eingeben");
       }
+      if (getGeschlecht() == null)
+      {
+        throw new ApplicationException("Bitte Geschlecht eingeben");
+      }
     }
     if (getIban() == null || getIban().length() == 0)
     {