From c795803da1e49cdd3d00868094fe35a2f9dee0cb Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Thu, 5 May 2016 20:19:56 +0200 Subject: [PATCH] FIX ref_supplier was not set. We also refered to old var names. --- .../fourn/class/fournisseur.product.class.php | 4 ++- htdocs/fourn/commande/card.php | 31 +++++++++++-------- 2 files changed, 21 insertions(+), 14 deletions(-) diff --git a/htdocs/fourn/class/fournisseur.product.class.php b/htdocs/fourn/class/fournisseur.product.class.php index 03be7e77d36..48896689dce 100644 --- a/htdocs/fourn/class/fournisseur.product.class.php +++ b/htdocs/fourn/class/fournisseur.product.class.php @@ -613,7 +613,9 @@ class ProductFournisseur extends Product if ($fourn_unitprice < $min || $min == -1) { $this->product_fourn_price_id = $record["product_fourn_price_id"]; - $this->fourn_ref = $record["ref_fourn"]; + $this->ref_supplier = $record["ref_fourn"]; + $this->ref_fourn = $record["ref_fourn"]; // deprecated + $this->fourn_ref = $record["ref_fourn"]; // deprecated $this->fourn_price = $fourn_price; $this->fourn_qty = $record["quantity"]; $this->fourn_remise_percent = $record["remise_percent"]; diff --git a/htdocs/fourn/commande/card.php b/htdocs/fourn/commande/card.php index b35ffa799da..4454c0ac28c 100644 --- a/htdocs/fourn/commande/card.php +++ b/htdocs/fourn/commande/card.php @@ -984,17 +984,16 @@ if (empty($reshook)) // If creation from another object of another module (Example: origin=propal, originid=1) if (! empty($origin) && ! empty($originid)) { - if ($origin == 'order' || $origin == 'commande') { + if ($origin == 'order' || $origin == 'commande') + { $element = $subelement = 'commande'; } - elseif ($origin =='supplier_proposal'){ + else + { $element = 'supplier_proposal'; $subelement = 'supplier_proposal'; - } - else { - $element = 'comm/askpricesupplier'; - $subelement = 'askpricesupplier'; - } + } + $object->origin = $origin; $object->origin_id = $originid; @@ -1056,11 +1055,17 @@ if (empty($reshook)) } $result = $productsupplier->find_min_price_product_fournisseur($lines[$i]->fk_product, $lines[$i]->qty); - if ($result>0) { - $productsupplier->fetch($productsupplier->id); - $soc=new societe($db); - $soc->fetch($socid); - $tva_tx=($origin=="commande")?get_default_tva($soc,$mysoc,$lines[$i]->fk_product,$idprod):$lines[$i]->tva_tx; + if ($result>0) + { + $tva_tx = $lines[$i]->tva_tx; + + if ($origin=="commande") + { + $soc=new societe($db); + $soc->fetch($socid); + $tva_tx=get_default_tva($soc, $mysoc, $lines[$i]->fk_product, $productsupplier->product_fourn_price_id); + } + $result = $object->addline( $desc, $lines[$i]->subprice, @@ -1070,7 +1075,7 @@ if (empty($reshook)) $lines[$i]->localtax2_tx, $lines[$i]->fk_product, $productsupplier->product_fourn_price_id, - $productsupplier->ref_fourn, + $productsupplier->ref_supplier, $lines[$i]->remise_percent, 'HT', 0,