Uses trig utilities whenever it is possible.

diff --git a/modules/admin.php b/modules/admin.php
index 84af751..97f3d2c 100644 (file)
--- a/modules/admin.php
+++ b/modules/admin.php
@@ -759,6 +759,7 @@ class AdminModule extends PLModule
                     send_warning_homonyme($prenom, $nom, $forlife, $loginbis);
                     switch_bestalias($target, $loginbis);
                     $op = 'list';
+                    $page->trigSuccess('Email envoyé à ' . $forlife . '.');
                     break;
 
                 case 'correct':
@@ -769,6 +770,7 @@ class AdminModule extends PLModule
                     XDB::execute("REPLACE INTO homonymes (homonyme_id,user_id) VALUES({?},{?})", $target, $target);
                     send_robot_homonyme($prenom, $nom, $forlife, $loginbis);
                     $op = 'list';
+                    $page->trigSuccess('Email envoyé à ' . $forlife . ', alias supprimé.');
                     break;
             }
         }
@@ -837,7 +839,6 @@ class AdminModule extends PLModule
         if ($validate) {
             S::assert_xsrf_token();
 
-            $new_deces = array();
             $res = XDB::iterRow("SELECT user_id,matricule,nom,prenom,deces FROM auth_user_md5 WHERE promo = {?}", $promo);
             while (list($uid,$mat,$nom,$prenom,$deces) = $res->next()) {
                 $val = Env::v($mat);
@@ -846,7 +847,7 @@ class AdminModule extends PLModule
                 }
 
                 XDB::execute('UPDATE auth_user_md5 SET deces={?} WHERE matricule = {?}', $val, $mat);
-                $new_deces[] = array('name' => "$prenom $nom", 'date' => "$val");
+                $page->trigSuccess('Ajout du décès de ' . $prenom . " " . $nom . ' le ' . $val . '.');
                 if($deces == '0000-00-00' || empty($deces)) {
                     require_once('notifs.inc.php');
                     register_watch_op($uid, WATCH_DEATH, $val);
@@ -854,7 +855,6 @@ class AdminModule extends PLModule
                     user_clear_all_subs($uid, false);   // by default, dead ppl do not loose their email
                 }
             }
-            $page->assign('new_deces',$new_deces);
         }
 
         $res = XDB::iterator('SELECT matricule, nom, prenom, deces FROM auth_user_md5 WHERE promo = {?} ORDER BY nom,prenom', $promo);

diff --git a/modules/email.php b/modules/email.php
index 3776043..43e4c19 100644 (file)
--- a/modules/email.php
+++ b/modules/email.php
@@ -220,7 +220,6 @@ class EmailModule extends PLModule
 
         if ($action == 'remove' && $email) {
             $retour = $redirect->delete_email($email);
-            $page->assign('retour', $retour);
         }
 
         if ($action == 'active' && $email) {
@@ -242,15 +241,29 @@ class EmailModule extends PLModule
             $actifs = Env::v('emails_actifs', Array());
             print_r(Env::v('emails_rewrite'));
             if (Env::v('emailop') == "ajouter" && Env::has('email')) {
-                $page->assign('retour', $redirect->add_email(Env::v('email')));
+                $retour = $redirect->add_email(Env::v('email'));
             } elseif (empty($actifs)) {
-                $page->assign('retour', ERROR_INACTIVE_REDIRECTION);
+                $retour = ERROR_INACTIVE_REDIRECTION;
             } elseif (is_array($actifs)) {
-                $page->assign('retour', $redirect->modify_email($actifs,
-                    Env::v('emails_rewrite',Array())));
+                $retour = $redirect->modify_email($actifs, Env::v('emails_rewrite', Array()));
             }
         }
 
