From b32ffdbbb940095af1152a4d8b6d1ff2d48f5d01 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Fri, 19 Oct 2018 14:59:00 +0200 Subject: [PATCH] Clean code --- .../compta/paiement/cheque/class/remisecheque.class.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/htdocs/compta/paiement/cheque/class/remisecheque.class.php b/htdocs/compta/paiement/cheque/class/remisecheque.class.php index ade6895ac41..64d0f8601fa 100644 --- a/htdocs/compta/paiement/cheque/class/remisecheque.class.php +++ b/htdocs/compta/paiement/cheque/class/remisecheque.class.php @@ -152,6 +152,8 @@ class RemiseCheque extends CommonObject $now=dol_now(); + dol_syslog("RemiseCheque::Create start", LOG_DEBUG); + $this->db->begin(); $sql = "INSERT INTO ".MAIN_DB_PREFIX."bordereau_cheque ("; @@ -178,7 +180,6 @@ class RemiseCheque extends CommonObject $sql.= ", ''"; $sql.= ")"; - dol_syslog("RemiseCheque::Create", LOG_DEBUG); $resql = $this->db->query($sql); if ( $resql ) { @@ -195,7 +196,6 @@ class RemiseCheque extends CommonObject $sql.= " SET ref='(PROV".$this->id.")'"; $sql.= " WHERE rowid=".$this->id.""; - dol_syslog("RemiseCheque::Create", LOG_DEBUG); $resql = $this->db->query($sql); if (! $resql) { @@ -242,13 +242,12 @@ class RemiseCheque extends CommonObject if($linetoremise==$lineid) $checkremise=true; } - if($checkremise==true) + if ($checkremise) { $sql = "UPDATE ".MAIN_DB_PREFIX."bank"; $sql.= " SET fk_bordereau = ".$this->id; $sql.= " WHERE rowid = ".$lineid; - dol_syslog("RemiseCheque::Create", LOG_DEBUG); $resql = $this->db->query($sql); if (!$resql) { @@ -284,11 +283,13 @@ class RemiseCheque extends CommonObject if (! $this->errno) { $this->db->commit(); + dol_syslog("RemiseCheque::Create end", LOG_DEBUG); return $this->id; } else { $this->db->rollback(); + dol_syslog("RemiseCheque::Create end", LOG_DEBUG); return $this->errno; } }