Bugfixed et réactivé test date reception > date achat "mais pas trop"

v4.101.7-3.7.9

Bugfixed et réactivé test date reception > date achat "mais pas trop"
v4.101.7-3.7.9
Etienne Pallier
1 parent 26863608
Showing 5 changed files with 16 additions and 12 deletions Show diff stats
CHANGES.txt
README.md
src/Controller/MaterielsController.php
src/Model/Entity/Materiel.php
src/Model/Table/MaterielsTable.php
@@ -94,6 +94,10 @@ Outre ces changements, voici d&#39;autres changements importants :
 ======= CHANGES =======
 -------
+16/09/2020	v4.101.7-3.7.9 (EP)
+	- (b) Bugfixed et réactivé test date reception > date achat "mais pas trop" (bug sur php5)
+
+-------
 15/09/2020	v4.101.6-3.7.9 (EP)
 	- (b) Temporairement désactivé test date reception > date achat "mais pas trop" à cause bug sur php5 !!!
 	- (e) Améliorations de la vue liste des projets et de la vue détaillée
@@ -43,7 +43,7 @@ Logiciel testé et validé sur les configurations suivantes :
 --------------------------------------------------------------------------------------------
 Date: 15/09/2020
-Version: 4.101.6-3.7.9
+Version: 4.101.7-3.7.9
 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
@@ -1595,6 +1595,7 @@ class MaterielsController extends AppController {
             $materiel = $this->Materiels->patchEntity($materiel, $this->request->getData());
             //debug($materiel); exit;
+            /*
             // AVIRER
             // BUGFIX temporaire pour php5 !!!
             if ($this->isLabinventDebugMode()) {
@@ -1602,13 +1603,13 @@ class MaterielsController extends AppController {
                 $d1 = $materiel->date_acquisition;
                 debug("d1 achat avant:"); debug($d1);
                 debug("d2 recep avant:"); debug($d2);
-                /*
+                /S
                 $tz = new \DateTimeZone('Europe/Paris');
                 $d1 = new \DateTime(strtr($d1,'/','-'),$tz);
                 $d2 = new \DateTime(strtr($d2,'/','-'),$tz);
                 debug("d1 achat après:"); debug($d1);
                 debug("d2 recep après:"); debug($d2);
-                */
+                S/
                 if ($d2 < $d1) debug("d2 < d1 !!!");
                 // $d2 > $d1 oui mais pas trop...
                 $diff = $d2->diff($d1);
@@ -1619,6 +1620,7 @@ class MaterielsController extends AppController {
                 exit;
             }
             // FIN BUGFIX
+            */
             // ADD : Set the user_id from the session.
             //$materiel->user_id = $this->Auth->user('id');
@@ -3,8 +3,8 @@ namespace App\Model\Entity;
 use Cake\ORM\Entity;
-// Max 10 ans entre 2 dates
-const MAX_DIFF_YEARS = 10;
+// Max 2 ans entre 2 dates
+const MAX_DIFF_YEARS = 2;
 /**
  * Materiel Entity.
@@ -94,6 +94,7 @@ class Materiel extends Entity {
         $d2 = $this->$d2_name;
         $d1 = $this->$d1_name;
         if (!$d2 || !$d1) return true;
+        /*
         $tz = new \DateTimeZone('Europe/Paris');
         //date_default_timezone_set('Europe/Paris');
         // DateTime lit les dates au format JJ-MM-YYYY (et non pas JJ/MM/YYYY)
@@ -101,19 +102,16 @@ class Materiel extends Entity {
         $d1 = new \DateTime(strtr($d1,'/','-'),$tz);
         //$d1_text = $d1->format('Ymd');
         $d2 = new \DateTime(strtr($d2,'/','-'),$tz);
+        */
         //$d2_text = $d2->format('Ymd');
         //$today = (new \DateTime('now',$tz))->format('Ymd');
         //$ok = ($d2_text >= $d1_text);
         if ($d2 < $d1) return false;
         // $d2 > $d1 oui mais pas trop...
         $diff = $d2->diff($d1);
-        /* 
-         * (EP202009 test temporairement désactivé car bug sur php5 !!!)
-         * TODO: à réactiver
         //debug($diff->y);
         return $diff->y < MAX_DIFF_YEARS;
-        */
-        return true;
+        //return true;
     }
     protected function hasStatus($status) { return $this->status == $status; }
@@ -636,11 +636,11 @@ class MaterielsTable extends AppTable
         // Check dates reception et fin garantie
         $rules->add($dateIsAfterDateAchatAndNotTooFar, [
             'errorField' => 'date_reception',
-            'message' => "La date doit être postérieure à la date d'achat (et pas trop loin)"
+            'message' => "La date doit être postérieure à la date d'achat (mais pas trop loin)"
         ]);
         $rules->add($dateIsAfterDateReceptionAndNotTooFar, [
             'errorField' => 'date_fin_garantie',
-            'message' => "La date doit être postérieure à la date de livraison (et pas trop loin)"
+            'message' => "La date doit être postérieure à la date de livraison (mais pas trop loin)"
         ]);