Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision c6d1bb55

Von Jan Büren vor mehr als 13 Jahren hinzugefügt

  • ID c6d1bb55635333f190afa2daed451761352873e1
  • Vorgänger 4b4c57c4
  • Nachfolger 9434c14a

Teilweiser Bugfix zu 1575
Bei einem Kunden ist ein Kundentyp mit Rabatt hinterlegt.

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.

Unterschiede anzeigen:

bin/mozilla/do.pl
337 337
  $payment_id = $form->{payment_id} if $form->{payment_id};
338 338

  
339 339
  check_name($form->{vc});
340

  
340
  $form->{discount} =  $form->{"$form->{vc}_discount"} if defined $form->{"$form->{vc}_discount"};
341
  # Problem: Wenn man ohne Erneuern einen Kunden/Lieferanten
342
  # wechselt, wird der entsprechende Kunden/ Lieferantenrabatt
343
  # nicht übernommen. Grundproblem: In Commit 82574e78
344
  # hab ich aus discount customer_discount und vendor_discount
345
  # gemacht und entsprechend an den Oberflächen richtig hin-
346
  # geschoben. Die damals bessere Lösung wäre gewesen:
347
  # In den Templates nur die hidden für form-discount wieder ein-
348
  # setzen dann wäre die Verrenkung jetzt nicht notwendig.
349
  # TODO: Ggf. Bugfix 1284, 1575 und 817 wieder zusammenführen
350
  # Testfälle: Kunden mit Rabatt 0 -> Rabatt 20 i.O.
351
  #            Kunde mit Rabatt 20 -> Rabatt 0  i.O.
352
  #            Kunde mit Rabatt 20 -> Rabatt 5,5 i.O.
341 353
  $form->{payment_id} = $payment_id if $form->{payment_id} eq "";
342 354

  
343 355
  # for pricegroups

Auch abrufbar als: Unified diff