korrigiert build error von 497 (#500)

* BuildFix Java11 StringBuilder.length anstelle StringBuilder.isEmpty * fix length check of where clause
openjverein · Nov 27, 2024 · c43096e · c43096e
1 parent 156b97b
commit c43096e
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/src/de/jost_net/JVerein/Queries/SollbuchungQuery.java b/src/de/jost_net/JVerein/Queries/SollbuchungQuery.java
@@ -228,7 +228,7 @@ else if (i == anzahl)
     ArrayList<Object> param = new ArrayList<>();
     if (mitglied != null)
     {
-      where.append(where.isEmpty() ? "" : " AND ")
+      where.append(where.length() == 0 ? "" : " AND ")
           .append("mitgliedskonto.mitglied = ? ");
       param.add(Long.valueOf(mitglied.getID()));
     }
@@ -238,7 +238,7 @@ else if (i == anzahl)
     {
       // Der Name kann so verwendet werden ohne Umwandeln der Umlaute
       String tmpSuchname = (String) control.getSuchname().getValue();
-      where.append(where.isEmpty() ? "" : " AND ")
+      where.append(where.length() == 0 ? "" : " AND ")
           .append("((LOWER(mitglied.name) LIKE ?) OR (LOWER(mitglied.vorname) LIKE ?))");
       param.add(tmpSuchname.toLowerCase() + "%");
       param.add(tmpSuchname.toLowerCase() + "%");
@@ -260,12 +260,12 @@ else if (control.isSuchnameAktiv()
           count++;
           if (anzahl == 1)
           {
-            where.append(where.isEmpty() ? "" : " AND ")
+            where.append(where.length() == 0 ? "" : " AND ")
                 .append("mitgliedskonto.mitglied = ?");
           }
           else if (count == 1)
           {
-            where.append(where.isEmpty() ? "" : " AND ")
+            where.append(where.length() == 0 ? "" : " AND ")
                 .append("(mitgliedskonto.mitglied = ?");
           }
           else if (count < anzahl)
@@ -282,20 +282,20 @@ else if (count == anzahl)
     }
     if (vd != null)
     {
-      where.append(where.isEmpty() ? "" : " AND ")
+      where.append(where.length() == 0 ? "" : " AND ")
           .append("mitgliedskonto.datum >= ?");
       param.add(vd);
     }
     if (bd != null)
     {
-      where.append(where.isEmpty() ? "" : " AND ")
+      where.append(where.length() == 0 ? "" : " AND ")
           .append("mitgliedskonto.datum <= ?");
       param.add(bd);
     }
     if (control.isOhneAbbucherAktiv()
         && (Boolean) control.getOhneAbbucher().getValue())
     {
-      where.append(where.isEmpty() ? "" : " AND ")
+      where.append(where.length() == 0 ? "" : " AND ")
           .append("mitgliedskonto.zahlungsweg <> ?");
       param.add(Zahlungsweg.BASISLASTSCHRIFT);
     }
@@ -304,17 +304,17 @@ else if (count == anzahl)
       int mailauswahl = (Integer) control.getMailauswahl().getValue();
       if (mailauswahl == MailAuswertungInput.OHNE)
       {
-        where.append(where.isEmpty() ? "" : " AND ")
+        where.append(where.length() == 0 ? "" : " AND ")
             .append("(email IS NULL OR LENGTH(email) = 0)");
       }
       if (mailauswahl == MailAuswertungInput.MIT)
       {
-        where.append(where.isEmpty() ? "" : " AND ")
+        where.append(where.length() == 0 ? "" : " AND ")
             .append("(email IS NOT NULL AND LENGTH(email) > 0)");
       }
     }
 
-    if (!where.isEmpty())
+    if (where.length() > 0)
     {
       sql.append(" WHERE ").append(where);
     }