Projekt

Allgemein

Profil

« Zurück | Weiter » 

Revision e5a76ed1

Von Moritz Bunkus vor etwa 2 Jahren hinzugefügt

  • ID e5a76ed158219e3d2b19f142213f8a80c11627f1
  • Vorgänger 2d43271a
  • Nachfolger 96670fe8

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:

my %hash = (undef);
und das ergibt eine Warnung, dass eine Liste mit ungerader Anzahl von
Elementen in ein Hash umgewandelt werden soll.

Dateien

  • hinzugefügt
  • geändert
  • kopiert
  • umbenannt
  • gelöscht

Unterschiede anzeigen