From: Jan Büren <jan@lx-office-hosting.de>
Date: Thu, 18 Nov 2010 09:13:58 +0000 (+0100)
Subject: Sobald ein alter Zahlungseingang in einem abgeschlossen Zeitraum fällt und man
X-Git-Tag: release-2.6.2beta1~90
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=b2a7bb87ffbc75c753d106c0e6c62b9b2d0654a8;p=kivitendo-erp.git

Sobald ein alter Zahlungseingang in einem abgeschlossen Zeitraum fällt und man
einen neuen buchen möchte erhält man, dass die Buchung nicht möglich ist. Entsprechend in Bug 1502 dokumentiert und weiterführende Ideen kommentiert
---

diff --git a/bin/mozilla/is.pl b/bin/mozilla/is.pl
index 450d0b613..9ec9becb8 100644
--- a/bin/mozilla/is.pl
+++ b/bin/mozilla/is.pl
@@ -607,8 +607,6 @@ sub post_payment {
 
       $form->isblank("datepaid_$i", $locale->text('Payment date missing!'));
 
-      $form->error($locale->text('Cannot post payment for a closed period!'))
-        if ($form->date_closed($form->{"datepaid_$i"}, \%myconfig));
 
       if ($form->{currency} ne $form->{defaultcurrency}) {
         $form->{"exchangerate_$i"} = $form->{exchangerate}
@@ -618,6 +616,18 @@ sub post_payment {
       }
     }
   }
+  # Abgeschlossene Zeiträume nur für den letzten (aktuellen) Zahlungseingang prüfen
+  # Details s.a. Bug 1502
+  # Das Problem ist jetzt, dass man Zahlungseingänge nachträglich ändern kann
+  # Wobei dies für Installationen die sowieso nicht mit Bücherkontrolle arbeiten keinen
+  # keinen Unterschied macht.
+  # Optimal wäre, wenn gegen einen Zeitstempel des Zahlungsfelds geprüft würde ...
+  # Das Problem hierbei ist, dass in IS.pm post_invoice IMMER alle Zahlungseingänge aus $form
+  # erneut gespeichert werden. Prinzipiell wäre es besser NUR die Änderungen des Rechnungs-
+  # belegs (neue Zahlung aber nichts anderes) zu speichern ...
+  # Vielleicht könnte man ähnlich wie bei Rechnung löschen verfahren 
+  $form->error($locale->text('Cannot post payment for a closed period!'))
+    if ($form->date_closed($form->{"datepaid_$form->{paidaccounts}"}, \%myconfig));
 
   ($form->{AR})      = split /--/, $form->{AR};
   ($form->{AR_paid}) = split /--/, $form->{AR_paid};