Zusatzbetrag Endedatum umbenannt

openjverein · Jan 30, 2025 · f49650a · f49650a
1 parent e4e627a
commit f49650a
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 36 deletions.
diff --git a/src/de/jost_net/JVerein/gui/control/ZusatzbetragControl.java b/src/de/jost_net/JVerein/gui/control/ZusatzbetragControl.java
@@ -258,14 +258,15 @@ public Part getZusatzbetraegeList() throws RemoteException
       zusatzbetraegeList = new TablePart(zusatzbetraege,
           new ZusatzbetraegeAction(null));
       zusatzbetraegeList.addColumn("Name", "mitglied");
-      zusatzbetraegeList.addColumn("Startdatum", "startdatum",
+      zusatzbetraegeList.addColumn("Erste Fälligkeit", "startdatum",
           new DateFormatter(new JVDateFormatTTMMJJJJ()));
       zusatzbetraegeList.addColumn("Nächste Fälligkeit", "faelligkeit",
           new DateFormatter(new JVDateFormatTTMMJJJJ()));
-      zusatzbetraegeList.addColumn("Letzte Ausführung", "ausfuehrung",
+      zusatzbetraegeList.addColumn("Letzte abgerechnete Fälligkeit",
+          "ausfuehrung",
           new DateFormatter(new JVDateFormatTTMMJJJJ()));
       zusatzbetraegeList.addColumn("Intervall", "intervalltext");
-      zusatzbetraegeList.addColumn("Endedatum", "endedatum",
+      zusatzbetraegeList.addColumn("Nicht mehr ausführen ab", "endedatum",
           new DateFormatter(new JVDateFormatTTMMJJJJ()));
       zusatzbetraegeList.addColumn("Buchungstext", "buchungstext");
       zusatzbetraegeList.addColumn("Betrag", "betrag",
@@ -423,11 +424,13 @@ public void run(ProgressMonitor monitor) throws ApplicationException
               BaseColor.LIGHT_GRAY);
           reporter.addHeaderColumn("Nächste Fälligkeit", Element.ALIGN_LEFT, 30,
               BaseColor.LIGHT_GRAY);
-          reporter.addHeaderColumn("Letzte Ausführung", Element.ALIGN_LEFT, 30,
+          reporter.addHeaderColumn("Letzte abgerechnete Fälligkeit",
+              Element.ALIGN_LEFT, 30,
               BaseColor.LIGHT_GRAY);
           reporter.addHeaderColumn("Intervall", Element.ALIGN_LEFT, 30,
               BaseColor.LIGHT_GRAY);
-          reporter.addHeaderColumn("Endedatum", Element.ALIGN_LEFT, 30,
+          reporter.addHeaderColumn("Nicht mehr ausführen ab",
+              Element.ALIGN_LEFT, 30,
               BaseColor.LIGHT_GRAY);
           reporter.addHeaderColumn("Buchungstext", Element.ALIGN_LEFT, 50,
               BaseColor.LIGHT_GRAY);

diff --git a/src/de/jost_net/JVerein/gui/menu/ZusatzbetraegeMenu.java b/src/de/jost_net/JVerein/gui/menu/ZusatzbetraegeMenu.java
@@ -16,11 +16,10 @@
  **********************************************************************/
 package de.jost_net.JVerein.gui.menu;
 
-import de.jost_net.JVerein.gui.action.ZusatzbetraegeAction;
-
 import java.rmi.RemoteException;
 
 import de.jost_net.JVerein.gui.action.MitgliedDetailAction;
+import de.jost_net.JVerein.gui.action.ZusatzbetraegeAction;
 import de.jost_net.JVerein.gui.action.ZusatzbetraegeDeleteAction;
 import de.jost_net.JVerein.gui.action.ZusatzbetraegeNaechsteFaelligkeitAction;
 import de.jost_net.JVerein.gui.action.ZusatzbetraegeResetAction;
@@ -78,7 +77,8 @@ public boolean isEnabledFor(Object o)
         Zusatzbetrag z = (Zusatzbetrag) o;
         try
         {
-          return z.getIntervall() == IntervallZusatzzahlung.KEIN ;
+          return z.getIntervall() == IntervallZusatzzahlung.KEIN
+              && z.getAusfuehrung() != null;
         }
         catch (RemoteException e)
         {

diff --git a/src/de/jost_net/JVerein/gui/parts/ZusatzbetragPart.java b/src/de/jost_net/JVerein/gui/parts/ZusatzbetragPart.java
@@ -25,9 +25,9 @@
 
 import de.jost_net.JVerein.Einstellungen;
 import de.jost_net.JVerein.gui.input.BuchungsartInput;
+import de.jost_net.JVerein.gui.input.BuchungsartInput.buchungsarttyp;
 import de.jost_net.JVerein.gui.input.BuchungsklasseInput;
 import de.jost_net.JVerein.gui.input.MitgliedInput;
-import de.jost_net.JVerein.gui.input.BuchungsartInput.buchungsarttyp;
 import de.jost_net.JVerein.keys.IntervallZusatzzahlung;
 import de.jost_net.JVerein.keys.Zahlungsweg;
 import de.jost_net.JVerein.rmi.Buchungsart;
@@ -88,10 +88,10 @@ public void paint(Composite parent) throws RemoteException
     {
       group.addLabelPair("Mitglied", getMitglied());
     }
-    group.addLabelPair("Startdatum", getStartdatum(true));
+    group.addLabelPair("Erste Fälligkeit ", getStartdatum(true));
     group.addLabelPair("Nächste Fälligkeit", getFaelligkeit());
     group.addLabelPair("Intervall", getIntervall());
-    group.addLabelPair("Endedatum", getEndedatum());
+    group.addLabelPair("Nicht mehr ausführen ab", getEndedatum());
     group.addLabelPair("Buchungstext", getBuchungstext());
     group.addLabelPair("Betrag", getBetrag());
     group.addLabelPair("Buchungsart", getBuchungsart());

diff --git a/src/de/jost_net/JVerein/io/AbrechnungSEPA.java b/src/de/jost_net/JVerein/io/AbrechnungSEPA.java
@@ -762,7 +762,7 @@ private void abbuchenZusatzbetraege(AbrechnungSEPAParam param,
             za.setZusatzbetrag(z);
             za.setLetzteAusfuehrung(z.getAusfuehrung());
             za.store();
-            z.setAusfuehrung(Datum.getHeute());
+            z.setAusfuehrung(param.stichtag);
             z.store();
           }
         }

diff --git a/src/de/jost_net/JVerein/server/ZusatzbetragImpl.java b/src/de/jost_net/JVerein/server/ZusatzbetragImpl.java
@@ -341,12 +341,10 @@ public Object getAttribute(String fieldName) throws RemoteException
   @Override
   public boolean isOffen(Date datum) throws RemoteException
   {
-    if (!getMitglied().isAngemeldet(datum))
+    if (!getMitglied().isAngemeldet(datum)
+        && !Einstellungen.getEinstellung().getZusatzbetragAusgetretene())
     {
-      if (!Einstellungen.getEinstellung().getZusatzbetragAusgetretene())
-      {
-        return false;
-      }
+      return false;
     }
     // Einmalige Ausführung
     if (getIntervall().intValue() == IntervallZusatzzahlung.KEIN)
@@ -368,38 +366,26 @@ && getFaelligkeit().getTime() >= getEndedatum().getTime())
   @Override
   public boolean isAktiv(Date datum) throws RemoteException
   {
-    if (!getMitglied().isAngemeldet(datum))
+    if (!getMitglied().isAngemeldet(datum)
+        && !Einstellungen.getEinstellung().getZusatzbetragAusgetretene())
     {
-      if (!Einstellungen.getEinstellung().getZusatzbetragAusgetretene())
-      {
-        return false;
-      }
+      return false;
     }
     // Einmalige Ausführung
     if (getIntervall().intValue() == IntervallZusatzzahlung.KEIN)
     {
       // Ist das Ausführungsdatum gesetzt?
-      if (getAusfuehrung() == null)
+      if (getAusfuehrung() != null)
       {
-        if (getFaelligkeit().getTime() <= datum.getTime())
-        {
-          return true;
-        }
-        else
-        {
-          return false;
-        }
-      }
-      else
-      {
-        // ja: nicht mehr ausführen
         return false;
       }
+      return (getFaelligkeit().getTime() <= datum.getTime());
     }
 
     // Wenn das Endedatum gesetzt ist und das Ausführungsdatum liegt hinter
     // dem Endedatum: nicht mehr ausführen
-    if ((getEndedatum() != null && datum.getTime() >= getEndedatum().getTime())
+    if ((getEndedatum() != null
+        && getFaelligkeit().getTime() >= getEndedatum().getTime())
         || getFaelligkeit().getTime() > datum.getTime())
     {
       return false;