Etienne Pallier / labinvent

Commit 8f1c67692dabfa10deb610669ea2b82312868c26

Authored by Etienne Pallier
1 parent 538f5992
Exists in master and in 2 other branches dev, dev-IRAP

bugfix email pourri envoyé quand on fait l'archivage d'un matos

Showing 1 changed file with 26 additions and 22 deletions   Show diff stats
Inline Side-by-side
src/Controller/AppController.php
  Diff comments   View file @8f1c676
... ... @@ -18,6 +18,7 @@ namespace App\Controller;
18 18 use Cake\Controller\Controller;
19 19 use Cake\Event\Event;
20 20 use Cake\ORM\TableRegistry;
  21 +use Cake\ORM\Locator\TableLocator;
21 22 use Cake\Mailer\Email;
22 23 use Cake\Core\Configure;
23 24 use Cake\I18n\Time;
... ... @@ -380,13 +381,13 @@ class AppController extends Controller
380 381 ]);
381 382 // On charge la configuration
382 383 /*
383   - $this->confLabinvent = TableRegistry::get('Configurations')->find()
  384 + $this->confLabinvent = TableLocator::get('Configurations')->find()
384 385 ->where([
385 386 'id =' => 1
386 387 ])
387 388 ->first();
388 389 */
389   - $this->confLabinvent = TableRegistry::get('Configurations')->find()->first();
  390 + $this->confLabinvent = TableLocator::get('Configurations')->find()->first();
