Bugfix et Améliorations Emprunt

Etienne Pallier
1 parent c95206aa
Showing 3 changed files with 53 additions and 10 deletions Show diff stats
README.md
src/Model/Table/EmpruntsTable.php
src/Template/Materiels/view.ctp
@@ -53,11 +53,11 @@ Logiciel testé et validé sur les configurations suivantes :
 VERSION ACTUELLE
-Date: 03/06/2019
-Version: 2.12.25
+Date: 04/06/2019
+Version: 2.12.26
 Author: EP
 Commentaire: 
-	LDAP CACHED v3
+	Bugfix et Améliorations Emprunt
 IMPORTANT : 
 	- Pour connaitre la version actuelle, taper "./VERSION"
@@ -93,6 +93,9 @@ La liste ci-dessous n&#39;est plus à jour, elle est désormais en ligne ici : https
 -----------------------------------------------------------------------------------------------------------
+03/06/2019	Version: 2.12.24-25 (EP)
+			LDAP CACHED v3
+	
 24/05/2019	Version: 2.12.22-23 (EP)
 			Bugfix QrCode : http et https bien gérés
 			Update du script install/installation.sh pour etre plus logique
@@ -56,9 +56,10 @@ class EmpruntsTable extends AppTable
         $validator->boolean('emprunt_interne')->allowEmpty('emprunt_interne');
-        //$validator->notEmpty('nom_emprunteur_int', 'Le champ doit être rempli');
+        /*
         $validator->notEmpty('nom_emprunteur_int', 'Le champ doit être rempli', function ($context) {
             //dump($context);
+            debug($context);
             //if ($context['providers']['entity']['emprunt_interne'] !== null) {
             if ( !empty($context['data']) && !is_null($context['data']['emprunt_interne']) ) {
                 return $context['data']['emprunt_interne'];
@@ -66,7 +67,6 @@ class EmpruntsTable extends AppTable
             }
         });
-        //$validator->notEmpty('email_emprunteur_int', "Cet Email n'est pas valide");
         $validator->notEmpty('email_emprunteur_int', "Cet Email n'est pas valide", function ($context) {
             //if ($context['providers']['entity']['emprunt_interne'] !== null) {
             if ( !empty($context['data']) && !is_null($context['data']['emprunt_interne']) ) {
@@ -74,9 +74,9 @@ class EmpruntsTable extends AppTable
                 //return $context['providers']['entity']['emprunt_interne'];
             }
         });
-
-
-                
+        */
+        
+        // Ce champ s'appelle "Pièce" (emprunt interne)
         $validator->notEmpty('e_lieu_detail', 'Le champ doit être rempli pour un emprunt interne.', 
             function ($context) {
                     //if ($context['providers']['entity']['emprunt_interne'] !== null) {
@@ -93,9 +93,11 @@ class EmpruntsTable extends AppTable
             'provider' => 'table'
         ]);
+        // Ce champ s'appelle "Lieu" (emprunt externe) 
         $validator->notEmpty('laboratoire', 'Le champ doit être rempli pour un emprunt externe.',
             function ($context) {
                 if ( !empty($context['data']) && !is_null($context['data']['emprunt_interne']) ) {
+                    //debug($context['data']['emprunt_interne']);
                     return ! $context['data']['emprunt_interne'];
                 }
             }
@@ -131,7 +133,13 @@ class EmpruntsTable extends AppTable
                  */
                 }
             }
-        );
+        )->add('email_emprunteur_ext', 'valid', [
+                'rule' => [
+                    'check_mail'
+                ],
+                'message' => "Cet email n'est pas valide",
+                'provider' => 'table'
+        ]);
         $validator->allowEmpty('commentaire')->add('commentaire', 'valid', [
             'rule' => [
@@ -141,6 +149,24 @@ class EmpruntsTable extends AppTable
             'provider' => 'table'
         ]);
+        
+
+        /*
+        $validator->notEmpty('site_id', 'Il faut préciser le site',
+            function ($context) {
+                if ( !empty($context['data']) && !is_null($context['data']['emprunt_interne']) ) {
+                    return $context['data']['emprunt_interne'];
+                }
+            }
+        )->add('site_id', 'valid', [
+                'rule' => function ($data, $provider) {
+                    return ($data != 9);
+                 },
+                'message' => "Il faut préciser le site (pour un emprunt interne)",
+                'provider' => 'table'
+        ]);
+        */
+        
         return $validator;
     }
@@ -166,6 +192,7 @@ class EmpruntsTable extends AppTable
             }
             return true;
         };
+
         $rules->add($checkReturnDateIsAfterLoanDate, [
             'errorField' => 'date_retour_emprunt',
@@ -176,9 +203,22 @@ class EmpruntsTable extends AppTable
             'materiel_id'
         ], 'Materiels'));
+        /* (EP) ne sert à rien, c'est toujours le cas
         $rules->add($rules->existsIn([
             'site_id'
         ], 'Sites'));
+        */
+        // (EP 4/6/19) Le site doit etre précisé, on ne laisse pas N/A (id=9)
+        $checkSiteIsGiven = function ($entity) {
+            // Si emprunt externe, ne pas appliquer cette règle
+            if (! $entity->emprunt_interne) return TRUE;
+            return ($entity->site_id != 9);
+        };
+        $rules->add($checkSiteIsGiven, [
+            'errorField' => 'site_id',
+            'message' => 'Il faut préciser le site'
+        ]);
+        
         return $rules;
     }
@@ -622,7 +622,7 @@ $bStyle = &#39;margin:0&#39;;
 				<th><?=__('N°')?></th>
 				<th><?=__('Emprunteur')?></th>
 				<th><?=__('Type d\'Emprunt')?></th>
-				<th><?=__('Où')?></th>
+				<th><?=__('Lieu')?></th>
 				<th><?=__('Date de l\'emprunt')?></th>
 				<th><?=__('Date de retour')?></th>
 			</tr>