Dolibarr\Code\Product\Classes\Product::updatePrice()

Complexity

Conditions	38
Paths	> 20000

Size

Total Lines	187
Code Lines	127

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function updatePrice($newprice, $newpricebase, $user, $newvat = null, $newminprice = 0, $level = 0, $newnpr = 0, $newpbq = 0, $ignore_autogen = 0, $localtaxes_array = array(), $newdefaultvatcode = '', $price_label = '', $notrigger = 0)
    {
        global $conf, $langs;

        $lastPriceData = $this->getArrayForPriceCompare($level); // temporary store current price before update

        $id = $this->id;

        dol_syslog(get_class($this) . "::update_price id=" . $id . " newprice=" . $newprice . " newpricebase=" . $newpricebase . " newminprice=" . $newminprice . " level=" . $level . " npr=" . $newnpr . " newdefaultvatcode=" . $newdefaultvatcode);

        // Clean parameters
        if (empty($this->tva_tx)) {
            $this->tva_tx = 0;
        }
        if (empty($newnpr)) {
            $newnpr = 0;
        }
        if (empty($newminprice)) {
            $newminprice = 0;
        }

        // Check parameters
        if ($newvat === null || $newvat == '') {  // Maintain '' for backwards compatibility
            $newvat = $this->tva_tx;
        }

        // If multiprices are enabled, then we check if the current product is subject to price autogeneration
        // Price will be modified ONLY when the first one is the one that is being modified
        if ((getDolGlobalString('PRODUIT_MULTIPRICES') || getDolGlobalString('PRODUIT_CUSTOMER_PRICES_BY_QTY_MULTIPRICES')) && !$ignore_autogen && $this->price_autogen && ($level == 1)) {
            return $this->generateMultiprices($user, $newprice, $newpricebase, $newvat, $newnpr, $newpbq);
        }

        if (!empty($newminprice) && ($newminprice > $newprice)) {
            $this->error = 'ErrorPriceCantBeLowerThanMinPrice';
            return -1;
        }

        if ($newprice !== '' || $newprice === 0) {
            if ($newpricebase == 'TTC') {
                $price_ttc = price2num($newprice, 'MU');
                $price = (float) price2num($newprice) / (1 + ((float) $newvat / 100));
                $price = price2num($price, 'MU');

                if ($newminprice != '' || $newminprice == 0) {
                    $price_min_ttc = price2num($newminprice, 'MU');
                    $price_min = (float) price2num($newminprice) / (1 + ($newvat / 100));
                    $price_min = price2num($price_min, 'MU');
                } else {
                    $price_min = 0;
                    $price_min_ttc = 0;
                }
            } else {
                $price = (float) price2num($newprice, 'MU');
                $price_ttc = ($newnpr != 1) ? price2num($newprice) * (1 + ($newvat / 100)) : $price;
                $price_ttc = (float) price2num($price_ttc, 'MU');

                if ($newminprice !== '' || $newminprice === 0) {
                    $price_min = price2num($newminprice, 'MU');
                    $price_min_ttc = (float) price2num($newminprice) * (1 + ($newvat / 100));
                    $price_min_ttc = price2num($price_min_ttc, 'MU');
                    //print 'X'.$newminprice.'-'.$price_min;
                } else {
                    $price_min = 0;
                    $price_min_ttc = 0;
                }
            }
            //print 'x'.$id.'-'.$newprice.'-'.$newpricebase.'-'.$price.'-'.$price_ttc.'-'.$price_min.'-'.$price_min_ttc;

            if (count($localtaxes_array) > 0) {
                $localtaxtype1 = $localtaxes_array['0'];
                $localtax1 = $localtaxes_array['1'];
                $localtaxtype2 = $localtaxes_array['2'];
                $localtax2 = $localtaxes_array['3'];
            } else {
                // if array empty, we try to use the vat code
                if (!empty($newdefaultvatcode)) {
                    global $mysoc;
                    // Get record from code
                    $sql = "SELECT t.rowid, t.code, t.recuperableonly as tva_npr, t.localtax1, t.localtax2, t.localtax1_type, t.localtax2_type";
                    $sql .= " FROM " . MAIN_DB_PREFIX . "c_tva as t, " . MAIN_DB_PREFIX . "c_country as c";
                    $sql .= " WHERE t.fk_pays = c.rowid AND c.code = '" . $this->db->escape($mysoc->country_code) . "'";
                    $sql .= " AND t.taux = " . ((float) $newdefaultvatcode) . " AND t.active = 1";
                    $sql .= " AND t.code = '" . $this->db->escape($newdefaultvatcode) . "'";
                    $resql = $this->db->query($sql);
                    if ($resql) {
                        $obj = $this->db->fetch_object($resql);
                        if ($obj) {
                            $npr = $obj->tva_npr;
                            $localtax1 = $obj->localtax1;
                            $localtax2 = $obj->localtax2;
                            $localtaxtype1 = $obj->localtax1_type;
                            $localtaxtype2 = $obj->localtax2_type;
                        }
                    }
                } else {
                    // old method. deprecated because we can't retrieve type
                    $localtaxtype1 = '0';
                    $localtax1 = get_localtax($newvat, 1);
                    $localtaxtype2 = '0';
                    $localtax2 = get_localtax($newvat, 2);
                }
            }
            if (empty($localtax1)) {
                $localtax1 = 0; // If = '' then = 0
            }
            if (empty($localtax2)) {
                $localtax2 = 0; // If = '' then = 0
            }

            $this->db->begin();

            // Ne pas mettre de quote sur les numeriques decimaux.
            // Ceci provoque des stockages avec arrondis en base au lieu des valeurs exactes.
            $sql = "UPDATE " . $this->db->prefix() . "product SET";
            $sql .= " price_base_type = '" . $this->db->escape($newpricebase) . "',";
            $sql .= " price = " . (float) $price . ",";
            $sql .= " price_ttc = " . (float) $price_ttc . ",";
            $sql .= " price_min = " . (float) $price_min . ",";
            $sql .= " price_min_ttc = " . (float) $price_min_ttc . ",";
            $sql .= " localtax1_tx = " . ($localtax1 >= 0 ? (float) $localtax1 : 'NULL') . ",";
            $sql .= " localtax2_tx = " . ($localtax2 >= 0 ? (float) $localtax2 : 'NULL') . ",";
            $sql .= " localtax1_type = " . ($localtaxtype1 != '' ? "'" . $this->db->escape($localtaxtype1) . "'" : "'0'") . ",";
            $sql .= " localtax2_type = " . ($localtaxtype2 != '' ? "'" . $this->db->escape($localtaxtype2) . "'" : "'0'") . ",";
            $sql .= " default_vat_code = " . ($newdefaultvatcode ? "'" . $this->db->escape($newdefaultvatcode) . "'" : "null") . ",";
            $sql .= " price_label = " . (!empty($price_label) ? "'" . $this->db->escape($price_label) . "'" : "null") . ",";
            $sql .= " tva_tx = " . (float) price2num($newvat) . ",";
            $sql .= " recuperableonly = '" . $this->db->escape($newnpr) . "'";
            $sql .= " WHERE rowid = " . ((int) $id);

            dol_syslog(get_class($this) . "::update_price", LOG_DEBUG);
            $resql = $this->db->query($sql);
            if ($resql) {
                $this->multiprices[$level] = $price;
                $this->multiprices_ttc[$level] = $price_ttc;
                $this->multiprices_min[$level] = $price_min;
                $this->multiprices_min_ttc[$level] = $price_min_ttc;
                $this->multiprices_base_type[$level] = $newpricebase;
                $this->multiprices_default_vat_code[$level] = $newdefaultvatcode;
                $this->multiprices_tva_tx[$level] = $newvat;
                $this->multiprices_recuperableonly[$level] = $newnpr;

                $this->price = $price;

It seems like $price can also be of type string. However, the property $price is declared as type double. Maybe add an additional type check?

                $this->price_label = $price_label;
                $this->price_ttc = $price_ttc;

It seems like $price_ttc can also be of type string. However, the property $price_ttc is declared as type double. Maybe add an additional type check?

                $this->price_min = $price_min;

It seems like $price_min can also be of type string. However, the property $price_min is declared as type double. Maybe add an additional type check?

                $this->price_min_ttc = $price_min_ttc;

It seems like $price_min_ttc can also be of type string. However, the property $price_min_ttc is declared as type double. Maybe add an additional type check?

                $this->price_base_type = $newpricebase;
                $this->default_vat_code = $newdefaultvatcode;
                $this->tva_tx = $newvat;
                $this->tva_npr = $newnpr;

                //Local taxes
                $this->localtax1_tx = $localtax1;
                $this->localtax2_tx = $localtax2;
                $this->localtax1_type = $localtaxtype1;
                $this->localtax2_type = $localtaxtype2;

                // Price by quantity
                $this->price_by_qty = $newpbq;

                // check if price have really change before log
                $newPriceData = $this->getArrayForPriceCompare($level);
                if (!empty(array_diff_assoc($newPriceData, $lastPriceData)) || !getDolGlobalString('PRODUIT_MULTIPRICES')) {
                    $this->_log_price($user, $level); // Save price for level into table product_price
                }

                $this->level = $level; // Store level of price edited for trigger

                // Call trigger
                if (!$notrigger) {
                    $result = $this->call_trigger('PRODUCT_PRICE_MODIFY', $user);
                    if ($result < 0) {
                        $this->db->rollback();
                        return -1;
                    }
                }
                // End call triggers

                $this->db->commit();
            } else {
                $this->db->rollback();
                $this->error = $this->db->lasterror();
                return -1;
            }
        }

        return 1;
    }