Filter auf Betrag vorzeichenlos (#390)

* Added Betrag to Suchstrategie * Added case for BETRAG Strategy * Added Prefix for BETRAG * Refactored sustrat * Added | to Delimiter * Fixed second Parameter for Betrag Strategy * Fixed Parentheses in DB Query
openjverein · Oct 20, 2024 · 5fd1a88 · 5fd1a88
1 parent 7535a7c
commit 5fd1a88
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 14 deletions.
diff --git a/src/de/jost_net/JVerein/Queries/BuchungQuery.java b/src/de/jost_net/JVerein/Queries/BuchungQuery.java
@@ -254,6 +254,9 @@ else if (buchungart.getNummer() >= 0)
           case KLEINERGLEICH:
             it.addFilter("buchung.betrag <= ?", suchbetrag.getBetrag());
             break;
+          case BETRAG:
+            it.addFilter("(buchung.betrag = ? OR buchung.betrag = ?)", suchbetrag.getBetrag(), suchbetrag.getBetrag().negate());
+            break;
           default:
             break;
         }

diff --git a/src/de/jost_net/JVerein/io/Suchbetrag.java b/src/de/jost_net/JVerein/io/Suchbetrag.java
@@ -29,10 +29,10 @@ public class Suchbetrag
 
   public enum Suchstrategie
   {
-    KEINE, GLEICH, GRÖSSER, GRÖSSERGLEICH, KLEINER, KLEINERGLEICH, BEREICH, UNGÜLTIG
+    KEINE, GLEICH, GRÖSSER, GRÖSSERGLEICH, KLEINER, KLEINERGLEICH, BEREICH, UNGÜLTIG, BETRAG
   }
 
-  private Suchstrategie sustrat = Suchstrategie.KEINE;
+  private Suchstrategie suchstrategie = Suchstrategie.KEINE;
 
   private BigDecimal betrag;
 
@@ -45,13 +45,13 @@ public Suchbetrag(String suchbetrag) throws Exception
   {
     if (suchbetrag == null || suchbetrag.length() == 0)
     {
-      sustrat = Suchstrategie.KEINE;
+      suchstrategie = Suchstrategie.KEINE;
       return;
     }
 
     // Suchstring in Einzelteile zerlegen
     ArrayList<String> liste = new ArrayList<>();
-    StringTokenizer tok = new StringTokenizer(suchbetrag, "<>=.", true);
+    StringTokenizer tok = new StringTokenizer(suchbetrag, "<>=.|", true);
     while (tok.hasMoreTokens())
     {
       liste.add(tok.nextToken().trim());
@@ -76,32 +76,37 @@ else if (liste.get(i).equals(".") && liste.get(i + 1).equals("."))
     }
     if (liste.get(0).equals(">"))
     {
-      sustrat = Suchstrategie.GRÖSSER;
+      suchstrategie = Suchstrategie.GRÖSSER;
       liste.remove(0);
     }
     else if (liste.get(0).equals(">="))
     {
-      sustrat = Suchstrategie.GRÖSSERGLEICH;
+      suchstrategie = Suchstrategie.GRÖSSERGLEICH;
       liste.remove(0);
     }
     else if (liste.get(0).equals("<"))
     {
-      sustrat = Suchstrategie.KLEINER;
+      suchstrategie = Suchstrategie.KLEINER;
       liste.remove(0);
     }
     else if (liste.get(0).equals("<="))
     {
-      sustrat = Suchstrategie.KLEINERGLEICH;
+      suchstrategie = Suchstrategie.KLEINERGLEICH;
       liste.remove(0);
     }
     else if (liste.get(0).equals("="))
     {
-      sustrat = Suchstrategie.GLEICH;
+      suchstrategie = Suchstrategie.GLEICH;
+      liste.remove(0);
+    }
+    else if (liste.get(0).equals("|"))
+    {
+      suchstrategie = Suchstrategie.BETRAG;
       liste.remove(0);
     }
     else if (liste.size() > 1 && liste.get(1).equals(".."))
     {
-      sustrat = Suchstrategie.BEREICH;
+      suchstrategie = Suchstrategie.BEREICH;
       liste.remove(1);
     }
     else if (liste.size() > 1 && !liste.get(1).equals(".."))
@@ -110,7 +115,7 @@ else if (liste.size() > 1 && !liste.get(1).equals(".."))
     }
     else if (liste.size() == 1)
     {
-      sustrat = Suchstrategie.GLEICH; // Nur Betrag angegeben.
+      suchstrategie = Suchstrategie.GLEICH; // Nur Betrag angegeben.
     }
 
     // jetzt muss ein Decimalwert kommen
@@ -119,21 +124,21 @@ else if (liste.size() == 1)
       NumberFormat nf = NumberFormat.getInstance(Locale.GERMAN);
       betrag = new BigDecimal(nf.parse(liste.get(0)).toString());
 
-      if (sustrat == Suchstrategie.BEREICH)
+      if (suchstrategie == Suchstrategie.BEREICH)
       {
         betrag2 = new BigDecimal(nf.parse(liste.get(1)).toString());
       }
     }
     catch (ParseException e)
     {
-      sustrat = Suchstrategie.UNGÜLTIG;
+      suchstrategie = Suchstrategie.UNGÜLTIG;
       throw new Exception("Wert ungültig");
     }
   }
 
   public Suchstrategie getSuchstrategie()
   {
-    return sustrat;
+    return suchstrategie;
   }
 
   public BigDecimal getBetrag()