Bugfixed envoi mail sur validation

et supprimé ancienne fonction sendEmail() (remplacée par sendmail()) v4.101.11-3.7.9

Bugfixed envoi mail sur validation
et supprimé ancienne fonction sendEmail() (remplacée par sendmail()) v4.101.11-3.7.9
Etienne Pallier
1 parent 1628a1bf
Showing 4 changed files with 52 additions and 23 deletions Show diff stats
CHANGES.txt
README.md
src/Controller/AppController.php
src/Controller/MaterielsController.php
@@ -94,10 +94,11 @@ Outre ces changements, voici d&#39;autres changements importants :
 ======= CHANGES =======
 -------
-16/09/2020	v4.101.10-3.7.9 (EP)
+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)
 	- (e) Changé format date cache : + nb jours
+	- (b) Bugfixed envoi mail sur validation, et supprimé ancienne fonction sendEmail() (remplacée par sendmail())
 	- (b) Bugfixed tests => tous ok
 	- (b) Bugfixed recherche générale : on peut maintenant chercher aussi un projet
 	- (b) Bugfixed et réactivé test date reception > date achat "mais pas trop" (bug sur php5)
@@ -43,7 +43,7 @@ Logiciel testé et validé sur les configurations suivantes :
 --------------------------------------------------------------------------------------------
 Date: 15/09/2020
-Version: 4.101.10-3.7.9
+Version: 4.101.11-3.7.9
 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
@@ -2388,6 +2388,8 @@ class AppController extends Controller
         }
     }
+    
+    
     /**
      * Envoi un mail avec un sujet, contenant un message à destination d'une liste de mails, selon l'action effectuée.
      *
@@ -2398,16 +2400,17 @@ class AppController extends Controller
      * @param $msg :
      *            Message à envoyer. Si $msg n'est pas renseigné, un message par défaut sera généré.
      */
+    /* (EP202009) remplacé par sendmail() plus bas
     public function sendEmail($entity, $subject=null, $msg=null)
     {
-        /*
+        /S
          * $_SESSION['Auth']['User'] pour retrouver TOUTES les infos de la session courante (tout est du string) :
          * nom $_SESSION['Auth']['User']['sn'][0]
          * prenom $_SESSION['Auth']['User']['givenname'][0]
          * mail $_SESSION['Auth']['User']['mail'][0]
          * login $_SESSION['Auth']['User']['xxx'][0] /!\ Ce champ est suceptible de changer de nom, dans les tests ce champ est ['cn'][0]
          * mdp $_SESSION['Auth']['User']['userpassword'][0]
-         */
+         S/
         $configuration = $this->confLabinvent;
         $action = $this->request->getAttribute('params')['action']; // add or edit or delete or ...
@@ -2526,7 +2529,7 @@ class AppController extends Controller
                 // Cela a été mis en commentaire car de toute façon l'utilisateur va voir un administratif pour faire valider sa fiche,
                 // Pas la peine de spam l'administration de mails non plus hein !
-                /*
+                /S
                  * if ($action != 'statusValidated' && $action != 'statusArchived') {
                  * $mailsAdmin = TableRegistry::getTableLocator()->get('Users')->find()->select('email')
                  * ->where(['role =' => 'Administration'])
@@ -2535,7 +2538,7 @@ class AppController extends Controller
                  * $mailList[sizeof($mailList)] = $mailsAdmin[$i]['email'];
                  * }
                  * }
-                 */
+                 S/
             }
         } // Materiel
@@ -2547,13 +2550,13 @@ class AppController extends Controller
         } else if ($entity instanceof Emprunt) {
             ;
         }
-        /*
+        /S
          * @todo:
          * else if ($entity instanceof Configuration) {
          * ;
          * }
          * ... etc ... (il faut qu'on soit plus précis)
-         */
+         S/
         // Si l'envoi à la liste spécifiée est activé (et que l'action ne correspond pas à 'printLabelRuban'):
         $specificUsers = [];
@@ -2605,7 +2608,11 @@ class AppController extends Controller
         }
         return $mailList;
     }
