"Alle" statt "Bitte auswählen" in den Filtern (openjverein#584)

* "Alle" statt "Bitte auswählen" in den Filtern * ALLE als Konstante
JohannMaierhofer · Jan 17, 2025 · 5f10323 · 5f10323
1 parent 5fc452b
commit 5f10323
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 17 deletions.
diff --git a/src/de/jost_net/JVerein/Queries/MitgliedQuery.java b/src/de/jost_net/JVerein/Queries/MitgliedQuery.java
@@ -292,8 +292,7 @@ else if (control.getMitgliedStatus().getValue().equals("Abgemeldet"))
       Date d = (Date) control.getSterbedatumbis().getValue();
       bedingungen.add(new java.sql.Date(d.getTime()));
     }
-    if (control.isSuchGeschlechtAktiv() && control.getSuchGeschlecht().getText() != null
-        && !control.getSuchGeschlecht().getText().equals("Bitte auswählen"))
+    if (control.isSuchGeschlechtAktiv() && control.getSuchGeschlecht().getValue() != null)
     {
       addCondition("geschlecht = ?");
       String g = (String) control.getSuchGeschlecht().getValue();

diff --git a/src/de/jost_net/JVerein/gui/control/BuchungsControl.java b/src/de/jost_net/JVerein/gui/control/BuchungsControl.java
@@ -828,6 +828,10 @@ public SelectInput getSuchBuchungsart() throws RemoteException
     suchbuchungsart.setList(liste);
     suchbuchungsart.setValue(b);
     suchbuchungsart.addListener(new FilterListener());
+    if (suchbuchungsart instanceof SelectInput)
+    {
+      suchbuchungsart.setPleaseChoose(FilterControl.ALLE);
+    }
     return suchbuchungsart;
   }
 

diff --git a/src/de/jost_net/JVerein/gui/control/FilterControl.java b/src/de/jost_net/JVerein/gui/control/FilterControl.java
@@ -73,7 +73,9 @@
 import de.willuhn.util.ApplicationException;
 
 public class FilterControl extends AbstractControl
-{  
+{
+  public final static String ALLE = "Alle";
+
   // String für allgemeine Settings z.B. settings1
   protected String settingsprefix = "";
 
@@ -266,7 +268,7 @@ else if (typ == Mitgliedstyp.NICHTMITGLIED || typ == Mitgliedstyp.ALLE)
       suchadresstyp = new SelectInput(new ArrayList<>(), null);
     }
     suchadresstyp.setName("Mitgliedstyp");
-    suchadresstyp.setPleaseChoose("Bitte auswählen");
+    suchadresstyp.setPleaseChoose(ALLE);
     suchadresstyp.addListener(new FilterListener());
     return suchadresstyp;
   }
@@ -318,7 +320,7 @@ public Input getMitgliedArt()
       Logger.error("Fehler beim lesen der Einstellungen");
     }
     art.setName("Mitgliedsart");
-    art.setPleaseChoose("Bitte auswählen");
+    art.setPleaseChoose(ALLE);
     art.addListener(new FilterListener());
     return art;
   }
@@ -548,7 +550,7 @@ public SelectInput getBeitragsgruppeAusw() throws RemoteException
     beitragsgruppeausw = new SelectInput(list != null ? PseudoIterator.asList(list) : null, bg);
     beitragsgruppeausw.setName("Beitragsgruppe");
     beitragsgruppeausw.setAttribute("bezeichnung");
-    beitragsgruppeausw.setPleaseChoose("Bitte auswählen");
+    beitragsgruppeausw.setPleaseChoose(ALLE);
     beitragsgruppeausw.addListener(new FilterListener());
     return beitragsgruppeausw;
   }
@@ -584,7 +586,7 @@ public GeschlechtInput getSuchGeschlecht() throws RemoteException
     suchgeschlecht = new GeschlechtInput(
         settings.getString(settingsprefix + "geschlecht", ""));
     suchgeschlecht.setName("Geschlecht");
-    suchgeschlecht.setPleaseChoose("Bitte auswählen");
+    suchgeschlecht.setPleaseChoose(ALLE);
     suchgeschlecht.addListener(new FilterListener());
     return suchgeschlecht;
   }
