date réception => "date livraison", et ne doit pas être future

v4.101.12-3.7.9

date réception => "date livraison", et ne doit pas être future
v4.101.12-3.7.9
Etienne Pallier
1 parent 0154a32b
Showing 6 changed files with 97 additions and 25 deletions Show diff stats
CHANGES.txt
README.md
src/Model/Entity/Materiel.php
src/Model/Table/MaterielsTable.php
src/Template/Materiels/add_edit.ctp
src/Template/Materiels/view.ctp
@@ -94,6 +94,10 @@ Outre ces changements, voici d&#39;autres changements importants :
 ======= CHANGES =======
 -------
+17/09/2020	v4.101.12-3.7.9 (EP)
+	- (e) date réception => "date livraison", et ne doit pas être future
+
+-------
 16/09/2020	v4.101.11-3.7.9 (EP)
 	- (i) Fixtures de tests simplifiées avec utilisation par défaut du format des champs de la BD de production (inutile de s'embêter à tout redéfinir et à tenir à jour !!!)
 	- (i) Corrigé petit bug sur site_id qui valait 2 par défaut (au lieu de null)
@@ -43,7 +43,7 @@ Logiciel testé et validé sur les configurations suivantes :
 --------------------------------------------------------------------------------------------
 Date: 15/09/2020
-Version: 4.101.11-3.7.9
+Version: 4.101.12-3.7.9
 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
@@ -3,8 +3,8 @@ namespace App\Model\Entity;
 use Cake\ORM\Entity;
-// Max 2 ans entre 2 dates
-const MAX_DIFF_YEARS = 2;
+// Max 15 ans entre 2 dates
+const MAX_DIFF_YEARS = 15;
 /**
  * Materiel Entity.
@@ -88,11 +88,30 @@ class Materiel extends Entity {
     // Ce qui s'affiche quand on fait echo $entity
     public function __toString() { return $this->designation; }
+    public function check_date_is_not_too_old($date_field_name) {
+        $d = $this->$date_field_name;
+        // today - 50 ans = trop vieux !
+        /* Avec TZ
+        $tz = new \DateTimeZone('Europe/Paris');
+        $date_too_old = new \DateTime('-50 years',$tz);
+        */
+        // Sans TimeZone (car inutile à ce niveau je crois)
+        $date_too_old = new \DateTime('-50 years');
+        return $d > $date_too_old;
+    }
+    public function check_date_is_not_future($date_field_name) {
+        $d = $this->$date_field_name;
+        $today = new \DateTime('now');
+        return $d <= $today;
+    }
+    
     // date_reception >= date_acquisition et aussi date_fin_garantie >= date_reception
     public function check_date_d2_gt_d1_but_not_too_much($d2_name,$d1_name) {
-        // Si une des 2 dates est nulle => return true
         $d2 = $this->$d2_name;
         $d1 = $this->$d1_name;
+        //debug("d2:"); debug($d2);
+        //debug("d1:"); debug($d1);
+        // Si une des 2 dates est nulle => return true
         if (!$d2 || !$d1) return true;
         /*
         $tz = new \DateTimeZone('Europe/Paris');
@@ -109,7 +128,7 @@ class Materiel extends Entity {
         if ($d2 < $d1) return false;
         // $d2 > $d1 oui mais pas trop...
         $diff = $d2->diff($d1);
-        //debug($diff->y);
+        //debug($diff->y); 
         return $diff->y < MAX_DIFF_YEARS;
         //return true;
     }
@@ -143,11 +143,29 @@ class MaterielsTable extends AppTable
     }
     public function dateIsValid($value, array $context) {
-        //debug($entity);
+        
+        /* 
+         * $value
+         * 
+         * La valeur du champ, souvent une string
+         * 
+         */
+        //debug($value);
+        
+        /* 
+         * $context
+         * 
+         * $context->newRecord est un boolean
+         * $context->data est l'entité complète, sous forme d'un tableau qui contient tous les champs (souvent des 'string')
+         * 
+         */
+        //debug($context);
+        
         // /^(((0[1-9]|[12]\d|3[01])\/(0[13578]|1[02])\/((19|[2-9]\d)\d{2}))|((0[1-9]|[12]\d|30)\/(0[13456789]|1[012])\/((19|[2-9]\d)\d{2}))|((0[1-9]|1\d|2[0-8])\/02\/((19|[2-9]\d)\d{2}))|(29\/02\/((1[6-9]|[2-9]\d)(0[48]|[2468][048]|[13579][26])|((16|[2468][048]|[3579][26])00))))$/g
         //$valid = preg_match("/^(0[1-9]|[1-2][0-9]|3[0-1])\/(0[1-9]|1[0-2])\/([1-2][0-9]{3})$/",$entity);
         //debug((bool)$valid);
         return (bool) preg_match("/^(0[1-9]|[1-2][0-9]|3[0-1])\/(0[1-9]|1[0-2])\/([1-2][0-9]{3})$/",$value);
+        
     }
