From 44b071c2ad111a7e41f00f614c04633601a93344 Mon Sep 17 00:00:00 2001 From: =?utf8?q?St=C3=A9phane=20Jacob?= Date: Sat, 27 Sep 2008 18:59:42 +0200 Subject: [PATCH] Uses trig utilities whenever it is possible. --- modules/admin.php | 6 +-- modules/email.php | 23 ++++++++--- modules/lists.php | 4 ++ modules/profile.php | 2 + templates/admin/deces_promo.tpl | 4 -- templates/admin/homonymes.tpl | 6 --- templates/core/password_prompt.tpl | 4 +- templates/core/password_prompt_logged.tpl | 4 +- templates/emails/redirect.tpl | 17 --------- templates/events/index.tpl | 63 +++++++++++++++++++------------ templates/lists/delete.tpl | 7 ---- templates/profile/nomusage.tpl | 9 +---- 12 files changed, 68 insertions(+), 81 deletions(-) diff --git a/modules/admin.php b/modules/admin.php index 84af751..97f3d2c 100644 --- 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 --- 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 --- 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.
' . 'Contact les administrateurs du site pour régler le problème : ' . 'support@polytechnique.org'); } } 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 --- 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 --- a/templates/admin/deces_promo.tpl +++ b/templates/admin/deces_promo.tpl @@ -22,10 +22,6 @@ -{foreach from=$new_deces item=i} -

Ajout du décès de {$i.name} le {$i.date}.

-{/foreach} -
diff --git a/templates/admin/homonymes.tpl b/templates/admin/homonymes.tpl index f37f12c..33c88c4 100644 --- a/templates/admin/homonymes.tpl +++ b/templates/admin/homonymes.tpl @@ -22,12 +22,6 @@

Gestion des homonymes

-{if $op eq 'mail'} -

email envoyé à {$forlife}

-{elseif $op eq 'correct'} -

email envoyé à {$forlife}, alias supprimé

-{/if} - {if $op eq 'list' || $op eq 'mail' || $op eq 'correct'}

diff --git a/templates/core/password_prompt.tpl b/templates/core/password_prompt.tpl index f43446d..3e55236 100644 --- 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} -

- Erreur d'identification. Essaie à nouveau ! -
+{include file="include/triggers.tpl" text="Erreur d'identification. Essaie à nouveau !" type="errors"}
{/if} diff --git a/templates/core/password_prompt_logged.tpl b/templates/core/password_prompt_logged.tpl index fde48ea..c980627 100644 --- a/templates/core/password_prompt_logged.tpl +++ b/templates/core/password_prompt_logged.tpl @@ -68,9 +68,7 @@
{if $smarty.request.response} -
- Erreur d'identification. Essaie à nouveau ! -
+{include file="include/triggers.tpl" text="Erreur d'identification. Essaie à nouveau !" type="errors"} {/if} diff --git a/templates/emails/redirect.tpl b/templates/emails/redirect.tpl index 27f50ed..0b00d3a 100644 --- a/templates/emails/redirect.tpl +++ b/templates/emails/redirect.tpl @@ -20,23 +20,6 @@ {* *} {**************************************************************************} -{if $retour == $smarty.const.ERROR_INACTIVE_REDIRECTION} -

- Tu ne peux pas avoir aucune adresse de redirection active, sinon ton adresse - {$smarty.session.forlife}@{#globals.mail.domain#} ne fonctionnerait plus. -

-{/if} -{if $retour == $smarty.const.ERROR_INVALID_EMAIL} -

- Erreur : l'email n'est pas valide. -

-{/if} -{if $retour == $smarty.const.ERROR_LOOP_EMAIL} -

- Erreur : {$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. -

-{/if}

Tes adresses de redirection

diff --git a/templates/events/index.tpl b/templates/events/index.tpl index 9ecbe18..6ac15f5 100644 --- 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} -

- Tu n'as plus de redirection valide ce qui rend ton adresse Polytechnique.org - inutilisable. Rends-toi au plus vite sur la page de - gestion des emails pour corriger ce problème. -

+
+
    +
  • + Tu n'as plus de redirection valide ce qui rend ton adresse Polytechnique.org + inutilisable. Rends-toi au plus vite sur la page de + gestion des emails pour corriger ce problème. +
    • +
+
{/if} {if $smarty.session.mx_failures|@count} -
- {icon name=error} Des problèmes sont actuellement recontrés sur tes redirections suivantes - {foreach from=$smarty.session.mx_failures item=mail} -
- {$mail.mail} -
{$mail.text}
-
- {/foreach} -
Gérer mes adresses de redirection
-
+
+ {icon name=error} Des problèmes sont actuellement recontrés sur tes redirections suivantes : +
    + {foreach from=$smarty.session.mx_failures item=mail} +
  • + {$mail.mail} : + {$mail.text} +
    • + {/foreach} +
+
Gérer mes adresses de redirection
+
{/if} +{if $fiche_incitation || $photo_incitation || ($geoloc_incitation > 0)} +
+
    {if $fiche_incitation} -

    La dernière mise à jour de ta - fiche - date du {$fiche_incitation|date_format}. - Il est possible qu'elle ne soit pas à jour. - Si tu souhaites la modifier, clique ici ! -

    +
  • + La dernière mise à jour de ta + fiche + date du {$fiche_incitation|date_format}. + Il est possible qu'elle ne soit pas à jour. + Si tu souhaites la modifier, clique ici ! +
    • {/if} {if $photo_incitation} -

    +

  • Tu n'as pas mis de photo de toi sur ta fiche, c'est dommage. Clique ici si tu souhaites en ajouter une. -

    +
    • {/if} {if $geoloc_incitation > 0} -

    +

  • Parmi tes adresses, il y en a {$geoloc_incitation} que nous n'avons pas pu localiser. Clique ici pour rectifier. -

    +
    • +{/if} +
+
{/if} {include file="include/tips.tpl" full=true} diff --git a/templates/lists/delete.tpl b/templates/lists/delete.tpl index 30ca554..4300641 100644 --- a/templates/lists/delete.tpl +++ b/templates/lists/delete.tpl @@ -24,16 +24,9 @@ {if $deleted}

[Voir toutes les listes]

-

La liste a été détruite !

{else} -{if !$details.own} -

-Tu n'es pas administrateur de la liste, mais du site. -

-{/if} - {include file="lists/header_listes.tpl" on=delete}

diff --git a/templates/profile/nomusage.tpl b/templates/profile/nomusage.tpl index d8fd991..e8bf0cb 100644 --- a/templates/profile/nomusage.tpl +++ b/templates/profile/nomusage.tpl @@ -23,15 +23,8 @@

Nom d'usage

{if $same} -

- Si ton nom d'usage est identique à ton nom à l'X, il n'est pas - nécessaire de le saisir ici! -

-

- Pour supprimer ton nom d'usage, fais une demande avec un nom vide et en précisant la raison. -

- [Refaire une demande] - [Retour au profil] + Tu peux soit refaire une demande de nom d'usage, soit retourner à l'édition de ton profil.

{else} {if $myusage} -- 2.1.4