Fix Sepa Check

src/de/jost_net/JVerein/gui/action/MitgliedLastschriftAction.java

@@ -122,7 +122,6 @@ private boolean checkSEPA(Mitglied m) throws RemoteException
     if (m.getZahlungsweg() == null
         || m.getZahlungsweg() != Zahlungsweg.BASISLASTSCHRIFT)
     {
-
       abortDialog("Fehler", "Zahlungsweg ist nicht Basislastschrift");
       return false;
     }
@@ -136,22 +135,24 @@ private boolean checkSEPA(Mitglied m) throws RemoteException
       }
     }
 
-    // pruefe Sepa Gueltigkeit: Datum der letzen Abbuchung
-    Date letzte_lastschrift = m.getLetzteLastschrift();
-    if (letzte_lastschrift != null)
+    // Pruefe Sepa Gueltigkeit:
+    // Bei Mandaten älter als 3 Jahre muss es eine Lastschrift
+    // innerhalb der letzten 3 Jahre geben
+    Calendar sepagueltigkeit = Calendar.getInstance();
+    sepagueltigkeit.add(Calendar.MONTH, -36);
+    if (m.getMandatDatum().before(sepagueltigkeit.getTime()))
     {
-      Calendar sepagueltigkeit = Calendar.getInstance();
-      sepagueltigkeit.add(Calendar.MONTH, -36);
-      if (letzte_lastschrift.before(sepagueltigkeit.getTime()))
+      Date letzte_lastschrift = m.getLetzteLastschrift();
+      if (letzte_lastschrift == null
+          || letzte_lastschrift.before(sepagueltigkeit.getTime()))
       {
-        if (!confirmDialog("Letzte Lastschrift",
-            "Letzte Lastschrift ist älter als 36 Monate"))
+        if (!confirmDialog("Mandat abgelaufen",
+            "Mandat-Datum ist älter als 36 Monate und keine Lastschrift in den letzten 36 Monaten"))
         {
           return false;
         }
-        }
+      }
     }
-
     return true;
   }
 

src/de/jost_net/JVerein/io/AbrechnungSEPA.java

@@ -981,20 +981,26 @@ private boolean checkSEPA(Mitglied m, ProgressMonitor monitor)
     {
       return true;
     }
-    Date letzte_lastschrift = m.getLetzteLastschrift();
-    if (letzte_lastschrift != null
-        && letzte_lastschrift.before(sepagueltigkeit.getTime()))
-    {
-      monitor.log(Adressaufbereitung.getNameVorname(m)
-          + ": Letzte Lastschrift ist älter als 36 Monate.");
-      return false;
-    }
+    // Ohne Mandat keine Lastschrift
     if (m.getMandatDatum() == Einstellungen.NODATE)
     {
       monitor.log(Adressaufbereitung.getNameVorname(m)
           + ": Kein Mandat-Datum vorhanden.");
       return false;
     }
+    // Bei Mandaten älter als 3 Jahre muss es eine Lastschrift
+    // innerhalb der letzten 3 Jahre geben
+    if (m.getMandatDatum().before(sepagueltigkeit.getTime()))
+    {
+      Date letzte_lastschrift = m.getLetzteLastschrift();
+      if (letzte_lastschrift == null
+          || letzte_lastschrift.before(sepagueltigkeit.getTime()))
+      {
+        monitor.log(Adressaufbereitung.getNameVorname(m)
+            + ": Mandat-Datum ist älter als 36 Monate und keine Lastschrift in den letzten 36 Monaten");
+        return false;
+      }
+    }
     return true;
   }