save_custom_variables zerstört nicht mehr Transaktionssicherheit

z.B. bei post_invoice sollten die Einträge für ar, invoice und acc_trans nur
zusammen in einem Block in die DB geschrieben werden. Dies wurde aber durch ein
$dbh->commit in CVar.pm ausgehebelt, da der übergebene Autocommitparameter in
$params{dbh} nicht berücksichtigt wurde.

diff --git a/SL/CVar.pm b/SL/CVar.pm
index 1b83121..49e3786 100644 (file)
--- a/SL/CVar.pm
+++ b/SL/CVar.pm
@@ -357,7 +357,7 @@ sub save_custom_variables {
 
   $sth->finish();
 
-  $dbh->commit();
+  $dbh->commit() unless $params{dbh};
 
   $main::lxdebug->leave_sub();
 }
@@ -658,7 +658,7 @@ sub save_custom_variables_validity {
 
   $sth->finish();
 
-  $dbh->commit();
+  $dbh->commit() unless $params{dbh};
 
   $main::lxdebug->leave_sub();
 }