From f6b5a508ada00fcc39afbea939d883b72ffe20fb Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@users.sourceforge.net>
Date: Sun, 24 Dec 2006 21:24:57 +0000
Subject: [PATCH] Fix: Creation facture depuis contrat

---
 htdocs/contrat/contrat.class.php |  4 +-
 htdocs/facture.class.php         | 89 +++++++++++++++++---------------
 2 files changed, 48 insertions(+), 45 deletions(-)

diff --git a/htdocs/contrat/contrat.class.php b/htdocs/contrat/contrat.class.php
index 336325073df..483b159fe7c 100644
--- a/htdocs/contrat/contrat.class.php
+++ b/htdocs/contrat/contrat.class.php
@@ -330,19 +330,19 @@ class Contrat extends CommonObject
      */
     function fetch_lignes()
     {
-        dolibarr_syslog("Contrat.class.php::fetch_lignes");
         $this->lignes = array();
     
         // Selectionne les lignes contrats liées à un produit
         $sql = "SELECT p.label, p.description as product_desc, p.ref,";
         $sql.= " d.rowid, d.statut, d.description, d.price_ht, d.tva_tx, d.qty, d.remise_percent, d.subprice,";
-        $sql.= " d.info_bits, d.fk_product";
+        $sql.= " d.info_bits, d.fk_product,";
         $sql.= " d.date_ouverture_prevue, d.date_ouverture,";
         $sql.= " d.date_fin_validite, d.date_cloture";
         $sql.= " FROM ".MAIN_DB_PREFIX."contratdet as d, ".MAIN_DB_PREFIX."product as p";
         $sql.= " WHERE d.fk_contrat = ".$this->id ." AND d.fk_product = p.rowid";
         $sql.= " ORDER by d.rowid ASC";
         
+        dolibarr_syslog("Contrat.class.php::fetch_lignes sql=".$sql);
         $result = $this->db->query($sql);
         if ($result)
         {
diff --git a/htdocs/facture.class.php b/htdocs/facture.class.php
index 9f7200b6c73..73b7090ae27 100644
--- a/htdocs/facture.class.php
+++ b/htdocs/facture.class.php
@@ -1267,50 +1267,53 @@ class Facture extends CommonObject
       }
   }
 
-  /**
-     \brief   	Ajoute une ligne dans le tableau products
-     \param    	idproduct		Id du produit a ajouter
-     \param    	qty			Quantité
-     \param    	remise_percent		Remise relative effectuée sur le produit
-     \param    	date_start
-     \param    	date_end
-     \return   	void
-     \remarks	$this->client doit etre chargé
-     \TODO	Remplacer les appels a cette fonction par generation objet Ligne 
-     inséré dans tableau $this->products
-   */
-  function add_product($idproduct, $qty, $remise_percent, $date_start='', $date_end='')
-  {
-    global $conf, $mysoc;
+	/**
+		\brief   	Ajoute une ligne dans le tableau products
+		\param    	idproduct		Id du produit a ajouter
+		\param    	qty			Quantit
+		\param    	remise_percent		Remise relative effectuée sur le produit
+		\param    	date_start
+		\param    	date_end
+		\return   	void
+		\remarks	$this->client doit etre charg
+		\TODO		Remplacer les appels a cette fonction par generation objet Ligne
+					inséré dans tableau $this->products
+	*/
+	function add_product($idproduct, $qty, $remise_percent, $date_start='', $date_end='')
+	{
+		global $conf, $mysoc;
+	
+		// Nettoyage parametres
+		if (! $qty) $qty = 1;
+	
+		dolibarr_syslog("Facture.class::add_product $idproduct, $qty, $remise_percent, $date_start, $date_end");
 
-    if (! $qty) $qty = 1;
-		
-    if ($idproduct > 0)
-      {
-	$prod=new Product($this->db);
-	$prod->fetch($idproduct);
-			
-	$tva_tx = get_default_tva($mysoc,$this->client,$prod->tva_tx);
-	// multiprix
-	if($conf->global->PRODUIT_MULTIPRICES == 1)
-	  $price = $prod->multiprices[$this->client->price_level];
-	else
-	  $price = $prod->price;
-
-	$line=new FactureLigne($this->db);
-	$line->rowid = $idproduct;
-	$line->fk_product = $idproduct;
-	$line->desc = $prod->description;
-	$line->qty = $qty;
-	$line->subprice = $price;
-	$line->remise_percent = $remise_percent;
-	$line->tva_tx = $tva_tx;
-	if ($date_start) { $line->date_start = $date_start; }
-	if ($date_end)   { $line->date_end = $date_end; }
-
-	$this->products[]=$line;
-      }
-  }
+		if ($idproduct > 0)
+		{
+			$prod=new Product($this->db);
+			$prod->fetch($idproduct);
+	
+			$tva_tx = get_default_tva($mysoc,$this->client,$prod->tva_tx);
+			// multiprix
+			if($conf->global->PRODUIT_MULTIPRICES == 1)
+			$price = $prod->multiprices[$this->client->price_level];
+			else
+			$price = $prod->price;
+	
+			$line=new FactureLigne($this->db);
+			$line->rowid = $idproduct;
+			$line->fk_product = $idproduct;
+			$line->desc = $prod->description;
+			$line->qty = $qty;
+			$line->subprice = $price;
+			$line->remise_percent = $remise_percent;
+			$line->tva_tx = $tva_tx;
+			if ($date_start) { $line->date_start = $date_start; }
+			if ($date_end)   { $line->date_end = $date_end; }
+	
+			$this->products[]=$line;
+		}
+	}
 
   /**
    * 		\brief    	Ajoute une ligne de facture (associé à un produit/service prédéfini ou non)