+        switch ($retour) {
+          case ERROR_INACTIVE_REDIRECTION:
+            $page->trigError('Tu ne peux pas avoir aucune adresse de redirection active, sinon ton adresse '
+                             . $forlife . '@' . $globals->mail->domain . ' ne fonctionnerait plus.');
+            break;
+          case ERROR_INVALID_EMAIL:
+            $page->trigError('Erreur: l\'email n\'est pas valide.');
+            break;
+          case ERROR_LOOP_EMAIL:
+            $page->trigError('Erreur : ' . $forlife . '@' . $globals->mail->domain
+                             . ' ne doit pas être renvoyé vers lui-même, ni vers son équivalent en '
+                             . $globals->mail->domain2 . ' ni vers polytechnique.edu.');
+            break;
+        }
+
         $res = XDB::query(
                 "SELECT  alias
                    FROM  virtual

diff --git a/modules/lists.php b/modules/lists.php
index 36b247b..8cb8723 100644 (file)
--- a/modules/lists.php
+++ b/modules/lists.php
@@ -810,12 +810,16 @@ class ListsModule extends PLModule
                                  $type, $liste.$app.$domain);
                 }
                 $page->assign('deleted', true);
+                $page->trigSuccess('La liste a été détruite !');
             } else {
                 $page->kill('Une erreur est survenue lors de la suppression de la liste.<br />'
                          . 'Contact les administrateurs du site pour régler le problème : '
                          . '<a href="mailto:support@polytechnique.org">support@polytechnique.org</a>');
             }
         } elseif (list($details,$options) = $this->client->get_owner_options($liste)) {
+            if (!$details['own']) {
+                $page->trigWarning('Tu n\'es pas administrateur de la liste, mais du site.');
+            }
             $page->assign_by_ref('details', $details);
             $page->assign_by_ref('options', $options);
             $page->assign('bogo_level', $this->client->get_bogo_level($liste));

diff --git a/modules/profile.php b/modules/profile.php
index 039f71a..8dace8e 100644 (file)
--- a/modules/profile.php
+++ b/modules/profile.php
@@ -677,6 +677,8 @@ class ProfileModule extends PLModule
 
             // on vient de recevoir une requete, differente de l'ancien nom d'usage
             if ($nom_usage == $nom) {
+                $page->trigWarning('Le nom d\'usage que tu demandes est identique à ton nom à l\'X, '
+                                   . 'aucune modification n\'a donc été effectuée.');
                 $page->assign('same', true);
             } else { // le nom de mariage est distinct du nom à l'X
                 // on calcule l'alias pour l'afficher

diff --git a/templates/admin/deces_promo.tpl b/templates/admin/deces_promo.tpl
index 9a64ba3..71c084a 100644 (file)
--- a/templates/admin/deces_promo.tpl
+++ b/templates/admin/deces_promo.tpl
@@ -22,10 +22,6 @@
 
 
 
-{foreach from=$new_deces item=i}
-<p class="erreur">Ajout du décès de {$i.name} le {$i.date}.</p>
-{/foreach}
-
 <form action="admin/deaths" method="post">
   <table class="tinybicol">
     <tr>

diff --git a/templates/admin/homonymes.tpl b/templates/admin/homonymes.tpl
index f37f12c..33c88c4 100644 (file)
--- a/templates/admin/homonymes.tpl
+++ b/templates/admin/homonymes.tpl
@@ -22,12 +22,6 @@
 
 <h1>Gestion des homonymes</h1>
 
-{if $op eq 'mail'}
-<p class="erreur">email envoyé à {$forlife}</p>
-{elseif $op eq 'correct'}
-<p class="erreur">email envoyé à {$forlife}, alias supprimé</p>
-{/if}
-
 {if $op eq 'list' || $op eq 'mail' || $op eq 'correct'}
 
 <p>

diff --git a/templates/core/password_prompt.tpl b/templates/core/password_prompt.tpl
index f43446d..3e55236 100644 (file)
--- a/templates/core/password_prompt.tpl
+++ b/templates/core/password_prompt.tpl
@@ -44,9 +44,7 @@ Si tu n'es pas {insert name="getName"}, change le login ci-dessous, ou rends-toi
 {/if}
 
 {if $smarty.request.response}<!-- failed login code //-->
-<div class="erreur">
-  Erreur d'identification. Essaie à nouveau !
-</div>
+{include file="include/triggers.tpl" text="Erreur d'identification. Essaie à nouveau !" type="errors"}
 <br />
 {/if}
 

diff --git a/templates/core/password_prompt_logged.tpl b/templates/core/password_prompt_logged.tpl
index fde48ea..c980627 100644 (file)
--- a/templates/core/password_prompt_logged.tpl
+++ b/templates/core/password_prompt_logged.tpl
@@ -68,9 +68,7 @@
 </form>
 <br />
 {if $smarty.request.response}<!-- failed login code -->
-<div class="erreur">
-  Erreur d'identification. Essaie à nouveau !
-</div>
+{include file="include/triggers.tpl" text="Erreur d'identification. Essaie à nouveau !" type="errors"}
 {/if}
 
 <!-- Set up the form with the challenge value and an empty reply value -->

diff --git a/templates/emails/redirect.tpl b/templates/emails/redirect.tpl
index 27f50ed..0b00d3a 100644 (file)
--- a/templates/emails/redirect.tpl
+++ b/templates/emails/redirect.tpl
@@ -20,23 +20,6 @@
 {*                                                                        *}
 {**************************************************************************}
 
-{if $retour == $smarty.const.ERROR_INACTIVE_REDIRECTION}
-  <p class="erreur">
-  Tu ne peux pas avoir aucune adresse de redirection active, sinon ton adresse
-  {$smarty.session.forlife}@{#globals.mail.domain#} ne fonctionnerait plus.
-  </p>
-{/if}
-{if $retour == $smarty.const.ERROR_INVALID_EMAIL}
-  <p class="erreur">
-  Erreur&nbsp;: l'email n'est pas valide.
-  </p>
-{/if}
-{if $retour == $smarty.const.ERROR_LOOP_EMAIL}
-  <p class="erreur">
-  Erreur&nbsp;: {$smarty.session.forlife}@{#globals.mail.domain#} ne doit pas être renvoyé
-  vers lui-même, ni vers son équivalent en {#globals.mail.domain2#} ni vers polytechnique.edu.
-  </p>
-{/if}
   <h1>
     Tes adresses de redirection
   </h1>

diff --git a/templates/events/index.tpl b/templates/events/index.tpl
index 9ecbe18..6ac15f5 100644 (file)
--- a/templates/events/index.tpl
+++ b/templates/events/index.tpl
@@ -42,47 +42,60 @@ Bienvenue {$smarty.session.prenom}{if $birthday}
 {/if}
 
 {if $smarty.session.no_redirect}
-<p class="erreur">
-  Tu n'as plus de redirection valide ce qui rend ton adresse Polytechnique.org
-  inutilisable. Rends-toi au plus vite sur <a href="emails/redirect">la page de 
-  gestion des emails</a> pour corriger ce problème.
-</p>
+<div class="errors">
+  <ul>
+    <li>
+      Tu n'as plus de redirection valide ce qui rend ton adresse Polytechnique.org
+      inutilisable. Rends-toi au plus vite sur <a href="emails/redirect">la page de 
+      gestion des emails</a> pour corriger ce problème.
+    </li>
+  </ul>
+</div>
 {/if}
 
 {if $smarty.session.mx_failures|@count}
-<fieldset>
-  <legend>{icon name=error} Des problèmes sont actuellement recontrés sur tes redirections suivantes</legend>
-  {foreach from=$smarty.session.mx_failures item=mail}
-  <div>
-    <span class="erreur">{$mail.mail}</span>
-    <div class="explication">{$mail.text}</div>
-  </div>
-  {/foreach}
-  <div><a href="emails/redirect">Gérer mes adresses de redirection</a></div>
-</fieldset>
+<div class="warnings">
+  {icon name=error} Des problèmes sont actuellement recontrés sur tes redirections suivantes :
+  <ul>
+    {foreach from=$smarty.session.mx_failures item=mail}
+    <li>
+      <span class="erreur">{$mail.mail}</span> :
+      <span class="explication">{$mail.text}</span>
+    </li>
+    {/foreach}
+  </ul>
+  <div style="text-align: center"><a href="emails/redirect">Gérer mes adresses de redirection</a></div>
+</div>
 {/if}
 
+{if $fiche_incitation || $photo_incitation || ($geoloc_incitation > 0)}
+<div class="warnings">
+  <ul>
 {if $fiche_incitation}
-  <p>La dernière mise à jour de ta
-  <a href="profile/{$smarty.session.forlife}" class="popup2">fiche</a>
-  date du {$fiche_incitation|date_format}.
-  Il est possible qu'elle ne soit pas à jour.
-  Si tu souhaites la modifier, <a href="profile/edit">clique ici !</a>
-  </p>
+  <li>
+    La dernière mise à jour de ta
+    <a href="profile/{$smarty.session.forlife}" class="popup2">fiche</a>
+    date du {$fiche_incitation|date_format}.
+    Il est possible qu'elle ne soit pas à jour.
+    Si tu souhaites la modifier, <a href="profile/edit">clique ici !</a>
+  </li>
 {/if}
 
 {if $photo_incitation}
-  <p>
+  <li>
     Tu n'as pas mis de photo de toi sur ta fiche, c'est dommage.
     Clique <a href="photo/change">ici</a> si tu souhaites en ajouter une.
-  </p>
+  </li>
 {/if}
 
 {if $geoloc_incitation > 0}
-  <p>
+  <li>
     Parmi tes adresses, il y en a {$geoloc_incitation} que nous n'avons pas pu localiser.
     Clique <a href="profile/edit/adresses">ici</a> pour rectifier.
-  </p>
+  </li>
+{/if}
+  </ul>
+</div>
 {/if}
 
 {include file="include/tips.tpl" full=true}

diff --git a/templates/lists/delete.tpl b/templates/lists/delete.tpl
index 30ca554..4300641 100644 (file)
--- a/templates/lists/delete.tpl
+++ b/templates/lists/delete.tpl
@@ -24,16 +24,9 @@
 {if $deleted}
 
 <p>[<a href='{$platal->ns}lists'>Voir toutes les listes</a>]</p>
-<p class="erreur">La liste a été détruite !</p>
 
 {else}
 
-{if !$details.own}
-<p class='erreur'>
-Tu n'es pas administrateur de la liste, mais du site.
-</p>
-{/if}
-
 {include file="lists/header_listes.tpl" on=delete}
 
 <h1>

diff --git a/templates/profile/nomusage.tpl b/templates/profile/nomusage.tpl
index d8fd991..e8bf0cb 100644 (file)
--- a/templates/profile/nomusage.tpl
+++ b/templates/profile/nomusage.tpl
@@ -23,15 +23,8 @@
 <h1>Nom d'usage</h1>
 
 {if $same}
-  <p class="erreur">
-      Si ton nom d'usage est identique à ton nom à l'X, il n'est pas
-      nécessaire de le saisir ici!
-  </p>
-  <p>
-    Pour supprimer ton nom d'usage, fais une demande avec un nom vide et en précisant la raison.
-  </p>
   <p>
-    [<a href='profile/usage'>Refaire une demande</a>] - [<a href='profile/edit'>Retour au profil</a>]
+    Tu peux soit <a href='profile/usage'>refaire une demande de nom d'usage</a>, soit <a href='profile/edit'>retourner à l'édition de ton profil</a>.
   </p>
 {else}
   {if $myusage}