From: G. Richardson <information@lx-office-hosting.de>
Date: Mon, 23 Jan 2012 10:19:24 +0000 (+0100)
Subject: save_custom_variables zerstört nicht mehr Transaktionssicherheit
X-Git-Tag: release-2.7.0beta1~17^2~3
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=7ecc9c39ec53104441b68ba802e356785d8ca199;p=kivitendo-erp.git

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 1b83121bf..49e378666 100644
--- 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();
 }