diff --git a/htdocs/contrat/contrat.class.php b/htdocs/contrat/contrat.class.php
index 4bf7500894f..61efc118698 100644
--- a/htdocs/contrat/contrat.class.php
+++ b/htdocs/contrat/contrat.class.php
@@ -375,11 +375,8 @@ class Contrat
         $qty = ereg_replace(",",".",$qty);
         $pu = ereg_replace(",",".",$pu);
         
-        if (strlen(trim($qty))==0)
-        {
-            $qty=1;
-        }
-        
+        dolibarr_syslog("Contrat::AddLine $desc, $pu, $qty, $txtva, $fk_product=0, $remise_percent=0, $datestart, $dateend");
+
         if ($fk_product > 0)
         {
             $prod = new Product($this->db, $fk_product);
@@ -394,7 +391,7 @@ class Contrat
         $remise = 0;
         $price = ereg_replace(",",".",round($pu, 2));
         $subprice = $price;
-        if (trim(strlen($remise_percent)) > 0)
+        if (strlen($remise_percent) > 0)
         {
             $remise = round(($pu * $remise_percent / 100), 2);
             $price = $pu - $remise;
diff --git a/htdocs/contrat/fiche.php b/htdocs/contrat/fiche.php
index 430cc9064ea..5dc7298d733 100644
--- a/htdocs/contrat/fiche.php
+++ b/htdocs/contrat/fiche.php
@@ -41,12 +41,26 @@ $user->getrights('contrat');
 if (! $user->rights->contrat->lire)
 accessforbidden();
 
-$startyear=isset($_POST["date_startyear"])&&$_POST["date_startyear"]?$_POST["date_startyear"]:0;
-$endyear=isset($_POST["date_endyear"])&&$_POST["date_endyear"]?$_POST["date_endyear"]:0;
-$date_start_update=mktime(12, 0 , 0, $_POST["date_start_updatemonth"], $_POST["date_start_updateday"], $startyear);
-$date_end_update=mktime(12, 0 , 0, $_POST["date_end_updatemonth"], $_POST["date_end_updateday"], $startyear);
-$date_start=mktime(12, 0 , 0, $_POST["date_startmonth"], $_POST["date_startday"], $endyear);
-$date_end=mktime(12, 0 , 0, $_POST["date_endmonth"], $_POST["date_endday"], $endyear);
+$date_start='';
+$date_end='';
+$date_start_update='';
+$date_end_update='';
+if ($_POST["date_start_updatemonth"] && $_POST["date_start_updateday"] && $_POST["date_start_updateyear"])
+{
+    $date_start_update=mktime(12, 0 , 0, $_POST["date_start_updatemonth"], $_POST["date_start_updateday"], $_POST["date_start_updateyear"]);
+}
+if ($_POST["date_end_updatemonth"] && $_POST["date_end_updateday"] && $_POST["date_end_updateyear"])
+{
+    $date_end_update=mktime(12, 0 , 0, $_POST["date_end_updatemonth"], $_POST["date_end_updateday"], $_POST["date_end_updateyear"]);
+}
+if ($_POST["date_startmonth"] && $_POST["date_startday"] && $_POST["date_startyear"])
+{
+    $date_start=mktime(12, 0 , 0, $_POST["date_startmonth"], $_POST["date_startday"], $_POST["date_startyear"]);
+}
+if ($_POST["date_endmonth"] && $_POST["date_endday"] && $_POST["date_endyear"])
+{
+    $date_end=mktime(12, 0 , 0, $_POST["date_endmonth"], $_POST["date_endday"], $_POST["date_endyear"]);
+}
 
 // Sécurité accés client
 if ($user->societe_id > 0)