diff --git a/htdocs/commande/commande.class.php b/htdocs/commande/commande.class.php index 3c44014fa51..8ad0b5c34af 100644 --- a/htdocs/commande/commande.class.php +++ b/htdocs/commande/commande.class.php @@ -1163,20 +1163,7 @@ class Commande extends CommonObject if ($ligne->fk_product_type == 0) $nb++; } return $nb; - } - - /** - * - * - */ - function fetch_adresse_livraison($id) - { - $idadresse = $id; - $adresse = new Societe($this->db); - $adresse->fetch_adresse_livraison($idadresse); - $this->adresse = $adresse; - } - + } /** * \brief Charge tableau avec les expéditions par ligne @@ -1303,8 +1290,8 @@ class Commande extends CommonObject { if ($this->statut == 0) { - $this->db->begin(); - + $this->db->begin(); + $sql = "SELECT fk_product, qty"; $sql.= " FROM ".MAIN_DB_PREFIX."commandedet"; $sql.= " WHERE rowid = '$idligne'"; @@ -1313,59 +1300,59 @@ class Commande extends CommonObject if ($result) { $obj = $this->db->fetch_object($result); - - if ($obj) + + if ($obj) { $product = new Product($this->db); $product->id = $obj->fk_product; - + $result=$product->ajust_stock_commande($obj->qty, 1); - - // Supprime ligne + + // Supprime ligne $ligne = new CommandeLigne($this->db); $ligne->id = $idligne; $ligne->fk_commande = $this->id; // On en a besoin dans les triggers $result=$ligne->delete(); - - if ($result > 0) + + if ($result > 0) { - $result=$this->update_price(); - - if ($result > 0) - { - $this->db->commit(); - return 1; - } - else - { - $this->db->rollback(); - $this->error=$this->db->lasterror(); - return -1; - } - } - else - { - $this->db->rollback(); - $this->error=$this->db->lasterror(); - return -1; + $result=$this->update_price(); + + if ($result > 0) + { + $this->db->commit(); + return 1; + } + else + { + $this->db->rollback(); + $this->error=$this->db->lasterror(); + return -1; + } } - } - else - { - $this->db->rollback(); - return 0; - } - } - else - { - $this->db->rollback(); - $this->error=$this->db->lasterror(); - return -1; + else + { + $this->db->rollback(); + $this->error=$this->db->lasterror(); + return -1; + } + } + else + { + $this->db->rollback(); + return 0; + } } - } - else - { - return -1; + else + { + $this->db->rollback(); + $this->error=$this->db->lasterror(); + return -1; + } + } + else + { + return -1; } } @@ -1908,7 +1895,7 @@ class Commande extends CommonObject { global $conf, $lang; - $err = 0; + $err = 0; $this->db->begin(); @@ -2411,9 +2398,9 @@ class CommandeLigne global $langs, $conf, $user; $sql = 'DELETE FROM '.MAIN_DB_PREFIX."commandedet WHERE rowid='".$this->id."';"; - - dolibarr_syslog("CommandeLigne::delete sql=".$sql); - $resql=$this->db->query($sql); + + dolibarr_syslog("CommandeLigne::delete sql=".$sql); + $resql=$this->db->query($sql); if ($resql) { // Appel des triggers @@ -2425,7 +2412,7 @@ class CommandeLigne return 1; } else - { + { $this->error=$this->db->lasterror(); dolibarr_syslog("CommandeLigne::delete ".$this->error); return -1; diff --git a/htdocs/commonobject.class.php b/htdocs/commonobject.class.php index 2fbe318e4ac..0afb5f56b9d 100644 --- a/htdocs/commonobject.class.php +++ b/htdocs/commonobject.class.php @@ -363,6 +363,17 @@ class CommonObject $user->fetch(); $this->user = $user; } + + /** + * \brief Charge l'adresse de livraison d'id $this->adresse_livraison_id dans this->adresse + * \param userid Id du contact + */ + function fetch_adresse_livraison($deliveryaddressid) + { + $address = new Societe($this->db); + $address->fetch_adresse_livraison($deliveryaddressid); + $this->deliveryaddress = $address; + } } diff --git a/htdocs/propal.class.php b/htdocs/propal.class.php index 88cd81765e4..23abbf599ab 100644 --- a/htdocs/propal.class.php +++ b/htdocs/propal.class.php @@ -427,20 +427,6 @@ class Propal extends CommonObject } } - - /** - * - * - */ - function fetch_adresse_livraison($id) - { - $idadresse = $id; - $adresse = new Societe($this->db); - $adresse->fetch_adresse_livraison($idadresse); - $this->adresse = $adresse; - } - - /** * \brief Supprime une ligne de detail * \param idligne Id de la ligne detail à supprimer