Beim Duplizieren von Lastschriften den Ziel-Termin in die Zukunft (mo…

…rgen) verschieben, wenn er sich in der Vergangenheit befindet. Duplizieren schlägt sonst wegen BUGZILLA 1736 fehl
willuhn · Jun 20, 2016 · 05dd7ba · 05dd7ba
1 parent df423b6
commit 05dd7ba
Show file tree

Hide file tree

Showing 5 changed files with 40 additions and 2 deletions.
diff --git a/build/ChangeLog b/build/ChangeLog
@@ -1,5 +1,6 @@
 HEAD 2.7 (nightly)
 
+  * BUG: 0530 Beim Duplizieren von Lastschriften den Ziel-Termin in die Zukunft (morgen) verschieben, wenn er sich in der Vergangenheit befindet. Duplizieren schl�gt sonst wegen BUGZILLA 1736 fehl
   * NEW: 0529 Label "Erinnerungstermin" bei Erfassung einer �berweisung in "Ausf�hrungstermin" �ndern, wenn als Auftragstyp "Bankseitige SEPA-Termin�berweisung" gew�hlt wird
   * NEW: 0528 BUGZILLA 1740 - Beim Aktualisieren eines existierenden Dauerauftrages das Datum der ersten Zahlung automatisch auf den n�chsten Termin setzen
 

diff --git a/src/de/willuhn/jameica/hbci/gui/parts/SepaLastschriftList.java b/src/de/willuhn/jameica/hbci/gui/parts/SepaLastschriftList.java
@@ -49,6 +49,7 @@ protected void initColums()
     addColumn(i18n.tr("Betrag"),"betrag", new CurrencyFormatter(HBCIProperties.CURRENCY_DEFAULT_DE,HBCI.DECIMALFORMAT));
     addColumn(i18n.tr("Termin"),"termin", new DateFormatter(HBCI.DATEFORMAT),false,Column.ALIGN_RIGHT);
     addColumn(new AusgefuehrtColumn());
+    addColumn(i18n.tr("Zieltermin"),"targetdate", new DateFormatter(HBCI.DATEFORMAT),false,Column.ALIGN_RIGHT);
     addColumn(i18n.tr("Art"),"sepatype");
     addColumn(i18n.tr("Sequenz"),"sequencetype");
   }

diff --git a/src/de/willuhn/jameica/hbci/gui/parts/SepaSammelLastschriftList.java b/src/de/willuhn/jameica/hbci/gui/parts/SepaSammelLastschriftList.java
@@ -11,6 +11,9 @@
 
 import de.willuhn.jameica.gui.Action;
 import de.willuhn.jameica.gui.Part;
+import de.willuhn.jameica.gui.formatter.DateFormatter;
+import de.willuhn.jameica.gui.parts.Column;
+import de.willuhn.jameica.hbci.HBCI;
 import de.willuhn.jameica.hbci.rmi.SepaSammelLastschrift;
 
 /**
@@ -26,6 +29,7 @@ public class SepaSammelLastschriftList extends AbstractSepaSammelTransferList im
   public SepaSammelLastschriftList(Action action) throws RemoteException
   {
     super(action);
+    addColumn(i18n.tr("Zieltermin"),"targetdate", new DateFormatter(HBCI.DATEFORMAT),false,Column.ALIGN_RIGHT);
     addColumn(i18n.tr("Art"),"sepatype");
     addColumn(i18n.tr("Sequenz"),"sequencetype");
     setContextMenu(new de.willuhn.jameica.hbci.gui.menus.SepaSammelLastschriftList());

diff --git a/src/de/willuhn/jameica/hbci/server/SepaLastschriftImpl.java b/src/de/willuhn/jameica/hbci/server/SepaLastschriftImpl.java
@@ -65,10 +65,26 @@ public Duplicatable duplicate() throws RemoteException {
     u.setSignatureDate(getSignatureDate());
     u.setCreditorId(getCreditorId());
     u.setSequenceType(getSequenceType());
-    u.setTargetDate(getTargetDate());
     u.setType(getType());
     u.setOrderId(getOrderId());
     u.setPurposeCode(getPurposeCode());
+
+    // Wenn sich das Target-Date in der Vergangenheit befindet, muessen wir ein neues erzeugen.
+    // Andernfalls wuerde das Speichern fehlschlagen, weil bei insertCheck geprueft wird, ob sich
+    // das Ziel-Datum in der Zukunft befindet
+    Date target = this.getTargetDate();
+    Date now = new Date();
+    if (target != null && !target.after(now))
+    {
+      // Wir nehmen morgen.
+      target = DateUtil.endOfDay(new Date(now.getTime() + (24 * 60 * 60 * 1000L)));
+      u.setTargetDate(target);
+    }
+    else
+    {
+      u.setTargetDate(target);
+    }
+
     return u;
   }
 

diff --git a/src/de/willuhn/jameica/hbci/server/SepaSammelLastschriftImpl.java b/src/de/willuhn/jameica/hbci/server/SepaSammelLastschriftImpl.java
@@ -201,10 +201,26 @@ public Duplicatable duplicate() throws RemoteException
       l.setKonto(this.getKonto());
       l.setTermin(new Date());
       l.setSequenceType(getSequenceType());
-      l.setTargetDate(getTargetDate());
       l.setType(getType());
       l.setOrderId(getOrderId());
       l.setPmtInfId(getPmtInfId());
+
+      // Wenn sich das Target-Date in der Vergangenheit befindet, muessen wir ein neues erzeugen.
+      // Andernfalls wuerde das Speichern fehlschlagen, weil bei insertCheck geprueft wird, ob sich
+      // das Ziel-Datum in der Zukunft befindet
+      Date target = this.getTargetDate();
+      Date now = new Date();
+      if (target != null && target.before(new Date()))
+      {
+        // Wir nehmen morgen.
+        target = DateUtil.endOfDay(new Date(now.getTime() + (24 * 60 * 60 * 1000L)));
+        l.setTargetDate(target);
+      }
+      else
+      {
+        l.setTargetDate(target);
+      }
+
       l.store();
 
       List<SepaSammelLastBuchung> list = this.getBuchungen();