@@ -931,7 +933,7 @@ public SelectInput getSuchLehrgangsart() throws RemoteException
       //
     }
     suchlehrgangsart = new SelectInput(it != null ? PseudoIterator.asList(it) : null, letztesuche);
-    suchlehrgangsart.setPleaseChoose("Bitte auswählen");
+    suchlehrgangsart.setPleaseChoose(ALLE);
     suchlehrgangsart.addListener(new FilterListener());
     suchlehrgangsart.setName("Lehrgangsart");
     return suchlehrgangsart;
@@ -1106,8 +1108,8 @@ public SelectInput getSuchStatus() throws RemoteException
     {
       return suchstatus;
     }
-    suchstatus = new SelectInput(new String[] { "Alle", "Ohne Deaktiviert" },
-        settings.getString(settingsprefix + "suchstatus", "Alle"));
+    suchstatus = new SelectInput(new String[] { ALLE, "Ohne Deaktiviert" },
+        settings.getString(settingsprefix + "suchstatus", ALLE));
     suchstatus.addListener(new FilterListener());
     suchstatus.setName("Status");
     return suchstatus;
@@ -1147,7 +1149,7 @@ public SelectInput getSuchBuchungsklasse() throws RemoteException
         list != null ? PseudoIterator.asList(list) : null, bk);
     suchbuchungsklasse.setName("Buchungsklasse");
     suchbuchungsklasse.setAttribute("bezeichnung");
-    suchbuchungsklasse.setPleaseChoose("Bitte auswählen");
+    suchbuchungsklasse.setPleaseChoose(ALLE);
     suchbuchungsklasse.addListener(new FilterListener());
     return suchbuchungsklasse;
   }
@@ -1178,7 +1180,7 @@ public SelectInput getSuchBuchungsartArt() throws RemoteException
     }
     suchbuchungsartart = new SelectInput(ArtBuchungsart.getArray(), artb);
     suchbuchungsartart.setName("Art");
-    suchbuchungsartart.setPleaseChoose("Bitte auswählen");
+    suchbuchungsartart.setPleaseChoose(ALLE);
     suchbuchungsartart.addListener(new FilterListener());
     return suchbuchungsartart;
   }
@@ -1309,7 +1311,7 @@ public void handleAction(Object context) throws ApplicationException
         if (suchspendenart != null)
           suchspendenart.setValue(SuchSpendenart.ALLE);
         if (suchstatus != null)
-          suchstatus.setValue("Alle");
+          suchstatus.setValue(ALLE);
         if (suchbuchungsklasse != null)
           suchbuchungsklasse.setValue(null);
         if (suchbuchungsartart != null)
@@ -1474,7 +1476,7 @@ public void saveFilterSettings() throws RemoteException
     if (art != null)
     {
       String tmp = (String) art.getValue();
-      if (tmp != null && !tmp.equals("Bitte auswählen"))
+      if (tmp != null)
       {
         settings.setAttribute(settingsprefix + "status.art", tmp);
       }
@@ -1539,7 +1541,7 @@ public void saveFilterSettings() throws RemoteException
     if (suchgeschlecht != null)
     {
       String tmp = (String) suchgeschlecht.getValue();
-      if (tmp != null && !getSuchGeschlecht().getText().equals("Bitte auswählen"))
+      if (tmp != null)
       {
         settings.setAttribute(settingsprefix + "geschlecht", tmp);
       }

diff --git a/src/de/jost_net/JVerein/io/MitgliedAuswertungPDF.java b/src/de/jost_net/JVerein/io/MitgliedAuswertungPDF.java
@@ -156,8 +156,7 @@ public void beforeGo() throws RemoteException
         params.put("Mail", control.getMailauswahl().getText());
       }
     }
-    if (control.isSuchGeschlechtAktiv() && control.getSuchGeschlecht().getText() != null
-        && !control.getSuchGeschlecht().getText().equals("Bitte auswählen"))
+    if (control.isSuchGeschlechtAktiv() && control.getSuchGeschlecht().getValue() != null)
     {
       params.put("Geschlecht", control.getSuchGeschlecht().getText());
     }