From b2a7bb87ffbc75c753d106c0e6c62b9b2d0654a8 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Jan=20B=C3=BCren?= <jan@lx-office-hosting.de>
Date: Thu, 18 Nov 2010 10:13:58 +0100
Subject: [PATCH] =?utf8?q?Sobald=20ein=20alter=20Zahlungseingang=20in=20ei?=
 =?utf8?q?nem=20abgeschlossen=20Zeitraum=20f=C3=A4llt=20und=20man=20einen?=
 =?utf8?q?=20neuen=20buchen=20m=C3=B6chte=20erh=C3=A4lt=20man,=20dass=20di?=
 =?utf8?q?e=20Buchung=20nicht=20m=C3=B6glich=20ist.=20Entsprechend=20in=20?=
 =?utf8?q?Bug=201502=20dokumentiert=20und=20weiterf=C3=BChrende=20Ideen=20?=
 =?utf8?q?kommentiert?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 bin/mozilla/is.pl | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

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