@@ -178,6 +196,8 @@ class MaterielsTable extends AppTable
             return (bool) preg_match("/^(0[1-9]|[1-2][0-9]|3[0-1])\/(0[1-9]|1[0-2])\/([1-2][0-9]{3})$/",$entity);
         };
         */
+        
+        /*
         // return true si date n'est pas future (maxi = today)
         $dateIsNotFutureAndNotTooOld = function ($date_string) {
             $tz = new \DateTimeZone('Europe/Paris');
@@ -190,23 +210,25 @@ class MaterielsTable extends AppTable
             $today = $today->format('Ymd');
             // today - 50 ans = trop vieux !
             $date_too_old = $date_too_old->sub(new \DateInterval('P50Y'))->format('Ymd');
-            /*
+            /S
             $time = Time::now(); // On récupère la date et l'heure actuelles
             $today = (new date("$time->year-$time->month-$time->day"))->format('Ymd'); // On extrait la date on la formatte en un format comparable de type 20171231
-            */
-            /*
+            S/
+            /S
             $timeEntity = new time($entity);
             $dateEntity = (new date("$timeEntity->year-$timeEntity->month-$timeEntity->day"))->format('Ymd');
-            */
-            /*
+            S/
+            /S
             debug($entity); // ex: '20/04/2020'
             debug($today); // '20200717'
             debug($date); // '20200718' => pas bon
             debug($date_too_old);
-            */
+            S/
             //return ($today >= $date);
             return ($date<=$today && $date>$date_too_old);
         };
+        */
+        
         /*
         $dateIsNotTooFarAway = function ($date_string) {
             $tz = new \DateTimeZone('Europe/Paris');
@@ -278,11 +300,14 @@ class MaterielsTable extends AppTable
                 'rule' => 'dateIsValid',
                 'message' => "La date n'est pas valide (JJ/MM/AAAA)",
                 'provider' => 'table',
-            ])
+            ]);
+            // migré dans buildRules() car c'est plus de la validation de cohérence
+            /*
             ->add($f, 'acceptable', [
                 'rule' => $dateIsNotFutureAndNotTooOld,
                 'message' => "La date ne doit être ni future ni trop ancienne"
             ]);
+            */
         // - Date livraison
         $f = 'date_reception';
@@ -434,7 +459,8 @@ class MaterielsTable extends AppTable
         $validator->allowEmpty('unite_duree_garantie');
         return $validator;
-    }
+
+    } // validationDefault()
     public function checkStatus($check) { return ($check !== null && in_array($check, $this->ALL_STATUS)); }
@@ -524,8 +550,19 @@ class MaterielsTable extends AppTable
         };