+    */
+    
+    
+    
     // Fonction d'envoi de mails
     private function sendEmailTo($subject, $msg, $mail, $config)
     {
@@ -2696,7 +2703,7 @@ class AppController extends Controller
      * @param $msg :
      *            Message à envoyer. Si $msg n'est pas renseigné, un message par défaut sera généré.
      */
-    public function sendmail($entity, $mode, $subject = null, $msg = null)
+    public function sendmail($entity, $mode=3, $subject = null, $msg = null)
     {
         //$this->myDebug("photo send mail");
@@ -3003,26 +3010,32 @@ class AppController extends Controller
 			$role =$role .'en tant que ' .'Utilisateur';
 			//}
 			*/
-			if ($entity != null && !in_array($action, ['delete','statusValidated','statusCreated'])) {
-				$message .= "\n\nVeuillez vérifier et compléter si besoin la fiche correspondante.";
-				$message .= "\n\nVous recevez ce message " . $role;
+			//if ($entity != null && !in_array($action, ['delete','statusValidated','statusCreated'])) {
+			if ($entity != null) {
+    		    if (!in_array($action, ['delete']))
+    				$message .= "\n\nVeuillez vérifier et compléter si besoin la fiche correspondante.";
+			    $message .= "\n\nVous recevez ce message " . $role;
 				//en fonction du mode à l'appel, on utilise un envoi de mail différent
 				switch($mode) {
-						//si le mode 1 est sélectionné c'est un envoi de mail avec ajout d'une photo
-						//explique le document qui a été ajouté, et le met en pièce jointe
+				
+				    //si le mode 1 est sélectionné c'est un envoi de mail avec ajout d'une photo
+					//explique le document qui a été ajouté, et le met en pièce jointe
 					case 1 :
 						$this->sendEmailImgTo("$subject", $message, $mail, $configuration, $nomFic);
 						break;
-						//si le mode 2 est sélectionné c'est un envoi de mail avec ajout d'un document
-						//à personnaliser
+					
+					//si le mode 2 est sélectionné c'est un envoi de mail avec ajout d'un document
+					//à personnaliser
 					case 2 :
 						$this->sendEmailPJTo("$subject", $message, $mail, $configuration, $nomFic);
 						break;
-						//si le mode defaut est sélectionné c'est un simple envoi de mail
-						//récapitule une action (mieux pour les add...)
+
+					//si le mode defaut est sélectionné c'est un simple envoi de mail
+					//récapitule une action (mieux pour les add...)
 					default :
 						$this->sendEmailTo("$subject", $message, $mail, $configuration);
 						break;
+				
 				} // switch
 			}
 	    } //foreach ($mailList as $mail)
@@ -1670,7 +1670,9 @@ class MaterielsController extends AppController {
                 */
                 $this->Flash->success(__("Le matériel a bien été $verb"));
                 if ($IS_ADD) {
-                    $this->sendEmail($materiel);
+                    //(EP202009)
+                    //$this->sendEmail($materiel);
+                    $this->sendmail($materiel);
                     $id = $materiel->id;
                 }
                 //$this->dlog("Materiel $verb = '$materiel->designation' (id=$id)");
@@ -2162,7 +2164,10 @@ class MaterielsController extends AppController {
         if ($this->Materiels->delete($materiel)) {
             $this->Flash->success(__('Le matériel a bien été supprimé'));
             $this->ilog("Materiel $verb = '$materiel' (id=$id)");
-            $this->sendEmail($materiel);
+            //(EP202009)
+            //$this->sendEmail($materiel);
+            $this->sendmail($materiel);
+            
         } else
             $this->Flash->error(__('Le matériel n\'a pas pu être supprimé'));
         return $this->redirect([
@@ -2287,6 +2292,7 @@ class MaterielsController extends AppController {
         } // VALIDATED
+        
         // SAVE
         $success = False;
         // - SAVE KO
@@ -2304,7 +2310,11 @@ class MaterielsController extends AppController {
             //évite le double envoi car il y a un envoi de mail a la fonction add et à l'état created
             //évite les envois succcessifs si qqn fait une erreur sur la fiche, valide la fiche, et dévalide pour remodifier
             //if ($newStatus == 'ARCHIVED' || $newStatus == 'TOBEARCHIVED') {
-            if (in_array($newStatus,['ARCHIVED','TOBEARCHIVED','VALIDATED'])) $this->sendmail($materiel,3); 
+            if (in_array($newStatus, [
+                //'CREATED',    // Pour Création ET Dé-validation
+                'ARCHIVED','TOBEARCHIVED','VALIDATED'])
+                ) 
+                $this->sendmail($materiel,3); 
 		} // SAVE OK
         if (! $onlyOneMateriel)
@@ -3500,7 +3510,9 @@ class MaterielsController extends AppController {
             // Envoi email seulement si etiquette posée
             if ($printed) {
                 // $this->sendEmailToManagement($id);
-                $this->sendEmail($materiel);
+                //(EP202009)
+                //$this->sendEmail($materiel);
+                $this->sendmail($materiel);
             }
         } else {
             //debug("save KO");
@@ -3545,7 +3557,10 @@ class MaterielsController extends AppController {
         //$this->OLD_printLabel('ruban', $id, $from);
         // (EP 202006) NEW METHODE => nécessite le template Materiels/xml/print_label_ruban.ctp
         $this->printLabel('ruban', $id, $from);
-        $this->sendEmail($this->Materiels->get($id));
+        //(EP202009)
+        //$this->sendEmail($this->Materiels->get($id));
+        $this->sendmail($this->Materiels->get($id));
+        
     }
     /**