390 391 }
391 392  
392 393 /**
... ... @@ -438,7 +439,7 @@ class AppController extends Controller
438 439 $configuration = $this->confLabinvent;
439 440 $role = $this->getUserRole($user);
440 441 /*
441   - * $role = TableRegistry::get('Users')->find()
  442 + * $role = TableLocator::get('Users')->find()
442 443 * ->where(['username' => $user[$configuration->authentificationType_ldap][0]])
443 444 * ->first()['role'];
444 445 */
... ... @@ -476,7 +477,7 @@ class AppController extends Controller
476 477 if (! $this->CURRENT_PRIVILEDGED_USER) {
477 478 $configuration = $this->confLabinvent;
478 479 $username = $user ? $user[$configuration->ldap_authenticationType][0] : $this->LdapAuth->user($configuration->ldap_authenticationType)[0];
479   - $priviledgedUser = TableRegistry::get('Users')->find()
  480 + $priviledgedUser = TableLocator::get('Users')->find()
480 481 ->where([
481 482 'username' => $username
482 483 ])
... ... @@ -648,7 +649,7 @@ class AppController extends Controller
648 649 // (c'est à dire s'il n'est pas dans la table "utilisateurs")
649 650 $priviledgedUser = $this->getTablePriviledgedUserFromCurrentSessionUserIfExists();
650 651 /*
651   - * $user = TableRegistry::get('Users')->find()
  652 + * $user = TableLocator::get('Users')->find()
652 653 * ->where(['username' => $this->LdapAuth->user($configuration->authentificationType_ldap)[0]])
653 654 * ->first();
654 655 * $role = $user['role'];
... ... @@ -711,12 +712,12 @@ class AppController extends Controller
711 712 }
712 713 }
713 714  
714   - $this->set('idGmNa', TableRegistry::get('GroupesMetiers')->find()
  715 + $this->set('idGmNa', TableLocator::get('GroupesMetiers')->find()
715 716 ->where([
716 717 'nom =' => 'N/A'
717 718 ])
718 719 ->first()['id']);
719   - $this->set('idGtNa', TableRegistry::get('GroupesThematiques')->find()
  720 + $this->set('idGtNa', TableLocator::get('GroupesThematiques')->find()
720 721 ->where([
721 722 'nom =' => 'N/A'
722 723 ])
... ... @@ -752,9 +753,9 @@ class AppController extends Controller
752 753  
753 754 static function isLabinventDebugMode()
754 755 {
755   - return TableRegistry::get('Configurations')->find()->first()->mode_debug;
  756 + return TableLocator::get('Configurations')->find()->first()->mode_debug;
756 757 /*
757   - return TableRegistry::get('Configurations')->find()
  758 + return TableLocator::get('Configurations')->find()
758 759 ->where([
759 760 'id =' => 1
760 761 ])
... ... @@ -881,14 +882,14 @@ class AppController extends Controller
881 882  
882 883 if ($materiel->groupes_metier_id != null && $materiel->groupes_metier_id != 1) {
883 884 // Le ..!= 1 c'est parce que le groupe métier/thématique d'id 1 correspond au groupe N/A, soit aucun groupe
884   - $mailsRespMetier = TableRegistry::get('Users')->find()
  885 + $mailsRespMetier = TableLocator::get('Users')->find()
885 886 ->select('email')
886 887 ->where([
887 888 'role =' => 'Responsable',
888 889 'groupes_metier_id =' => $materiel->groupes_metier_id
889 890 ])
890 891 ->toArray();
891   - $mailRespThematique = TableRegistry::get('Users')->find()
  892 + $mailRespThematique = TableLocator::get('Users')->find()
892 893 ->select('email')
893 894 ->where([
894 895 'role =' => 'Responsable',
... ... @@ -911,7 +912,7 @@ class AppController extends Controller
911 912  
912 913 /*
913 914 * if ($action != 'statusValidated' && $action != 'statusArchived') {
914   - * $mailsAdmin = TableRegistry::get('Users')->find()->select('email')
  915 + * $mailsAdmin = TableLocator::get('Users')->find()->select('email')
915 916 * ->where(['role =' => 'Administration'])
916 917 * ->toArray();
917 918 * for ($i = 0; $i < sizeof($mailsAdmin); $i ++) {
... ... @@ -966,7 +967,7 @@ class AppController extends Controller
966 967 if ($specificUsers)
967 968 $role = 'car vous etes dans la liste spécifique des emails de LabInvent. Pour faire retirer votre mail de cette liste, veuillez contacter un SuperAdmin.';
968 969 else {
969   - $role = TableRegistry::get('Users')->find()
  970 + $role = TableLocator::get('Users')->find()
970 971 ->select('role')
971 972 ->where([
972 973 'email =' => $mail
... ... @@ -1159,14 +1160,14 @@ class AppController extends Controller
1159 1160  
1160 1161 if ($materiel->groupes_metier_id != null && $materiel->groupes_metier_id != 1) {
1161 1162 // Le ..!= 1 c'est parce que le groupe métier/thématique d'id 1 correspond au groupe N/A, soit aucun groupe
1162   - $mailsRespMetier = TableRegistry::get('Users')->find()
  1163 + $mailsRespMetier = TableLocator::get('Users')->find()
1163 1164 ->select('email')
1164 1165 ->where([
1165 1166 'role =' => 'Responsable',
1166 1167 'groupes_metier_id =' => $materiel->groupes_metier_id
1167 1168 ])
1168 1169 ->toArray();
1169   - $mailRespThematique = TableRegistry::get('Users')->find()
  1170 + $mailRespThematique = TableLocator::get('Users')->find()
1170 1171 ->select('email')
1171 1172 ->where([
1172 1173 'role =' => 'Responsable',
... ... @@ -1189,7 +1190,7 @@ class AppController extends Controller
1189 1190  
1190 1191 /*
1191 1192 * if ($action != 'statusValidated' && $action != 'statusArchived') {
1192   - * $mailsAdmin = TableRegistry::get('Users')->find()->select('email')
  1193 + * $mailsAdmin = TableLocator::get('Users')->find()->select('email')
1193 1194 * ->where(['role =' => 'Administration'])
1194 1195 * ->toArray();
1195 1196 * for ($i = 0; $i < sizeof($mailsAdmin); $i ++) {
... ... @@ -1256,7 +1257,7 @@ class AppController extends Controller
1256 1257 //$mailList[0] = $entity->materiel->email_responsable;
1257 1258  
1258 1259 //MI - gestionaire ratachée au matériel
1259   - $mailList[1]= TableRegistry::get('Users')->find()
  1260 + $mailList[1]= TableLocator::get('Users')->find()
1260 1261 ->select('email')
1261 1262 ->where([
1262 1263 'role =' => 'Administration',
... ... @@ -1304,9 +1305,12 @@ class AppController extends Controller
1304 1305 // Génération du message "Vous recevez ce message en tant que $role"
1305 1306 // Si $role inexistant (lorsque c'est un mail de la liste entrée en configuration), le message est plutot "Vous recevez ce message car vous avez demandé à le recevoir. [...]"
1306 1307 //if ($specificUsers)
1307   - $role = 'car vous etes dans la liste spécifique des emails de LabInvent. Pour faire retirer votre mail de cette liste, veuillez contacter un SuperAdmin. + ';
  1308 + $role = 'car vous etes dans la liste spécifique des emails de LabInvent. Pour faire retirer votre mail de cette liste, veuillez contacter un super-administrateur.';
  1309 + /* (EP 13/319) : à quoi sert toute cette suite du texte du mail ???
  1310 + * Ca sent le bon vieux copier-coller sans réfléchir...
1308 1311 //else {
1309   - $role = $role. TableRegistry::get('Users')->find()
  1312 + //$role = $role. TableRegistry::get('Users')->find()
  1313 + $role = $role. TableLocator::get('Users')->find()
1310 1314 ->select('role')
1311 1315 ->where([
1312 1316 'email =' => $mail
... ... @@ -1315,8 +1319,8 @@ class AppController extends Controller
1315 1319 // Default role is Utilisateur (for people in LDAP but without priviledge, not in the users table)
1316 1320 // if (is_null($role))
1317 1321 $role =$role .'en tant que ' .'Utilisateur';
1318   - // $role = $role;
1319 1322 //}
  1323 + */
1320 1324 if ($entity != null && ! in_array($action, [
1321 1325 'delete',
1322 1326 'statusValidated',
... ... @@ -1331,12 +1335,12 @@ class AppController extends Controller
1331 1335 //si le mode 1 est sélectionné c'est un envoi de mail avec ajout d'une photo
1332 1336 //explique le document qui a été ajouté, et le met en pièce jointe
1333 1337 case 1 :
1334   - $this->sendEmailImgTo("$subject", $message, $mail, $configuration,$nomFic);
  1338 + $this->sendEmailImgTo("$subject", $message, $mail, $configuration, $nomFic);
1335 1339 break;
1336 1340 //si le mode 2 est sélectionné c'est un envoi de mail avec ajout d'un document
1337 1341 //à personnaliser
1338 1342 case 2 :
1339   - $this->sendEmailPJTo("$subject", $message, $mail, $configuration,$nomFic);
  1343 + $this->sendEmailPJTo("$subject", $message, $mail, $configuration, $nomFic);
1340 1344 break;
1341 1345 //si le mode defaut est sélectionné c'est un simple envoi de mail
1342 1346 //récapitule une action (mieux pour les add...)
... ...