From c6d1bb55635333f190afa2daed451761352873e1 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jan=20B=C3=BCren?= Date: Fri, 13 May 2011 12:53:48 +0200 Subject: [PATCH] Teilweiser Bugfix zu 1575 Bei einem Kunden ist ein Kundentyp mit Rabatt hinterlegt. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Wird der Workflow Lieferschein -> Rechnung durchgeführt, wird der Rabatt nicht berücksichtigt, sondern der normale Preis berechnet. Auch eine zugeordnete Preisgruppe wird nicht verwendet. Wenn der Kunde per Auswahl OHNE erneuern des Lieferscheins geändert wird, wird der vorhergehende Kundenrabatt anstatt der Kundenrabatts des aktuellen Kunden genommen. OFFEN: Preisgruppe geht verloren > Auftrag -> Lieferschein -> Rechnung. Hab ich auch nochmal getestet, die Preisgruppe geht in der Tat verloren, obwohl es hierfür hidden Felder in Lieferschein gibt. --- bin/mozilla/do.pl | 14 +++++++++++++- doc/changelog | 1 + 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/bin/mozilla/do.pl b/bin/mozilla/do.pl index 10fdf2227..1b7a10960 100644 --- a/bin/mozilla/do.pl +++ b/bin/mozilla/do.pl @@ -337,7 +337,19 @@ sub update_delivery_order { $payment_id = $form->{payment_id} if $form->{payment_id}; check_name($form->{vc}); - + $form->{discount} = $form->{"$form->{vc}_discount"} if defined $form->{"$form->{vc}_discount"}; + # Problem: Wenn man ohne Erneuern einen Kunden/Lieferanten + # wechselt, wird der entsprechende Kunden/ Lieferantenrabatt + # nicht übernommen. Grundproblem: In Commit 82574e78 + # hab ich aus discount customer_discount und vendor_discount + # gemacht und entsprechend an den Oberflächen richtig hin- + # geschoben. Die damals bessere Lösung wäre gewesen: + # In den Templates nur die hidden für form-discount wieder ein- + # setzen dann wäre die Verrenkung jetzt nicht notwendig. + # TODO: Ggf. Bugfix 1284, 1575 und 817 wieder zusammenführen + # Testfälle: Kunden mit Rabatt 0 -> Rabatt 20 i.O. + # Kunde mit Rabatt 20 -> Rabatt 0 i.O. + # Kunde mit Rabatt 20 -> Rabatt 5,5 i.O. $form->{payment_id} = $payment_id if $form->{payment_id} eq ""; # for pricegroups diff --git a/doc/changelog b/doc/changelog index 338528c98..7f9d45062 100644 --- a/doc/changelog +++ b/doc/changelog @@ -29,6 +29,7 @@ - Bugfix 1569: Zahlungseingang löschen bei Rechnung - Bugfix 1632: Nach Installation 2.6.2-0 aus .deb funktioniert CSV-Import nicht - Bugfix 1633: Stammdatenaufruf aus SEPA-Modul + - Bugfix 1575: Kundenrabatt geht verloren 2011-02-02 - Release 2.6.2 -- 2.20.1