-        // Check dates
-        $dateIsAfterDateAchatAndNotTooFar = function (Entity $entity) {
+        // Check DATES
+        
+        $dateAchatIsNotTooOld = function (Entity $entity) {
+            return $entity->check_date_is_not_too_old('date_acquisition');
+        };
+        $dateAchatIsNotFuture = function (Entity $entity) {
+            return $entity->check_date_is_not_future('date_acquisition');
+        };
+        $dateReceptionIsNotFuture = function (Entity $entity) {
+            return $entity->check_date_is_not_future('date_reception');
+        };
+            
+        $dateReceptionIsAfterDateAchatAndNotTooFar = function (Entity $entity) {
             return $entity->check_date_d2_gt_d1_but_not_too_much('date_reception','date_acquisition');
             //return $this->check_date_d2_gt_d1_but_not_too_much($entity->date_reception,$entity->date_acquisition);
             /*
@@ -548,7 +585,8 @@ class MaterielsTable extends AppTable
             return $diff->y < MAX_DIFF_YEARS;
             */ 
         };
-        $dateIsAfterDateReceptionAndNotTooFar = function (Entity $entity) {
+        $dateFinGarantieIsAfterDateReceptionAndNotTooFar = function (Entity $entity) {
+            //debug($entity);
             return $entity->check_date_d2_gt_d1_but_not_too_much('date_fin_garantie','date_reception');
             //return $this->check_date_d2_gt_d1_but_not_too_much($entity->date_fin_garantie,$entity->date_reception);
             /*
@@ -633,19 +671,30 @@ class MaterielsTable extends AppTable
         //$rules->add($rules->existsIn(['photo_id'], 'Photos'));
         ///$rules->add($rules->existsIn(['fournisseur_id'], 'Fournisseurs'));
-        // Check dates reception et fin garantie
-        $rules->add($dateIsAfterDateAchatAndNotTooFar, [
+        // Check dates achat, reception et fin garantie
+        $rules->add($dateAchatIsNotTooOld, [
+            'errorField' => 'date_acquisition',
+            'message' => "La date ne doit pas être trop ancienne"
+        ]);
+        $rules->add($dateAchatIsNotFuture, [
+            'errorField' => 'date_acquisition',
+            'message' => "La date ne doit pas être future"
+        ]);
+        $rules->add($dateReceptionIsNotFuture, [
+            'errorField' => 'date_reception',
+            'message' => "La date ne doit pas être future"
+        ]);
+        $rules->add($dateReceptionIsAfterDateAchatAndNotTooFar, [
             'errorField' => 'date_reception',
             'message' => "La date doit être postérieure à la date d'achat (mais pas trop loin)"
         ]);
-        $rules->add($dateIsAfterDateReceptionAndNotTooFar, [
+        $rules->add($dateFinGarantieIsAfterDateReceptionAndNotTooFar, [
             'errorField' => 'date_fin_garantie',
             'message' => "La date doit être postérieure à la date de livraison (mais pas trop loin)"
         ]);
-        
         return $rules;
-    }
+    } // buildRules()
     /*
     private function getEntity($id) {
@@ -618,7 +618,7 @@ if (isset($cpMateriel)) {
 		if ($IS_EDIT && $materiel->date_reception) $value_edit = $materiel->date_reception->format('d/m/Y');
         echo $this->Form->control('date_reception', [
             'type' => 'text',
-            'label' => 'Date de réception',
+            'label' => 'Date de livraison',
             'class' => 'datepicker',
             'placeholder' => $comment,
             'empty' => true,
@@ -615,8 +615,8 @@ if ($configuration-&gt;metrologie == 1) {
     if (h($entity->metrologie) == 0)
     $displayElement(__('Métrologie'), $metro);
 }
-$displayElement(__('Date d\'achat'), h($entity->date_acquisition));
-$displayElement(__('Date de reception'), h($entity->date_reception));
+$displayElement(__("Date d'achat"), h($entity->date_acquisition));
+$displayElement(__('Date de livraison'), h($entity->date_reception));
 if (! empty(h($entity->duree_garntie))) {
     $displayElement(__('Duree garantie'), h($entity->duree_garantie) . ' ' . h($entity->unite_duree_garantie));
 }