lainwir3d/dolibarr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Clean code

Browse Source
This commit is contained in:
Laurent Destailleur 2021-09-30 14:51:38 +02:00
parent 9460ad152e
commit aaf170313f
1 changed files with 12 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
htdocs/product/class/product.class.php
View File

@ -4036,7 +4036,7 @@ class Product extends CommonObject
$rank = $obj->max_rank + 1; $rank = $obj->max_rank + 1;
//Addition of a product with the highest rank +1 //Addition of a product with the highest rank +1
$sql = "INSERT INTO ".MAIN_DB_PREFIX."product_association(fk_product_pere,fk_product_fils,qty,incdec,rang)"; $sql = "INSERT INTO ".MAIN_DB_PREFIX."product_association(fk_product_pere,fk_product_fils,qty,incdec,rang)";
$sql .= " VALUES (".((int) $id_pere).", ".((int) $id_fils).", ".((float) $qty).", ".((float) $incdec).", ".$this->db->escape($rank).")"; $sql .= " VALUES (".((int) $id_pere).", ".((int) $id_fils).", ".price2num($qty, 'MS').", ".price2num($incdec, 'MS').", ".((int) $rank).")";
if (! $this->db->query($sql)) { if (! $this->db->query($sql)) {
dol_print_error($this->db); dol_print_error($this->db);
return -1; return -1;
@ -4092,10 +4092,10 @@ class Product extends CommonObject
// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
/** /**
* Retire le lien entre un sousproduit et un produit/service * Remove a link between a subproduct and a parent product/service
* *
* @param int $fk_parent Id du produit auquel ne sera plus lie le produit lie * @param int $fk_parent Id of parent product (child will no more be linked to it)
* @param int $fk_child Id du produit a ne plus lie * @param int $fk_child Id of child product
* @return int < 0 if KO, > 0 if OK * @return int < 0 if KO, > 0 if OK
*/ */
public function del_sousproduit($fk_parent, $fk_child) public function del_sousproduit($fk_parent, $fk_child)
@ -4119,17 +4119,17 @@ class Product extends CommonObject
} }
// Updated ranks so that none are missing // Updated ranks so that none are missing
$sqlrank = 'SELECT rowid, rang FROM '.MAIN_DB_PREFIX.'product_association'; $sqlrank = "SELECT rowid, rang FROM ".MAIN_DB_PREFIX."product_association";
$sqlrank.= ' WHERE fk_product_pere = '.$this->db->escape($fk_parent); $sqlrank.= " WHERE fk_product_pere = ".((int) $fk_parent);
$sqlrank.= ' ORDER BY rang'; $sqlrank.= " ORDER BY rang";
$resqlrank = $this->db->query($sqlrank); $resqlrank = $this->db->query($sqlrank);
if ($resqlrank) { if ($resqlrank) {
$cpt = 0; $cpt = 0;
while ($objrank = $this->db->fetch_object($resqlrank)) { while ($objrank = $this->db->fetch_object($resqlrank)) {
$cpt++; $cpt++;
$sql = 'UPDATE '.MAIN_DB_PREFIX.'product_association'; $sql = "UPDATE ".MAIN_DB_PREFIX."product_association";
$sql.= ' SET rang ='.$cpt; $sql.= " SET rang = ".((int) $cpt);
$sql.= ' WHERE rowid ='.$this->db->escape($objrank->rowid); $sql.= " WHERE rowid = ".((int) $objrank->rowid);
if (! $this->db->query($sql)) { if (! $this->db->query($sql)) {
dol_print_error($this->db); dol_print_error($this->db);
return -1; return -1;