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/gitweb/gitweb.cgi/mfinanz.git/commitdiff_plain/b2a7bb87ffbc75c753d106c0e6c62b9b2d0654a8

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};