From 87574acf1c83a08f6065b9c6cb13567e0da5a576 Mon Sep 17 00:00:00 2001
From: Etienne Pallier <epallier@irap.omp.eu>
Date: Mon, 28 Sep 2020 10:49:12 +0200
Subject: [PATCH] Ajout d'explications sur la page des notifications et de liens vers la page de log

---
 CHANGES.txt                          |  3 ++-
 README.md                            |  2 +-
 src/Template/Pages/notifications.ctp | 60 +++++++++++++++++++++++++++++++++++++++++-------------------
 3 files changed, 44 insertions(+), 21 deletions(-)

diff --git a/CHANGES.txt b/CHANGES.txt
index b45a12a..7a7a280 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -113,7 +113,8 @@ Outre ces changements, voici d'autres changements importants :
 ======= CHANGES =======
 
 -------
-28/09/2020	v4.103.10-3.7.9 (EP)
+28/09/2020	v4.103.11-3.7.9 (EP)
+	- (e) Ajout d'explications sur la page des notifications et de liens vers la page de log
 	- (e) Ajout d'une page web "/pages/Notifications" qui donne l'état actuel des notifications (quelles notifs pour quelles actions)
 	(sur le modèle de /pages/acls)
 
diff --git a/README.md b/README.md
index e3b4ccd..9db59c1 100644
--- a/README.md
+++ b/README.md
@@ -43,7 +43,7 @@ Logiciel testé et validé sur les configurations suivantes :
 --------------------------------------------------------------------------------------------
 
 Date: 25/09/2020
-Version: 4.103.10-3.7.9
+Version: 4.103.11-3.7.9
 
 	
 HISTORIQUE DES CHANGEMENTS DE VERSION : voir le fichier CHANGES.txt (ou la page web /pages/changes)
diff --git a/src/Template/Pages/notifications.ctp b/src/Template/Pages/notifications.ctp
index 7272338..ee86add 100644
--- a/src/Template/Pages/notifications.ctp
+++ b/src/Template/Pages/notifications.ctp
@@ -18,7 +18,15 @@ use App\Controller\QrCodesController;
 // Variables passées à la vue par le controleur
 $lab_name = $lab_name;
 
-
+$log_info_page_link = $this->Html->link("log", ['action' => "logs?level=info"]);
+
+$log_page = '/pages/acls'; 
+$nice_notifs = [
+    //'both' => 'log et courriel',
+    'both' => $log_info_page_link.' et courriel',
+    'log' => $log_info_page_link.' seulement',
+    'mail' => 'mail seulement',
+];
 
 //$controllers = ['Materiels'];
 $controllers = [
@@ -49,7 +57,7 @@ $controllers = [
 
 
 
-function displayNotificationsForController($c, $lab_name) {
+function displayNotificationsForController($c, $lab_name, $nice_notifs) {
     
     ?>
 
@@ -71,8 +79,10 @@ function displayNotificationsForController($c, $lab_name) {
     </li>
     </ul>
     <i>
-    Veuillez ensuite partager ces modifications en les intégrant au code source général du logiciel (ainsi que toute autre modification effectuée) 
-    via la commande "./PUSH_MODIFS"
+    Pour modifier la liste spécifique d'emails, aller sur la page de configuration du logiciel.
+    <br>
+    (Veuillez ensuite partager ces modifications en les intégrant au code source général du logiciel (ainsi que toute autre modification effectuée) 
+    via la commande "./PUSH_MODIFS")
     </i>
     <p>
     
@@ -97,7 +107,7 @@ function displayNotificationsForController($c, $lab_name) {
 			<!--  Notifications pour chaque action ($a) -->
     		<tr>
     			<th class="action" scope="row"><?=$a?></th>
-		        <td><?=$notif?></td>
+		        <td><?=$nice_notifs[$notif]?></td>
     		</tr>
 		<?php 
 		}
@@ -157,12 +167,36 @@ th.action {
 	<br />
 	<p>
 		<!--  Lien GDOC -->
-		<br>
 		=> <a href="https://docs.google.com/document/d/1JTi3YX6X33asn9vfPGAPaFZ3u8IMGnAqCf2Ge0nWdtI/edit#heading=h.jaa6i3mtbmnr">
 			Lien vers le chapitre sur les Notifications dans la documentation technique (TODO)
 		</a>
 	</p>
     <hr />
+
+	<p>
+    <b><u>Par défaut, les notifications par email sont envoyées :</u></b>
+    <ul>
+        <li>
+    		à l'utilisateur destinataire du matériel
+        </li>
+        <li>
+    		au gestionnaire de référence du matériel
+        </li>
+        <li>
+    		aux responsables (thématique ou métier) du matériel
+        </li>
+        <li>
+    		et aussi à une liste de mails prédéfinie et configurable via la page web de configuration générale
+        </li>
+        <li>
+    		(TODO) peut-être plus tard, aux responsables (scientifique et chef projet) du projet auquel le matériel est associé
+        </li>
+    </ul>
+    </p>
+    <hr />
+
+    
+    
     
 
 	<u><b>Tableaux des notifications par entité</b></u>
@@ -180,18 +214,6 @@ th.action {
 	?>
 	<hr />
 	
-    <p>
-    <u><b>Légende</b></u> :
-    <ul>
-    	<li><b>both = log + mail</b></li>
-    	<li><b>log = log seulement</b></li>
-    	<li><b>mail = mail seulement</b></li>
-    </ul>
-    </p>
-    <hr />
-	
-	
-    
 
 <?php
 
@@ -208,7 +230,7 @@ foreach ($controllers as $controller) {
     $controller = new $controller();
     if ($controller->getNotifications()) {
         echo '<br>';
-        displayNotificationsForController($controller, $lab_name);
+        displayNotificationsForController($controller, $lab_name, $nice_notifs);
         echo '<br>';
         echo "<a href='#'>Haut de page</a>";
         echo '<hr />';
--
libgit2 0.21.2