Unterstützung #411
Massenerstellen Rechnungen aus Lieferscheinen: Pflege-Commit verloren gegangen
Status:
Gelöst
Priorität:
Normal
Zugewiesen an:
-
Zielversion:
-
Beginn:
10.12.2019
Abgabedatum:
% erledigt:
100%
Geschätzter Aufwand:
Beschreibung
Im e5a76ed158219e3d2b19f142213f8a80c11627f1 hat Moritz einige Code-Verbesserungen in SL/BackgroundJob/MassRecordCreationAndPrinting vorgenommen.
Die wurden in einem anderen 54ce51443886623bb785e7c54492ecbc8dc9b3aa dann später leider wieder überschrieben.
Die Änderungen von Moritz sollten wieder eingespielt werden.
Zugehörige Revisionen
Historie
Von Bernd Bleßmann vor fast 5 Jahren aktualisiert
- Thema wurde von Massenerstellen Rechnungen aus Lieferscheinen: Plege-Commit verloren gegangen zu Massenerstellen Rechnungen aus Lieferscheinen: Pflege-Commit verloren gegangen geändert
Von Moritz Bunkus vor fast 5 Jahren aktualisiert
- Status wurde von Neu zu Gelöst geändert
- % erledigt wurde von 0 zu 100 geändert
Status geändert durch Changeset kivitendo|27a1f59580ab25940096c9a479b760b9f2cfd8b5.
Rechnungsmassenerstellen: keine DB-Transaktion um convert_to_invoice()
convert_to_invoice() macht selber intern bereits eine Transaktion auf;
daher ist die außen unnötig.
Weiterhin waren die Parameter merkwürdig übergeben:
• Für eine On-The-Fly-Unterscheidung für »diese Parameter« vs. »keine
Parameter« benötigt man keine sub; das geht einfach mit einem ternären
Operator.
• »Keine Parameter« drückt man nicht durch »undef« aus, sondern durch
die leere Liste »()«. Wird »undef« als einziger Parameter übergeben,
so wird in der Funktion de Facto das hier gemacht:
(cherry picked from commit e5a76ed158219e3d2b19f142213f8a80c11627f1)
Cherry pick aus dem master selber, da dieser commit durch
54ce51443886623bb785e7c54492ecbc8dc9b3aa "Dateimanagement: Massendruck"
überschrieben wurde.
Behebt #411 (redmine).