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

Fixing style errors.

Browse Source
This commit is contained in:
stickler-ci 2020-08-13 09:48:28 +00:00
parent 2405082bb1
commit 959ccce777
1 changed files with 36 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

73
htdocs/variants/class/ProductCombination.class.php
View File

@ -141,13 +141,13 @@ class ProductCombination
global $conf; global $conf;
// Check cache // Check cache
if(!empty($this->combination_price_levels) && $useCache){ if (!empty($this->combination_price_levels) && $useCache){
if((!empty($fk_price_level) && isset($this->combination_price_levels[$fk_price_level])) || empty($fk_price_level)){ if ((!empty($fk_price_level) && isset($this->combination_price_levels[$fk_price_level])) || empty($fk_price_level)){
return 1; return 1;
} }
} }
if(!is_array($this->combination_price_levels) if (!is_array($this->combination_price_levels)
|| empty($fk_price_level) // if fetch an unique level dont erase all already fetched || empty($fk_price_level) // if fetch an unique level dont erase all already fetched
){ ){
$this->combination_price_levels = array(); $this->combination_price_levels = array();
@ -160,7 +160,7 @@ class ProductCombination
return -1; return -1;
} }
if(empty($combination_price_levels)){ if (empty($combination_price_levels)){
/** /**
* for auto retrocompatibility with last behavior * for auto retrocompatibility with last behavior
@ -171,10 +171,10 @@ class ProductCombination
$productCombinationLevel->variation_price = $this->variation_price; $productCombinationLevel->variation_price = $this->variation_price;
$productCombinationLevel->variation_price_percentage = $this->variation_price_percentage; $productCombinationLevel->variation_price_percentage = $this->variation_price_percentage;
if($fk_price_level>0){ if ($fk_price_level>0){
$combination_price_levels[$fk_price_level] = $productCombinationLevel; $combination_price_levels[$fk_price_level] = $productCombinationLevel;
} }
else{ else {
for ($i = 1; $i <= $conf->global->PRODUIT_MULTIPRICES_LIMIT; $i++){ for ($i = 1; $i <= $conf->global->PRODUIT_MULTIPRICES_LIMIT; $i++){
$combination_price_levels[$i] = $productCombinationLevel; $combination_price_levels[$i] = $productCombinationLevel;
} }
@ -201,15 +201,15 @@ class ProductCombination
$staticProductCombinationLevel = new ProductCombinationLevel($this->db); $staticProductCombinationLevel = new ProductCombinationLevel($this->db);
// Delete all // Delete all
if(empty($this->combination_price_levels)){ if (empty($this->combination_price_levels)){
return $staticProductCombinationLevel->deleteAllForCombination($this->id); return $staticProductCombinationLevel->deleteAllForCombination($this->id);
} }
// Clean not needed price levels // Clean not needed price levels
if($clean){ if ($clean){
$res = $staticProductCombinationLevel->clean($this->id); $res = $staticProductCombinationLevel->clean($this->id);
if($res<0){ if ($res<0){
$this->errors[] = 'Fail to clean not needed price levels'; $this->errors[] = 'Fail to clean not needed price levels';
return -1; return -1;
} }
@ -217,17 +217,17 @@ class ProductCombination
foreach ($this->combination_price_levels as $fk_price_level => $combination_price_level){ foreach ($this->combination_price_levels as $fk_price_level => $combination_price_level){
$res = $combination_price_level->save(); $res = $combination_price_level->save();
if($res<1){ if ($res<1){
$this->error = 'save combination price level '.$fk_price_level . ' '.$combination_price_level->error; $this->error = 'save combination price level '.$fk_price_level . ' '.$combination_price_level->error;
$this->errors[] = $this->error; $this->errors[] = $this->error;
$errors ++; $errors ++;
} }
} }
if($errors > 0){ if ($errors > 0){
return $errors*-1; return $errors*-1;
} }
else{ else {
return 1; return 1;
} }
} }
@ -354,7 +354,7 @@ class ProductCombination
if (!empty($conf->global->PRODUIT_MULTIPRICES)) { if (!empty($conf->global->PRODUIT_MULTIPRICES)) {
$res = $this->saveCombinationPriceLevels(); $res = $this->saveCombinationPriceLevels();
if($res<0){ if ($res<0){
return -2; return -2;
} }
} }
@ -388,7 +388,7 @@ class ProductCombination
if (!empty($conf->global->PRODUIT_MULTIPRICES)) { if (!empty($conf->global->PRODUIT_MULTIPRICES)) {
$res = $this->saveCombinationPriceLevels(); $res = $this->saveCombinationPriceLevels();
if($res<0){ if ($res<0){
return -2; return -2;
} }
} }
@ -415,7 +415,7 @@ class ProductCombination
$comb2val->deleteByFkCombination($this->id); $comb2val->deleteByFkCombination($this->id);
// remove combination price levels // remove combination price levels
if(!$this->db->query("DELETE FROM ".MAIN_DB_PREFIX."product_attribute_combination_price_level WHERE fk_product_attribute_combination = ".(int) $this->id)) { if (!$this->db->query("DELETE FROM ".MAIN_DB_PREFIX."product_attribute_combination_price_level WHERE fk_product_attribute_combination = ".(int) $this->id)) {
$this->db->rollback(); $this->db->rollback();
return -1; return -1;
} }
@ -705,10 +705,10 @@ WHERE c.fk_product_parent = ".(int) $productid." AND p.tosell = 1";
$weight_impact = (float) $forced_weightvar; // If false, return 0 $weight_impact = (float) $forced_weightvar; // If false, return 0
//Final price impact //Final price impact
if(!is_array($forced_pricevar)){ if (!is_array($forced_pricevar)){
$price_impact[1] = (float) $forced_pricevar; // If false, return 0 $price_impact[1] = (float) $forced_pricevar; // If false, return 0
} }
else{ else {
$price_impact = $forced_pricevar; $price_impact = $forced_pricevar;
} }
@ -755,7 +755,7 @@ WHERE c.fk_product_parent = ".(int) $productid." AND p.tosell = 1";
$price_impact[1] += (float) price2num($variations[$currcombattr][$currcombval]['price']); $price_impact[1] += (float) price2num($variations[$currcombattr][$currcombval]['price']);
// Manage Price levels // Manage Price levels
if($conf->global->PRODUIT_MULTIPRICES){ if ($conf->global->PRODUIT_MULTIPRICES){
for ($i = 2; $i <= $conf->global->PRODUIT_MULTIPRICES_LIMIT; $i++) for ($i = 2; $i <= $conf->global->PRODUIT_MULTIPRICES_LIMIT; $i++)
{ {
$price_impact[$i] += (float) price2num($variations[$currcombattr][$currcombval]['price']); $price_impact[$i] += (float) price2num($variations[$currcombattr][$currcombval]['price']);
@ -783,16 +783,16 @@ WHERE c.fk_product_parent = ".(int) $productid." AND p.tosell = 1";
$newcomb->variation_weight = $weight_impact; $newcomb->variation_weight = $weight_impact;
// Init price level // Init price level
if($conf->global->PRODUIT_MULTIPRICES){ if ($conf->global->PRODUIT_MULTIPRICES){
for ($i = 1; $i <= $conf->global->PRODUIT_MULTIPRICES_LIMIT; $i++){ for ($i = 1; $i <= $conf->global->PRODUIT_MULTIPRICES_LIMIT; $i++){
$productCombinationLevel = new ProductCombinationLevel($this->db); $productCombinationLevel = new ProductCombinationLevel($this->db);
$productCombinationLevel->fk_product_attribute_combination = 0; $productCombinationLevel->fk_product_attribute_combination = 0;
$productCombinationLevel->fk_price_level = $i; $productCombinationLevel->fk_price_level = $i;
$productCombinationLevel->variation_price = $price_impact[$i]; $productCombinationLevel->variation_price = $price_impact[$i];
if(is_array($price_var_percent)){ if (is_array($price_var_percent)){
$productCombinationLevel->variation_price_percentage = !empty($price_var_percent[$i]) ? $price_var_percent[$i] : 0; $productCombinationLevel->variation_price_percentage = !empty($price_var_percent[$i]) ? $price_var_percent[$i] : 0;
}else{ }else {
$productCombinationLevel->variation_price_percentage = $price_var_percent; $productCombinationLevel->variation_price_percentage = $price_var_percent;
} }
@ -1023,11 +1023,11 @@ class ProductCombinationLevel
*/ */
public function fetch($rowid) public function fetch($rowid)
{ {
$sql = "SELECT rowid, fk_product_attribute_combination, fk_price_level, variation_price, variation_price_percentage FROM " . MAIN_DB_PREFIX . $this->table_element." WHERE rowid = " . (int)$rowid; $sql = "SELECT rowid, fk_product_attribute_combination, fk_price_level, variation_price, variation_price_percentage FROM " . MAIN_DB_PREFIX . $this->table_element." WHERE rowid = " . (int) $rowid;
$obj = $this->db->getRow($sql); $obj = $this->db->getRow($sql);
if($obj){ if ($obj){
return $this->fetchFormObj($obj); return $this->fetchFormObj($obj);
} }
@ -1049,7 +1049,7 @@ class ProductCombinationLevel
." FROM ".MAIN_DB_PREFIX.$this->table_element ." FROM ".MAIN_DB_PREFIX.$this->table_element
." WHERE fk_product_attribute_combination = ".intval($fk_product_attribute_combination); ." WHERE fk_product_attribute_combination = ".intval($fk_product_attribute_combination);
if(!empty($fk_price_level)){ if (!empty($fk_price_level)){
$sql.= ' AND fk_price_level = '.intval($fk_price_level); $sql.= ' AND fk_price_level = '.intval($fk_price_level);
} }
@ -1061,9 +1061,9 @@ class ProductCombinationLevel
$result = array(); $result = array();
if(!empty($combination_price_levels)) { if (!empty($combination_price_levels)) {
// For more simple usage set level as array key // For more simple usage set level as array key
foreach($combination_price_levels as $k => $row){ foreach ($combination_price_levels as $k => $row){
$productCombinationLevel = new ProductCombinationLevel($this->db); $productCombinationLevel = new ProductCombinationLevel($this->db);
$productCombinationLevel->fetchFormObj($row); $productCombinationLevel->fetchFormObj($row);
$result[$row->fk_price_level] = $productCombinationLevel; $result[$row->fk_price_level] = $productCombinationLevel;
@ -1089,7 +1089,7 @@ class ProductCombinationLevel
$this->fk_product_attribute_combination = doubleval($obj->fk_product_attribute_combination); $this->fk_product_attribute_combination = doubleval($obj->fk_product_attribute_combination);
$this->fk_price_level = intval($obj->fk_price_level); $this->fk_price_level = intval($obj->fk_price_level);
$this->variation_price = doubleval($obj->variation_price); $this->variation_price = doubleval($obj->variation_price);
$this->variation_price_percentage = (bool)$obj->variation_price_percentage; $this->variation_price_percentage = (bool) $obj->variation_price_percentage;
return 1; return 1;
} }
@ -1105,41 +1105,41 @@ class ProductCombinationLevel
$errors = 0; $errors = 0;
if(empty($this->fk_product_attribute_combination) || empty($this->fk_price_level)){ if (empty($this->fk_product_attribute_combination) || empty($this->fk_price_level)){
return -1; return -1;
} }
// check if level exist in DB before add // check if level exist in DB before add
if(empty($this->id)){ if (empty($this->id)){
$sql = "SELECT rowid id" $sql = "SELECT rowid id"
." FROM ".MAIN_DB_PREFIX . $this->table_element ." FROM ".MAIN_DB_PREFIX . $this->table_element
." WHERE fk_product_attribute_combination = ".(int) $this->fk_product_attribute_combination ." WHERE fk_product_attribute_combination = ".(int) $this->fk_product_attribute_combination
.' AND fk_price_level = '.intval($this->fk_price_level); .' AND fk_price_level = '.intval($this->fk_price_level);
$existObj = $this->db->getRow($sql); $existObj = $this->db->getRow($sql);
if($existObj){ if ($existObj){
$this->id = $existObj->id; $this->id = $existObj->id;
} }
} }
// Update // Update
if(!empty($this->id)) { if (!empty($this->id)) {
$sql = 'UPDATE ' . MAIN_DB_PREFIX . $this->table_element . ' SET ' $sql = 'UPDATE ' . MAIN_DB_PREFIX . $this->table_element . ' SET '
. ' variation_price = '.doubleval($this->variation_price) . ' variation_price = '.doubleval($this->variation_price)
. ' , variation_price_percentage = '.intval($this->variation_price_percentage) . ' , variation_price_percentage = '.intval($this->variation_price_percentage)
. ' WHERE rowid = '.intval($this->id); . ' WHERE rowid = '.intval($this->id);
$res = $this->db->query($sql); $res = $this->db->query($sql);
if($res>0){ if ($res>0){
return $this->id; return $this->id;
} }
else{ else {
$this->error = $this->db->error(); $this->error = $this->db->error();
$this->errors[] = $this->error; $this->errors[] = $this->error;
return -1; return -1;
} }
} }
else{ else {
// ADD // ADD
$sql = "INSERT INTO " . MAIN_DB_PREFIX . $this->table_element . " (" $sql = "INSERT INTO " . MAIN_DB_PREFIX . $this->table_element . " ("
. " fk_product_attribute_combination, fk_price_level, variation_price, variation_price_percentage" . " fk_product_attribute_combination, fk_price_level, variation_price, variation_price_percentage"
@ -1151,10 +1151,10 @@ class ProductCombinationLevel
. " )"; . " )";
$res = $this->db->query($sql); $res = $this->db->query($sql);
if($res){ if ($res){
$this->id = $this->db->last_insert_id(MAIN_DB_PREFIX . $this->table_element); $this->id = $this->db->last_insert_id(MAIN_DB_PREFIX . $this->table_element);
} }
else{ else {
$this->error = $this->db->error(); $this->error = $this->db->error();
$this->errors[] = $this->error; $this->errors[] = $this->error;
return -1; return -1;
@ -1211,5 +1211,4 @@ class ProductCombinationLevel
return $res ? 1 : -1; return $res ? 1 : -1;
} }
} }