wiederkehrende Rechnungen: automatischer Druck enthält nicht alle Variablen

[bibi@…]

Beim automatischen Drucken von wiederkehrenden Rechnungen werden z.B: folgende Vorlagen-Variablen nicht berücksichtigt:

alle template_meta.*
Variablen für jeden Posten auf der Rechnung:
currency
drawing, ean, image, make, model, weight

Angehängter diff (bug_2247_auto_print_part1.diff) behebt einen Teil des Bugs.

Der Patch bug_2247_auto_print_part2.diff setzt $form->{currency} = $form->{curr} in FlattenToForm?. Damit erscheint dann auch currency im Ausdruck.

Äh.... dein part1 ist erst einmal eine NOOP -- weil in der existierenden unstable sämtliche Aufrufe von prepare_parts_for_printing ohne Parameter erfolgen. Daher gehe ich einfach mal davon aus, dass du in kundenspezifischem oder noch nicht in die unstable gepushtem Code die Funktion mit entsprechenden Parametern aufrufst. part1 tut aber nicht weh und kann dementsprechend committet werden.

Bei part2 hätte ich höchstens zu bemängeln, dass curr weiter oben erhalten bleibt. Andererseits kann ich nicht sagen, ob curr nicht doch in irgend einer Situation benötigt wird, nachdem das Objekt durch flatten_to_form laufen gelassen wurde. Insofern: so lassen.

Summary: beide so committen.

Du hast Recht. Bei part1 habe ich tatsächlich vergessen, die Änderung in SL/IS.pm mit anzuhängen:

IC->prepare_parts_for_printing();

wird zu

IC->prepare_parts_for_printing(myconfig => $myconfig, form => $form);

Da wäre noch die Frage, ob das in OE.pm und DO.pm (order_details) auch gemacht werden soll. Zumindest OE::order_details bekommt myconfig und form auch übergeben.

In 79ae46a435c65fa188981f6b5c61e0e15d32fb23/erp:

FlattenToForm?: auch currency (nicht nur curr) in die Form bringen.

Betrifft #2247.

Es lohnt sich nur und ausschließlich dann, $form und $myconfig an Funktionen explizit zu übergeben, wenn jemals die Chance bestehen sollte, dass sie sich von den globalen Variablen $::form und %::myconfig unterscheiden. Im Allgemeinen ist das bei Code in den alten Modulen in SL/*.pm nicht der Fall.

In der Tat ist es so, dass die Funktion auch auf lokalen Kopien arbeiten können muss (von SL/BackgroundJob/CreatePeriodicInvoice.pm über prepare_for_printing in SL/Form.pm). Insofern sollte das auch auf order_details ausgedehnt werden, ja.

In c5651754689f5072d6f9896af9c85312128d6e18/erp:

OE::, DO::order_details und IC::prepare_parts_for_printing mit lokaler form ...

und myconfig arbeiten lassen
Betrifft #2247.

Mit den commits 79ae46a435c65fa188981f6b5c61e0e15d32fb23/erp und c5651754689f5072d6f9896af9c85312128d6e18/erp ist alles bis auf die fehlenden template_meta.* behoben. Diese werden beim "normalen" Drucken in bin/mozilla/io.pl (sub print_form) gesetzt.
Evtl. ist es sinnvoll, print_form und prepare_for_printing in SL/Form.pm irgendwie zu verheiraten, da da vieles gleich oder ähnlich ist.

In 79ae46a435c65fa188981f6b5c61e0e15d32fb23/erp:

FlattenToForm?: auch currency (nicht nur curr) in die Form bringen.

Betrifft #2247.

In c5651754689f5072d6f9896af9c85312128d6e18/erp:

OE::, DO::order_details und IC::prepare_parts_for_printing mit lokaler form ...

und myconfig arbeiten lassen
Betrifft #2247.

In 142628e0282159c5c7f06b8cd6ec1c7707a69e55/erp:

Form->prepare_for_printing: template_meta zur Verfügung stellen

Behebt #2247.