From a7d35093a40837b3a17d7fbac0259995f084812c Mon Sep 17 00:00:00 2001 From: Florent Bruneau Date: Sun, 8 Jun 2008 18:05:23 +0200 Subject: [PATCH] Switch to the new trigger pattern. Signed-off-by: Florent Bruneau --- classes/platalpage.php | 6 ++-- classes/pltableeditor.php | 16 +++++----- include/banana/moderate.inc.php | 2 +- include/user.func.inc.php | 4 +-- include/validations.inc.php | 30 ++++++++++++------ include/validations/evts.inc.php | 6 ++-- include/validations/paiements.inc.php | 4 +-- include/validations/photos.inc.php | 4 +-- include/wiki/engine.php | 4 +-- include/xorg/session.inc.php | 2 +- modules/admin.php | 40 ++++++++++++------------ modules/axletter.php | 12 ++++---- modules/carnet.php | 18 +++++------ modules/core.php | 2 +- modules/email.php | 28 ++++++++--------- modules/events.php | 12 ++++---- modules/geoloc.php | 6 ++-- modules/googleapps.php | 14 ++++----- modules/lists.php | 18 +++++------ modules/marketing.php | 16 +++++----- modules/newsletter.php | 4 +-- modules/payment.php | 6 ++-- modules/platal.php | 14 ++++----- modules/profile.php | 14 ++++----- modules/profile/general.inc.php | 2 +- modules/profile/mentor.inc.php | 4 +-- modules/profile/page.inc.php | 12 ++++---- modules/register.php | 2 +- modules/search/classes.inc.php | 4 +-- modules/xnet.php | 8 ++--- modules/xnetevents.php | 14 ++++----- modules/xnetevents/xnetevents.inc.php | 4 +-- modules/xnetgrp.php | 58 +++++++++++++++++------------------ modules/xnetlists.php | 24 +++++++-------- 34 files changed, 213 insertions(+), 201 deletions(-) diff --git a/classes/platalpage.php b/classes/platalpage.php index 59c3c20..f3bb2fa 100644 --- a/classes/platalpage.php +++ b/classes/platalpage.php @@ -53,7 +53,7 @@ abstract class PlatalPage extends Smarty $this->_failure = false; if ($globals->mode != 'rw') { - $this->trig("En raison d'une maintenance, une partie des fonctionnalités du site sont" + $this->trigError("En raison d'une maintenance, une partie des fonctionnalités du site sont" . " actuellement désactivée, en particuliers aucune donnée ne sera sauvegardée"); } $this->register_prefilter('at_to_globals'); @@ -194,7 +194,7 @@ abstract class PlatalPage extends Smarty // }}} // {{{ function trig() - public function trig($msg, $type = 'errors') + private function trig($msg, $type = 'errors') { if (!isset($this->_errors[$type])) { $this->_errors[$type] = array(); @@ -225,7 +225,7 @@ abstract class PlatalPage extends Smarty global $platal; $this->assign('platal', $platal); - $this->trig($msg); + $this->trigError($msg); $this->_failure = true; $this->run(); } diff --git a/classes/pltableeditor.php b/classes/pltableeditor.php index 826faa5..d42d6b8 100644 --- a/classes/pltableeditor.php +++ b/classes/pltableeditor.php @@ -199,16 +199,16 @@ class PLTableEditor foreach ($this->jtables as $table => $j) XDB::execute("DELETE FROM {$table} WHERE {$j['joinid']} = {?}{$j['joinextra']}", $id); XDB::execute("DELETE FROM {$this->table} WHERE {$this->idfield} = {?}",$id); - $page->trig("L'entrée ".$id." a été supprimée."); + $page->trigSuccess("L'entrée ".$id." a été supprimée."); } else if ($this->delete_action) { XDB::execute($this->delete_action, $id); if (isset($this->delete_message)) { - $page->trig($this->delete_message); + $page->trigSuccess($this->delete_message); } else { - $page->trig("L'entrée ".$id." a été supprimée."); + $page->trigSuccess("L'entrée ".$id." a été supprimée."); } } else { - $page->trig("Impossible de supprimer l'entrée."); + $page->trigError("Impossible de supprimer l'entrée."); } } if ($action == 'edit') { @@ -271,7 +271,7 @@ class PLTableEditor $val = "'".addslashes($val)."'"; } else { $cancel = true; - $page->trig("Il manque le champ ".$field); + $page->trigError("Il manque le champ ".$field); } $values .= $val; } @@ -280,13 +280,13 @@ class PLTableEditor XDB::execute("UPDATE {$this->table} SET {$this->idfield} = {?} WHERE {$this->idfield} = {?} AND {$this->whereclause}", Post::v($this->idfield), $id); XDB::execute("REPLACE INTO {$this->table} VALUES ($values)"); if ($id !== false) - $page->trig("L'entrée ".$id." a été mise à jour."); + $page->trigSuccess("L'entrée ".$id." a été mise à jour."); else { - $page->trig("Une nouvelle entrée a été créée."); + $page->trigSuccess("Une nouvelle entrée a été créée."); $id = XDB::insertId(); } } else - $page->trig("Impossible de mettre à jour."); + $page->trigError("Impossible de mettre à jour."); if (!$this->auto_return) { return $this->apply($page, 'edit', $id); } diff --git a/include/banana/moderate.inc.php b/include/banana/moderate.inc.php index 8096bb1..70cc2fa 100644 --- a/include/banana/moderate.inc.php +++ b/include/banana/moderate.inc.php @@ -91,7 +91,7 @@ class ModerationPage extends BananaPage { global $page; if ($page) { - $page->trig($msg); + $page->trigError($msg); } return true; } diff --git a/include/user.func.inc.php b/include/user.func.inc.php index 45666bb..b903654 100644 --- a/include/user.func.inc.php +++ b/include/user.func.inc.php @@ -88,7 +88,7 @@ function user_clear_all_subs($user_id, $really_del=true) function _default_user_callback($login) { global $page; - $page->trig("Il n'y a pas d'utilisateur avec l'identifiant : $login"); + $page->trigError("Il n'y a pas d'utilisateur avec l'identifiant : $login"); return; } @@ -171,7 +171,7 @@ function get_user_login($data, $get_forlife = false, $callback = '_default_user_ default: if (S::has_perms()) { $aliases = $res->fetchColumn(); - $page->trig("Il y a $i utilisateurs avec cette adresse mail : ".join(', ', $aliases)); + $page->trigError("Il y a $i utilisateurs avec cette adresse mail : ".join(', ', $aliases)); } else { $res->free(); } diff --git a/include/validations.inc.php b/include/validations.inc.php index 10bfc82..9814510 100644 --- a/include/validations.inc.php +++ b/include/validations.inc.php @@ -159,7 +159,7 @@ abstract class Validate { if (Env::has('delete')) { $this->clean(); - $this->trig('requete supprimée'); + $this->trigSuccess('Requête supprimée'); return true; } @@ -167,7 +167,7 @@ abstract class Validate if (Env::has('edit')) { if ($this->handle_editor()) { $this->update(); - $this->trig('requête mise à jour'); + $this->trigSuccess('Requête mise à jour'); return true; } return false; @@ -202,7 +202,7 @@ abstract class Validate $mailer->send(); $this->update(); - $this->trig('commentaire ajouté'); + $this->trigSuccess('Commentaire ajouté'); return true; } @@ -210,10 +210,10 @@ abstract class Validate if ($this->commit()) { $this->sendmail(true); $this->clean(); - $this->trig('mail envoyé'); + $this->trigSuccess('Mail de validation envoyé'); return true; } else { - $this->trig('erreur lors de la validation'); + $this->trigError('Erreur lors de la validation'); return false; } } @@ -222,10 +222,10 @@ abstract class Validate if (Env::v('comm')) { $this->sendmail(false); $this->clean(); - $this->trig('mail envoyé'); + $this->trigSuccess('Mail de refus envoyé'); return true; } else { - $this->trig('pas de motivation pour le refus !!!'); + $this->trigError('pas de motivation pour le refus !!!'); } } @@ -256,10 +256,22 @@ abstract class Validate // }}} // {{{ function trig() - protected function trig($msg) + protected function trigError($msg) { global $page; - $page->trig($msg); + $page->trigError($msg); + } + + protected function trigWarning($msg) + { + global $page; + $page->trigWarning($msg); + } + + protected function trigSuccess($msg) + { + global $page; + $page->trigSuccess($msg); } // }}} diff --git a/include/validations/evts.inc.php b/include/validations/evts.inc.php index a1a3985..c33ce5e 100644 --- a/include/validations/evts.inc.php +++ b/include/validations/evts.inc.php @@ -96,12 +96,12 @@ class EvtReq extends Validate if (@$_FILES['image']['tmp_name']) { $upload = PlUpload::get($_FILES['image'], S::v('forlife'), 'event'); if (!$upload) { - $this->trig("Impossible de télécharger le fichier"); + $this->trigError("Impossible de télécharger le fichier"); } elseif (!$upload->isType('image')) { - $page->trig('Le fichier n\'est pas une image valide au format JPEG, GIF ou PNG'); + $page->trigError('Le fichier n\'est pas une image valide au format JPEG, GIF ou PNG'); $upload->rm(); } elseif (!$upload->resizeImage(200, 300, 100, 100, 32284)) { - $page->trig('Impossible de retraiter l\'image'); + $page->trigError('Impossible de retraiter l\'image'); } else { $this->readImage($upload); } diff --git a/include/validations/paiements.inc.php b/include/validations/paiements.inc.php index 5d9347d..2343ac3 100644 --- a/include/validations/paiements.inc.php +++ b/include/validations/paiements.inc.php @@ -86,11 +86,11 @@ class PayReq extends Validate { // no text [AI JMIAJM IJA MIJ] if (preg_match('/\[[-\'"A-Z ]+\]/', $this->msg_reponse)) { - $this->trig("La demande de paiement n'est pas valide. Merci de compléter le texte avant de la soumettre"); + $this->trigError("La demande de paiement n'est pas valide. Merci de compléter le texte avant de la soumettre"); return false; } if (!preg_match('//', $this->msg_reponse)) { - $this->trig("Le demande de paiement ne contient pas la balise obligatoire <montant>"); + $this->trigError("Le demande de paiement ne contient pas la balise obligatoire <montant>"); return false; } return true; diff --git a/include/validations/photos.inc.php b/include/validations/photos.inc.php index 26f45f8..b2fc0a1 100644 --- a/include/validations/photos.inc.php +++ b/include/validations/photos.inc.php @@ -56,7 +56,7 @@ class PhotoReq extends Validate { $this->valid = $upload->resizeImage(240, 300, 160, 0, SIZE_MAX); if (!$this->valid) { - $this->trig('Le fichier que tu as transmis n\'est pas une image valide, ou est trop gros pour être traité'); + $this->trigError('Le fichier que tu as transmis n\'est pas une image valide, ou est trop gros pour être traité'); } $this->data = $upload->getContents(); list($this->x, $this->y, $this->mimetype) = $upload->imageInfo(); @@ -103,7 +103,7 @@ class PhotoReq extends Validate if (isset($_FILES['userfile'])) { $upload =& PlUpload::get($_FILES['userfile'], S::v('forlife'), 'photo'); if (!$upload) { - $this->trig('Une erreur est survenue lors du téléchargement du fichier'); + $this->trigError('Une erreur est survenue lors du téléchargement du fichier'); return false; } $this->read($upload); diff --git a/include/wiki/engine.php b/include/wiki/engine.php index e7a45a9..f8c01fc 100644 --- a/include/wiki/engine.php +++ b/include/wiki/engine.php @@ -51,7 +51,7 @@ if ($p = Post::v('setrperms')) { wiki_apply_perms('admin'); if (wiki_set_perms($n, $p, $perms[1])) { $perms = wiki_get_perms($n); - $page->trig('Permissions mises à jour'); + $page->trigSuccess('Permissions mises à jour'); } } @@ -59,7 +59,7 @@ if ($p = Post::v('setwperms')) { wiki_apply_perms('admin'); if (wiki_set_perms($n, $perms[0], $p)) { $perms = wiki_get_perms($n); - $page->trig('Permissions mises à jour'); + $page->trigSuccess('Permissions mises à jour'); } } diff --git a/include/xorg/session.inc.php b/include/xorg/session.inc.php index 62f5d30..34581a1 100644 --- a/include/xorg/session.inc.php +++ b/include/xorg/session.inc.php @@ -294,7 +294,7 @@ function start_connexion ($uid, $identified) new_skinned_page('platal/index.tpl'); $newpage = true; } - $page->trig("Une erreur est survenue lors de la procédure d'authentification. " + $page->trigError("Une erreur est survenue lors de la procédure d'authentification. " ."Merci de contacter au plus vite " ."support@polytechnique.org"); if ($newpage) { diff --git a/modules/admin.php b/modules/admin.php index 5afa055..bfb4b27 100644 --- a/modules/admin.php +++ b/modules/admin.php @@ -70,11 +70,11 @@ class AdminModule extends PLModule if (Env::has('del')) { $crc = Env::v('crc'); XDB::execute("UPDATE postfix_mailseen SET release = 'del' WHERE crc = {?}", $crc); - $page->trig($crc." verra tous ses mails supprimés !"); + $page->trigSuccess($crc." verra tous ses mails supprimés !"); } elseif (Env::has('ok')) { $crc = Env::v('crc'); XDB::execute("UPDATE postfix_mailseen SET release = 'ok' WHERE crc = {?}", $crc); - $page->trig($crc." a le droit de passer !"); + $page->trigSuccess($crc." a le droit de passer !"); } $sql = XDB::iterator( @@ -419,10 +419,10 @@ class AdminModule extends PLModule case "add_fwd": $email = trim(Env::v('email')); if (!isvalid_email_redirection($email)) { - $page->trig("invalid email $email"); + $page->trigError("Email non valide: $email"); } else { $redirect->add_email($email); - $page->trig("Ajout de $email effectué"); + $page->trigSuccess("Ajout de $email effectué"); } break; @@ -442,7 +442,7 @@ class AdminModule extends PLModule WHERE uid = {?} AND rewrite LIKE CONCAT({?}, '@%')", $mr['user_id'], $val); fix_bestalias($mr['user_id']); - $page->trig($val." a été supprimé"); + $page->trigSuccess($val." a été supprimé"); } break; case "activate_fwd": @@ -475,24 +475,24 @@ class AdminModule extends PLModule $domain = $globals->mail->domain; } if (!preg_match('/[-a-z0-9\.]+/s', $alias)) { - $page->trig("'$alias' n'est pas un alias valide"); + $page->trigError("'$alias' n'est pas un alias valide"); } if ($domain == $globals->mail->alias_dom || $domain == $globals->mail->alias_dom2) { $req = new AliasReq($mr['user_id'], $alias, 'Admin request', false); if ($req->commit()) { - $page->trig("Nouvel alias '$alias@$domain' attribué"); + $page->trigSuccess("Nouvel alias '$alias@$domain' attribué"); } else { - $page->trig("Impossible d'ajouter l'alias '$alias@$domain', il est probablement déjà attribué"); + $page->trigError("Impossible d'ajouter l'alias '$alias@$domain', il est probablement déjà attribué"); } } elseif ($domain == $globals->mail->domain || $domain == $globals->mail->domain2) { if (XDB::execute("INSERT INTO aliases (id,alias,type) VALUES ({?}, {?}, 'alias')", $mr['user_id'], $alias)) { - $page->trig("Nouvel alias '$alias' ajouté"); + $page->trigSuccess("Nouvel alias '$alias' ajouté"); } else { - $page->trig("Impossible d'ajouter l'alias '$alias', il est probablement déjà attribué"); + $page->trigError("Impossible d'ajouter l'alias '$alias', il est probablement déjà attribué"); } } else { - $page->trig("Le domaine '$domain' n'est pas valide"); + $page->trigError("Le domaine '$domain' n'est pas valide"); } break; @@ -532,7 +532,7 @@ class AdminModule extends PLModule } if ($watch && !$comm) { - $page->trig("Il est nécessaire de mettre un commentaire pour surveiller un compte"); + $page->trigError("Il est nécessaire de mettre un commentaire pour surveiller un compte"); break; } @@ -575,7 +575,7 @@ class AdminModule extends PLModule // update number of subscribers (perms or deceased may have changed) update_NbIns(); - $page->trig("updaté correctement."); + $page->trigSuccess("updaté correctement."); } if (Env::v('nomusageN') != $mr['nom_usage']) { require_once "xorg.misc.inc.php"; @@ -618,7 +618,7 @@ class AdminModule extends PLModule user_clear_all_subs($mr['user_id']); // update number of subscribers (perms or deceased may have changed) update_NbIns(); - $page->trig("'{$mr['user_id']}' a été désinscrit !"); + $page->trigSuccess("'{$mr['user_id']}' a été désinscrit !"); $mailer = new PlMailer("admin/useredit.mail.tpl"); $mailer->assign("admin", S::v('forlife')); $mailer->assign("user", $mr['forlife']); @@ -698,7 +698,7 @@ class AdminModule extends PLModule $action = Env::v('valid_promo') == 'Ajouter des membres' ? 'add' : 'ax'; pl_redirect('admin/promo/' . $action . '/' . Env::i('promo')); } else { - $page->trig('Promo non valide'); + $page->trigError('Promo non valide'); } } @@ -1063,9 +1063,9 @@ class AdminModule extends PLModule S::assert_xsrf_token(); if (wiki_delete_page($wikipage)) { - $page->trig("La page ".$wikipage." a été supprimée."); + $page->trigSuccess("La page ".$wikipage." a été supprimée."); } else { - $page->trig("Impossible de supprimer la page ".$wikipage."."); + $page->trigError("Impossible de supprimer la page ".$wikipage."."); } } @@ -1077,9 +1077,9 @@ class AdminModule extends PLModule if (is_numeric($changedLinks)) { $s .= $changedLinks.' lien'.(($changedLinks>1)?'s ont été modifiés.':' a été modifié.'); } - $page->trig($s); + $page->trigSuccess($s); } else { - $page->trig("Impossible de déplacer la page ".$wikipage); + $page->trigError("Impossible de déplacer la page ".$wikipage); } } @@ -1229,7 +1229,7 @@ class AdminModule extends PLModule $page->changeTpl('admin/icons.tpl'); $dh = opendir('../htdocs/images/icons'); if (!$dh) { - $page->trig('Dossier des icones introuvables.'); + $page->trigError('Dossier des icones introuvables.'); } $icons = array(); while (($file = readdir($dh)) !== false) { diff --git a/modules/axletter.php b/modules/axletter.php index 68577c5..270e2ca 100644 --- a/modules/axletter.php +++ b/modules/axletter.php @@ -119,27 +119,27 @@ class AXLetterModule extends PLModule $title = $subject; } if (!$subject || !$title || !$body) { - $page->trig("L'article doit avoir un sujet et un contenu"); + $page->trigError("L'article doit avoir un sujet et un contenu"); Post::kill('valid'); } if (($promo_min > $promo_max && $promo_max != 0)|| ($promo_min != 0 && ($promo_min <= 1900 || $promo_min >= 2020)) || ($promo_max != 0 && ($promo_max <= 1900 || $promo_max >= 2020))) { - $page->trig("L'intervalle de promotions n'est pas valide"); + $page->trigError("L'intervalle de promotions n'est pas valide"); Post::kill('valid'); } if (empty($short_name)) { - $page->trig("L'annonce doit avoir un nom raccourci pour simplifier la navigation dans les archives"); + $page->trigError("L'annonce doit avoir un nom raccourci pour simplifier la navigation dans les archives"); Post::kill('valid'); } elseif (!preg_match('/^[a-z][-a-z0-9]*[a-z0-9]$/', $short_name)) { - $page->trig("Le nom raccourci n'est pas valide, il doit comporter au moins 2 caractères et n'être composé " + $page->trigError("Le nom raccourci n'est pas valide, il doit comporter au moins 2 caractères et n'être composé " . "que de chiffres, lettres et tirets"); Post::kill('valid'); } elseif ($short_name != Post::v('old_short_name')) { $res = XDB::query("SELECT id FROM axletter WHERE short_name = {?}", $short_name); if ($res->numRows() && $res->fetchOneCell() != $id) { - $page->trig("Le nom $short_name est déjà utilisé, merci d'en choisir un autre"); + $page->trigError("Le nom $short_name est déjà utilisé, merci d'en choisir un autre"); $short_name = Post::v('old_short_name'); if (empty($short_name)) { Post::kill('valid'); @@ -302,7 +302,7 @@ class AXLetterModule extends PLModule break; } if (!$res) { - $page->trig("Personne ne correspond à l'identifiant '$uid'"); + $page->trigError("Personne ne correspond à l'identifiant '$uid'"); } } } diff --git a/modules/carnet.php b/modules/carnet.php index cf1cd16..febcf3b 100644 --- a/modules/carnet.php +++ b/modules/carnet.php @@ -85,7 +85,7 @@ class CarnetModule extends PLModule if(preg_match('!^ *(\d{4}) *$!', $arg, $matches)) { $p = intval($matches[1]); if($p<1900 || $p>2100) { - $page->trig("la promo entrée est invalide"); + $page->trigError("la promo entrée est invalide"); } else { if ($action == 'add_promo') { $watch->_promos->add($p); @@ -97,9 +97,9 @@ class CarnetModule extends PLModule $p1 = intval($matches[1]); $p2 = intval($matches[2]); if($p1<1900 || $p1>2100) { - $page->trig('la première promo de la plage entrée est invalide'); + $page->trigError('la première promo de la plage entrée est invalide'); } elseif($p2<1900 || $p2>2100) { - $page->trig('la seconde promo de la plage entrée est invalide'); + $page->trigError('la seconde promo de la plage entrée est invalide'); } else { if ($action == 'add_promo') { $watch->_promos->addRange($p1, $p2); @@ -108,7 +108,7 @@ class CarnetModule extends PLModule } } } else { - $page->trig("La promo (ou la plage de promo) entrée est dans un format incorrect."); + $page->trigError("La promo (ou la plage de promo) entrée est dans un format incorrect."); } } @@ -196,7 +196,7 @@ class CarnetModule extends PLModule function searchErrorHandler($explain) { global $page; - $page->trig($explain); + $page->trigError($explain); $this->handler_contacts($page); } @@ -220,7 +220,7 @@ class CarnetModule extends PLModule WHERE uid = {?} AND contact = {?}', $uid, $user)) { - $page->trig("Contact retiré !"); + $page->trigSuccess("Contact retiré !"); } } else { if (XDB::execute( @@ -229,7 +229,7 @@ class CarnetModule extends PLModule INNER JOIN aliases AS a ON (c.contact=a.id and a.type!="homonyme") WHERE c.uid = {?} AND a.alias={?}', $uid, $user)) { - $page->trig("Contact retiré !"); + $page->trigSuccess("Contact retiré !"); } } break; @@ -243,9 +243,9 @@ class CarnetModule extends PLModule FROM aliases WHERE alias = {?}', $uid, $login)) { - $page->trig('Contact ajouté !'); + $page->trigSuccess('Contact ajouté !'); } else { - $page->trig('Contact déjà dans la liste !'); + $page->trigWarning('Contact déjà dans la liste !'); } } } diff --git a/modules/core.php b/modules/core.php index f044d9b..a490d17 100644 --- a/modules/core.php +++ b/modules/core.php @@ -145,7 +145,7 @@ class CoreModule extends PLModule $mymail->setTxtBody($body); $mymail->send(); } elseif (Env::has('send')) { - $page->trig("Merci de remplir une explication du problème rencontré"); + $page->trigError("Merci de remplir une explication du problème rencontré"); } } diff --git a/modules/email.php b/modules/email.php index a2ada02..f3fc4f4 100644 --- a/modules/email.php +++ b/modules/email.php @@ -145,17 +145,17 @@ class EmailModule extends PLModule //Quelques vérifications sur l'alias (caractères spéciaux) if (!preg_match( "/^[a-zA-Z0-9\-.]{3,20}$/", $alias)) { - $page->trig("L'adresse demandée n'est pas valide. - Vérifie qu'elle comporte entre 3 et 20 caractères - et qu'elle ne contient que des lettres non accentuées, - des chiffres ou les caractères - et ."); + $page->trigError("L'adresse demandée n'est pas valide." + . " Vérifie qu'elle comporte entre 3 et 20 caractères" + . " et qu'elle ne contient que des lettres non accentuées," + . " des chiffres ou les caractères - et ."); return; } else { //vérifier que l'alias n'est pas déja pris $res = XDB::query('SELECT COUNT(*) FROM virtual WHERE alias={?}', $alias.'@'.$globals->mail->alias_dom); if ($res->fetchOneCell() > 0) { - $page->trig("L'alias $alias@{$globals->mail->alias_dom} a déja été attribué. + $page->trigError("L'alias $alias@{$globals->mail->alias_dom} a déja été attribué. Tu ne peux donc pas l'obtenir."); return; } @@ -164,7 +164,7 @@ class EmailModule extends PLModule $it = new ValidateIterator (); while($req = $it->next()) { if ($req->type == "alias" and $req->alias == $alias . '@' . $globals->mail->alias_dom) { - $page->trig("L'alias $alias@{$globals->mail->alias_dom} a déja été demandé. + $page->trigError("L'alias $alias@{$globals->mail->alias_dom} a déja été demandé. Tu ne peux donc pas l'obtenir pour l'instant."); return ; } @@ -300,13 +300,13 @@ class EmailModule extends PLModule $upload = PlUpload::get($_FILES['mail'], S::v('forlife'), 'spam.submit', true); if (!$upload) { - $page->trig('Une erreur a été rencontrée lors du transfert du fichier'); + $page->trigError('Une erreur a été rencontrée lors du transfert du fichier'); return; } $mime = $upload->contentType(); if ($mime != 'text/x-mail' && $mime != 'message/rfc822') { $upload->clear(); - $page->trig('Le fichier ne contient pas un mail complet'); + $page->trigError('Le fichier ne contient pas un mail complet'); return; } global $globals; @@ -317,7 +317,7 @@ class EmailModule extends PLModule $mailer->setTxtBody(Post::v('type') . ' soumis par ' . S::v('forlife') . ' via le web'); $mailer->addUploadAttachment($upload, Post::v('type') . '.mail'); $mailer->send(); - $page->trig('Le message a été transmis à ' . $box); + $page->trigSuccess('Le message a été transmis à ' . $box); $upload->clear(); } } @@ -364,7 +364,7 @@ class EmailModule extends PLModule $error = false; foreach ($_FILES as &$file) { if ($file['name'] && !PlUpload::get($file, S::v('forlife'), 'emails.send', false)) { - $page->trig(PlUpload::$lastError); + $page->trigError(PlUpload::$lastError); $error = true; break; } @@ -384,7 +384,7 @@ class EmailModule extends PLModule $bcc = trim(Env::v('bcc')); if (empty($to) && empty($cc) && empty($to2) && empty($bcc) && empty($cc2)) { - $page->trig("Indique au moins un destinataire."); + $page->trigError("Indique au moins un destinataire."); $page->assign('uploaded_f', PlUpload::listFilenames(S::v('forlife'), 'emails.send')); } else { $mymail = new PlMailer(); @@ -405,11 +405,11 @@ class EmailModule extends PLModule $mymail->setWikiBody($txt); } if ($mymail->send()) { - $page->trig("Ton mail a bien été envoyé."); + $page->trigSuccess("Ton mail a bien été envoyé."); $_REQUEST = array('bcc' => S::v('bestalias').'@'.$globals->mail->domain); PlUpload::clear(S::v('forlife'), 'emails.send'); } else { - $page->trig("Erreur lors de l'envoi du courriel, réessaye."); + $page->trigError("Erreur lors de l'envoi du courriel, réessaye."); $page->assign('uploaded_f', PlUpload::listFilenames(S::v('forlife'), 'emails.send')); } } @@ -545,7 +545,7 @@ L'équipe d'administration mail->domain . '>'; $mail->setSubject("Une de tes adresse de redirection Polytechnique.org ne marche plus !!"); $mail->setTxtBody($message); $mail->send(); - $page->trig("Mail envoyé ! :o)"); + $page->trigSuccess("Mail envoyé !"); } } elseif (Post::has('email')) { S::assert_xsrf_token(); diff --git a/modules/events.php b/modules/events.php index b4a3681..b4400e0 100644 --- a/modules/events.php +++ b/modules/events.php @@ -87,14 +87,14 @@ class EventsModule extends PLModule return true; } if (!$upload->upload($_FILES['image']) && !$upload->download(Env::v('image_url'))) { - $page->trig('Impossible de télécharger l\'image'); + $page->trigError('Impossible de télécharger l\'image'); return false; } elseif (!$upload->isType('image')) { - $page->trig('Le fichier n\'est pas une image valide au format JPEG, GIF ou PNG.'); + $page->trigError('Le fichier n\'est pas une image valide au format JPEG, GIF ou PNG.'); $upload->rm(); return false; } elseif (!$upload->resizeImage(200, 300, 100, 100, 32284)) { - $page->trig('Impossible de retraiter l\'image'); + $page->trigError('Impossible de retraiter l\'image'); return false; } return true; @@ -289,7 +289,7 @@ class EventsModule extends PLModule ($promo_min != 0 && ($promo_min <= 1900 || $promo_min >= 2020)) || ($promo_max != 0 && ($promo_max <= 1900 || $promo_max >= 2020))) { - $page->trig("L'intervalle de promotions n'est pas valide"); + $page->trigError("L'intervalle de promotions n'est pas valide"); $action = null; } @@ -306,7 +306,7 @@ class EventsModule extends PLModule $upload->rm(); $page->assign('action', false); } elseif ($action && (!trim($texte) || !trim($titre))) { - $page->trig("L'article doit avoir un titre et un contenu"); + $page->trigError("L'article doit avoir un titre et un contenu"); } elseif ($action) { require_once 'validations.inc.php'; $evtreq = new EvtReq($titre, $texte, $promo_min, $promo_max, @@ -373,7 +373,7 @@ class EventsModule extends PLModule if (($promo_min != 0 && ($promo_min <= 1900 || $promo_min >= 2020)) || ($promo_max != 0 && ($promo_max <= 1900 || $promo_max >= 2020 || $promo_max < $promo_min))) { - $page->trig("L'intervalle de promotions $promo_min -> $promo_max n'est pas valide"); + $page->trigError("L'intervalle de promotions $promo_min -> $promo_max n'est pas valide"); $action = 'edit'; } else { $res = XDB::query('SELECT flags FROM evenements WHERE id = {?}', $eid); diff --git a/modules/geoloc.php b/modules/geoloc.php index 86bfbc9..895c5f4 100644 --- a/modules/geoloc.php +++ b/modules/geoloc.php @@ -56,7 +56,7 @@ class GeolocModule extends PLModule } if ($nb_synchro) - $page->trig(($nb_synchro > 1)?($nb_synchro." villes ont été synchronisées"):"Une ville a été synchronisée"); + $page->trigSuccess(($nb_synchro > 1)?($nb_synchro." villes ont été synchronisées"):"Une ville a été synchronisée"); $res = XDB::query("SELECT COUNT(*) FROM geoloc_city WHERE lat = 0 AND lon = 0"); $page->assign("nb_missinglat", $res->fetchOneCell()); @@ -68,7 +68,7 @@ class GeolocModule extends PLModule if ($action == 'cities_not_on_map') { require_once('geoloc.inc.php'); if (!fix_cities_not_on_map(20)) - $page->trig("Impossible d'accéder au webservice"); + $page->trigError("Impossible d'accéder au webservice"); else $refresh = true; } @@ -87,7 +87,7 @@ class GeolocModule extends PLModule if ($action == 'newmaps') { require_once('geoloc.inc.php'); if (!get_new_maps(Env::v('url'))) - $page->trig("Impossible d'accéder aux nouvelles cartes"); + $page->trigError("Impossible d'accéder aux nouvelles cartes"); } $countMissing = XDB::query("SELECT COUNT(*) FROM geoloc_city AS c LEFT JOIN geoloc_city_in_maps AS m ON(c.id = m.city_id) WHERE m.city_id IS NULL"); diff --git a/modules/googleapps.php b/modules/googleapps.php index d22f9b2..fcfb153 100644 --- a/modules/googleapps.php +++ b/modules/googleapps.php @@ -77,18 +77,18 @@ class GoogleAppsModule extends PLModule S::assert_xsrf_token(); if ($account->pending_update_suspension) { - $page->trig("Ton compte est déjà en cours de désactivation."); + $page->trigWarning("Ton compte est déjà en cours de désactivation."); } else { if ($redirect->modify_one_email('googleapps', false) == SUCCESS) { $account->suspend(); - $page->trig("Ton compte Google Apps est dorénavant désactivé."); + $page->trigSuccess("Ton compte Google Apps est dorénavant désactivé."); } else { - $page->trig("Ton compte Google Apps est ta seule adresse de redirection. Ton compte ne peux pas être désactivé."); + $page->trigError("Ton compte Google Apps est ta seule adresse de redirection. Ton compte ne peux pas être désactivé."); } } } elseif ($action == 'unsuspend' && Post::has('unsuspend') && $account->suspended()) { $account->unsuspend(Post::b('redirect_mails', true)); - $page->trig("Ta demande de réactivation a bien été prise en compte."); + $page->trigSuccess("Ta demande de réactivation a bien été prise en compte."); } if ($action == 'create') { @@ -107,7 +107,7 @@ class GoogleAppsModule extends PLModule } $account->create($password_sync, $password, $redirect_mails); - $page->trig("La demande de création de ton compte Google Apps a bien été enregistrée."); + $page->trigSuccess("La demande de création de ton compte Google Apps a bien été enregistrée."); } } @@ -126,7 +126,7 @@ class GoogleAppsModule extends PLModule XDB::execute( "DELETE FROM gapps_queue WHERE q_id = {?} AND p_status = 'hardfail'", $qid); - $page->trig("La requête échouée a bien été retirée."); + $page->trigSuccess("La requête échouée a bien été retirée."); } } @@ -202,7 +202,7 @@ class GoogleAppsModule extends PLModule if ($action == 'forcesync' && $account->sync_password) { $res = XDB::query("SELECT password FROM auth_user_md5 WHERE user_id = {?}", $user); $account->set_password($res->fetchOneCell()); - $page->trig('Le mot de passe a été synchronisé.'); + $page->trigSuccess('Le mot de passe a été synchronisé.'); } // Displays basic account information. diff --git a/modules/lists.php b/modules/lists.php index 544feed..de95560 100644 --- a/modules/lists.php +++ b/modules/lists.php @@ -110,7 +110,7 @@ class ListsModule extends PLModule if ($promo >= 1900 and $promo < 2100) { $this->client->subscribe("promo$promo"); } else { - $page->trig("promo incorrecte, il faut une promo sur 4 chiffres."); + $page->trigSuccess("promo incorrecte, il faut une promo sur 4 chiffres."); } } $listes = $this->client->get_lists(); @@ -205,29 +205,29 @@ class ListsModule extends PLModule $liste = Post::v('liste'); if (empty($liste)) { - $page->trig('champs «adresse souhaitée» vide'); + $page->trigError('champs «adresse souhaitée» vide'); } if (!preg_match("/^[a-zA-Z0-9\-]*$/", $liste)) { - $page->trig('le nom de la liste ne doit contenir que des lettres non accentuées, chiffres et tirets'); + $page->trigError('le nom de la liste ne doit contenir que des lettres non accentuées, chiffres et tirets'); } $res = XDB::query("SELECT COUNT(*) FROM aliases WHERE alias={?}", $liste); $n = $res->fetchOneCell(); if ($n) { - $page->trig('cet alias est déjà pris'); + $page->trigError('cet alias est déjà pris'); } if (!Post::v('desc')) { - $page->trig('le sujet est vide'); + $page->trigError('le sujet est vide'); } if (!count($owners)) { - $page->trig('pas de gestionnaire'); + $page->trigError('pas de gestionnaire'); } if (count($members)<4) { - $page->trig('pas assez de membres'); + $page->trigError('pas assez de membres'); } if (!$page->nb_errs()) { @@ -562,7 +562,7 @@ class ListsModule extends PLModule $arr = $this->client->mass_subscribe($liste, $members); if (is_array($arr)) { foreach($arr as $addr) { - $page->trig("{$addr[0]} inscrit."); + $page->trigSuccess("{$addr[0]} inscrit."); } } } @@ -583,7 +583,7 @@ class ListsModule extends PLModule if ($owners) { foreach ($owners as $login) { if ($this->client->add_owner($liste, $login)) { - $page->trig($alias." ajouté aux modérateurs."); + $page->trigSuccess($alias." ajouté aux modérateurs."); } } } diff --git a/modules/marketing.php b/modules/marketing.php index a272c31..5c52a44 100644 --- a/modules/marketing.php +++ b/modules/marketing.php @@ -111,7 +111,7 @@ class MarketingModule extends PLModule if ($action == 'rel') { $market = Marketing::get($uid, $value); if ($market == null) { - $page->trig("Aucun marketing n'a été effectué vers $value"); + $page->trigWarning("Aucun marketing n'a été effectué vers $value"); } else { $to = $market->user['to']; $title = $market->getTitle(); @@ -133,12 +133,12 @@ class MarketingModule extends PLModule $market = new Marketing($uid, Post::v('to'), 'default', null, 'staff'); } $market->send(Post::v('title'), Post::v('message')); - $page->trig("Mail envoyé"); + $page->trigSuccess("Mail envoyé"); } if ($action == 'insrel') { if (Marketing::relance($uid)) { - $page->trig('relance faite'); + $page->trigSuccess('relance faite'); } } @@ -207,11 +207,11 @@ class MarketingModule extends PLModule WHERE e.email = {?} AND a.alias = {?}", $email, $user['forlife']); $state = $res->numRows() ? $res->fetchOneCell() : null; if ($state == 'panne') { - $page->trig("L'adresse que tu as fournie est l'adresse actuelle de {$user['prenom']} et est en panne."); + $page->trigWarning("L'adresse que tu as fournie est l'adresse actuelle de {$user['prenom']} et est en panne."); } elseif ($state == 'active') { - $page->trig("L'adresse que tu as fournie est l'adresse actuelle de {$user['prenom']}"); + $page->trigWarning("L'adresse que tu as fournie est l'adresse actuelle de {$user['prenom']}"); } elseif ($user['email'] && !trim(Post::v('comment'))) { - $page->trig("Il faut que tu ajoutes un commentaire à ta proposition pour justifier le " + $page->trigError("Il faut que tu ajoutes un commentaire à ta proposition pour justifier le " ."besoin de changer la redirection de " . $user['prenom']); } else { require_once 'validations.inc.php'; @@ -220,7 +220,7 @@ class MarketingModule extends PLModule $page->assign('sent', true); } } elseif ($email) { - $page->trig("L'adresse proposée n'est pas une adresse acceptable pour une redirection"); + $page->trigError("L'adresse proposée n'est pas une adresse acceptable pour une redirection"); } } @@ -265,7 +265,7 @@ class MarketingModule extends PLModule $email = trim(Post::v('mail')); if (!isvalid_email_redirection($email)) { - $page->trig("Email invalide !"); + $page->trigError("Email invalide !"); } else { // On cherche les marketings précédents sur cette adresse // email, en se restreignant au dernier mois diff --git a/modules/newsletter.php b/modules/newsletter.php index 37cb46a..5a41e9a 100644 --- a/modules/newsletter.php +++ b/modules/newsletter.php @@ -81,7 +81,7 @@ class NewsletterModule extends PLModule if (Post::has('see') || (Post::has('valid') && (!trim(Post::v('title')) || !trim(Post::v('body'))))) { if (!Post::has('see')) { - $page->trig("L'article doit avoir un titre et un contenu"); + $page->trigError("L'article doit avoir un titre et un contenu"); } $art = new NLArticle(Post::v('title'), Post::v('body'), Post::v('append')); $page->assign('art', $art); @@ -130,7 +130,7 @@ class NewsletterModule extends PLModule if (preg_match('/^[-a-z0-9]*$/i', $nl->_shortname) && !is_numeric($nl->_shortname)) { $nl->save(); } else { - $page->trig('Le nom de la NL n\'est pas valide'); + $page->trigError('Le nom de la NL n\'est pas valide'); pl_redirect('admin/newsletter/edit/' . $nl->_id); } } diff --git a/modules/payment.php b/modules/payment.php index 8bc8285..9de57d1 100644 --- a/modules/payment.php +++ b/modules/payment.php @@ -46,7 +46,7 @@ function paypal_erreur($text, $send=true) $mymail->setTxtBody("\n\n".var_export($_REQUEST,true)); $mymail->send(); - $page->trig($text); + $page->trigError($text); } /* http://fr.wikipedia.org/wiki/Formule_de_Luhn */ @@ -128,13 +128,13 @@ class PaymentModule extends PLModule $pay = new Payment($ref); if($pay->flags->hasflag('old')){ - $page->trig("La transaction selectionnée est périmée."); + $page->trigError("La transaction selectionnée est périmée."); $pay = new Payment(); } $val = Env::v('montant') != 0 ? Env::v('montant') : $pay->montant_def; if (($e = $pay->check($val)) !== true) { - $page->trig($e); + $page->trigError($e); } if ($op=='submit') { diff --git a/modules/platal.php b/modules/platal.php index f2d4d96..2e6570c 100644 --- a/modules/platal.php +++ b/modules/platal.php @@ -153,7 +153,7 @@ class PlatalModule extends PLModule SET redirecturl = {?} WHERE user_id = {?}', $url, S::v('uid')); $log->log('carva_add', 'http://'.Env::v('url')); - $page->trig("Redirection activée vers $url"); + $page->trigSuccess("Redirection activée vers $url"); } elseif (Env::v('submit') == "Supprimer") { XDB::execute("UPDATE auth_user_quick SET redirecturl = '' @@ -161,7 +161,7 @@ class PlatalModule extends PLModule S::v('uid')); $log->log("carva_del", $url); Post::kill('url'); - $page->trig('Redirection supprimée'); + $page->trigSuccess('Redirection supprimée'); } $res = XDB::query('SELECT redirecturl @@ -179,7 +179,7 @@ class PlatalModule extends PLModule if (Env::v('act_rss') == 'Activer') { $this->__set_rss_state(true); - $page->trig("Ton Fil RSS est activé."); + $page->trigSuccess("Ton Fil RSS est activé."); } } @@ -242,12 +242,12 @@ class PlatalModule extends PLModule { XDB::execute('UPDATE auth_user_md5 SET smtppass = {?} WHERE user_id = {?}', $pass, $uid); - $page->trig('Mot de passe enregistré'); + $page->trigSuccess('Mot de passe enregistré'); $log->log("passwd_ssl"); } elseif (Env::v('op') == "Supprimer") { XDB::execute('UPDATE auth_user_md5 SET smtppass = "" WHERE user_id = {?}', $uid); - $page->trig('Compte SMTP et NNTP supprimé'); + $page->trigSuccess('Compte SMTP et NNTP supprimé'); $log->log("passwd_del"); } @@ -268,7 +268,7 @@ class PlatalModule extends PLModule } if (!ereg('[0-3][0-9][0-1][0-9][1][9]([0-9]{2})', Env::v('birth'))) { - $page->trig('Date de naissance incorrecte ou incohérente'); + $page->trigError('Date de naissance incorrecte ou incohérente'); return; } @@ -337,7 +337,7 @@ Adresse de secours : " . Post::v('email') : "")); $logger = $_SESSION['log'] = new CoreLogger($uid); $logger->log('recovery', $mails); } else { - $page->trig('Les informations que tu as rentrées ne permettent pas de récupérer ton mot de passe.
'. + $page->trigError('Les informations que tu as rentrées ne permettent pas de récupérer ton mot de passe.
'. 'Si tu as un homonyme, utilise prenom.nom.promo comme login'); } } diff --git a/modules/profile.php b/modules/profile.php index 55afef4..9c65055 100644 --- a/modules/profile.php +++ b/modules/profile.php @@ -135,7 +135,7 @@ class ProfileModule extends PLModule if (Env::has('upload')) { $upload = new PlUpload(S::v('forlife'), 'photo'); if (!$upload->upload($_FILES['userfile']) && !$upload->download(Env::v('photo'))) { - $page->trig('Une erreur est survenue lors du téléchargement du fichier'); + $page->trigError('Une erreur est survenue lors du téléchargement du fichier'); } else { $myphoto = new PhotoReq(S::v('uid'), $upload); if ($myphoto->isValid()) { @@ -299,7 +299,7 @@ class ProfileModule extends PLModule } if (Env::v('synchro_ax') == 'confirm' && !is_ax_key_missing()) { ax_synchronize(S::v('bestalias'), S::v('uid')); - $page->trig('Ton profil a été synchronisé avec celui du site polytechniciens.com'); + $page->trigSuccess('Ton profil a été synchronisé avec celui du site polytechniciens.com'); } // Build the page @@ -324,7 +324,7 @@ class ProfileModule extends PLModule FROM auth_user_md5 WHERE user_id = {?} AND naissance = '0000-00-00'", S::i('uid')); if ($res->numRows()) { - $page->trig("Ta date de naissance n'est pas renseignée, ce qui t'empêcheras de réaliser" + $page->trigWarning("Ta date de naissance n'est pas renseignée, ce qui t'empêcheras de réaliser" . " la procédure de récupération de mot de passe si un jour tu le perdais"); } @@ -465,19 +465,19 @@ class ProfileModule extends PLModule $promo_sortie = Env::i('promo_sortie'); if ($promo_sortie < 1000 || $promo_sortie > 9999) { - $page->trig('L\'année de sortie doit être un nombre de quatre chiffres'); + $page->trigError('L\'année de sortie doit être un nombre de quatre chiffres'); } elseif ($promo_sortie < $promo + 3) { - $page->trig('Trop tôt'); + $page->trigError('Trop tôt'); } elseif ($promo_sortie == $promo_sortie_old) { - $page->trig('Tu appartiens déjà à la promotion correspondante à cette année de sortie.'); + $page->trigWarning('Tu appartiens déjà à la promotion correspondante à cette année de sortie.'); } elseif ($promo_sortie == $promo + 3) { XDB::execute( "UPDATE auth_user_md5 set promo_sortie={?} WHERE user_id={?}", $promo_sortie, S::v('uid')); - $page->trig('Ton statut "orange" a été supprimé.'); + $page->trigSuccess('Ton statut "orange" a été supprimé.'); $page->assign('promo_sortie_old', $promo_sortie); } else { diff --git a/modules/profile/general.inc.php b/modules/profile/general.inc.php index ac81f59..1382658 100644 --- a/modules/profile/general.inc.php +++ b/modules/profile/general.inc.php @@ -53,7 +53,7 @@ class ProfileNom implements ProfileSetting || ($field == 'nom' && $new == 'DE ' . $old); if (!$success) { global $page; - $page->trig("Le $field que tu as choisi ($value) est trop loin de ton $field initial ($init)" + $page->trigError("Le $field que tu as choisi ($value) est trop loin de ton $field initial ($init)" . (($init == $current)? "" : " et de ton prénom précédent ($current)")); } return $success ? $value : $current; diff --git a/modules/profile/mentor.inc.php b/modules/profile/mentor.inc.php index 64b1de7..cbe6a23 100644 --- a/modules/profile/mentor.inc.php +++ b/modules/profile/mentor.inc.php @@ -43,7 +43,7 @@ class ProfileSecteurs implements ProfileSetting $value = array(); } else if (count($value) > 10) { global $page; - $page->trig("Le nombre de secteurs d'expertise est limité à 10"); + $page->trigError("Le nombre de secteurs d'expertise est limité à 10"); $success = false; } ksort($value); @@ -91,7 +91,7 @@ class ProfileCountry implements ProfileSetting $value = array(); } else if (count($value) > 10) { global $page; - $page->trig("Le nombre de secteurs d'expertise est limité à 10"); + $page->trigError("Le nombre de secteurs d'expertise est limité à 10"); $success = false; } ksort($value); diff --git a/modules/profile/page.inc.php b/modules/profile/page.inc.php index e259f1e..139178e 100644 --- a/modules/profile/page.inc.php +++ b/modules/profile/page.inc.php @@ -54,7 +54,7 @@ class ProfileWeb extends ProfileNoSave $success = empty($value) || preg_match("{^(https?|ftp)://[a-zA-Z0-9._%#+/?=&~-]+$}i", $value); if (!$success) { global $page; - $page->trig('URL Incorrecte : une url doit commencer par http:// ou https:// ou ftp://' + $page->trigError('URL Incorrecte : une url doit commencer par http:// ou https:// ou ftp://' . ' et ne pas contenir de caractères interdits'); } return $value; @@ -73,7 +73,7 @@ class ProfileEmail extends ProfileNoSave $success = empty($value) || isvalid_email($value); if (!$success) { global $page; - $page->trig('Adresse Email invalide'); + $page->trigError('Adresse Email invalide'); } return $value; } @@ -90,7 +90,7 @@ class ProfileTel extends ProfileNoSave $success = !preg_match('/[<>{}@&#~\/:;?,!§*_`\[\]|%$^=]/', $value, $matches); if (!$success) { global $page; - $page->trig('Le numéro de téléphone contient un caractère interdit : ' . pl_entities($matches[0][0])); + $page->trigError('Le numéro de téléphone contient un caractère interdit : ' . pl_entities($matches[0][0])); } return $value; } @@ -136,7 +136,7 @@ class ProfileDate extends ProfileNoSave $success = preg_match('@(\d{2})/(\d{2})/(\d{4})@', $value, $matches); if (!$success) { global $page; - $page->trig("Les dates doivent être au format jj/mm/aaaa"); + $page->trigError("Les dates doivent être au format jj/mm/aaaa"); } else { $day = (int)$matches[1]; $month = (int)$matches[2]; @@ -144,7 +144,7 @@ class ProfileDate extends ProfileNoSave $success = ($day > 0 && $day <= 31) && ($month > 0 && $month <= 12) && ($year > 1900 && $year <= 2020); if (!$success) { global $page; - $page->trig("La date n'a pas une valeur valide"); + $page->trigError("La date n'a pas une valeur valide"); } } } @@ -330,7 +330,7 @@ abstract class ProfilePage implements PlWizardPage return Post::has('next_page') ? PlWizard::NEXT_PAGE : PlWizard::CURRENT_PAGE; } global $page; - $page->trig("Certains champs n'ont pas pu être validés, merci de corriger les informations " + $page->trigError("Certains champs n'ont pas pu être validés, merci de corriger les informations " . "de ton profil et de revalider ta demande"); return PlWizard::CURRENT_PAGE; } diff --git a/modules/register.php b/modules/register.php index 7fea993..43dc9e9 100644 --- a/modules/register.php +++ b/modules/register.php @@ -219,7 +219,7 @@ class RegisterModule extends PLModule } $page->changeTpl('register/step'.intval($sub_state['step']).'.tpl'); if (isset($err)) { - $page->trig($err); + $page->trigError($err); } } diff --git a/modules/search/classes.inc.php b/modules/search/classes.inc.php index 49cb896..91c924d 100644 --- a/modules/search/classes.inc.php +++ b/modules/search/classes.inc.php @@ -96,7 +96,7 @@ class ThrowError $page->changeTpl('search/index.tpl'); $page->assign('xorg_title','Polytechnique.org - Annuaire'); $page->assign('baseurl', $globals->baseurl); - $page->trig('Erreur : '.$explain); + $page->trigError($explain); $page->run(); } } @@ -254,7 +254,7 @@ class QuickSearch extends SField $this->strings = preg_split("![^a-zA-Z%]+!",$s, -1, PREG_SPLIT_NO_EMPTY); if (count($this->strings) > 5) { global $page; - $page->trig("Tu as indiqué trop d'éléments dans ta recherche, seuls les 5 premiers seront pris en compte"); + $page->trigWarning("Tu as indiqué trop d'éléments dans ta recherche, seuls les 5 premiers seront pris en compte"); $this->strings = array_slice($this->strings, 0, 5); } diff --git a/modules/xnet.php b/modules/xnet.php index c8e5611..b0d0151 100644 --- a/modules/xnet.php +++ b/modules/xnet.php @@ -84,26 +84,26 @@ class XnetModule extends PLModule S::assert_xsrf_token(); XDB::query('DELETE FROM groupex.membres WHERE asso_id={?}', $id); - $page->trig('membres supprimés'); + $page->trigSuccess('membres supprimés'); if ($domain) { XDB::query('DELETE FROM virtual_domains WHERE domain={?}', $domain); XDB::query('DELETE FROM virtual, virtual_redirect USING virtual INNER JOIN virtual_redirect USING (vid) WHERE alias LIKE {?}', '%@'.$domain); - $page->trig('suppression des alias mails'); + $page->trigSuccess('suppression des alias mails'); $mmlist = new MMList(S::v('uid'), S::v('password'), $domain); if ($listes = $mmlist->get_lists()) { foreach ($listes as $l) { $mmlist->delete_list($l['list'], true); } - $page->trig('mail lists surpprimées'); + $page->trigSuccess('mail lists surpprimées'); } } XDB::query('DELETE FROM groupex.asso WHERE id={?}', $id); - $page->trig("Groupe $nom supprimé"); + $page->trigSuccess("Groupe $nom supprimé"); Get::kill('del'); } if (!$id) { diff --git a/modules/xnetevents.php b/modules/xnetevents.php index 056ca5a..417fb3f 100644 --- a/modules/xnetevents.php +++ b/modules/xnetevents.php @@ -216,7 +216,7 @@ class XnetEventsModule extends PLModule if (!isset($pers[$j]) || !is_numeric($pers[$j]) || $pers[$j] < 0) { - $page->trig('Tu dois choisir un nombre d\'invités correct !'); + $page->trigError('Tu dois choisir un nombre d\'invités correct !'); return; } $subs[$j] = 1 + $pers[$j]; @@ -225,11 +225,11 @@ class XnetEventsModule extends PLModule // impossible to unsubscribe if you already paid sthing if (!array_sum($subs) && $evt['paid'] != 0) { - $page->trig("Impossible de te désinscrire complètement ". - "parce que tu as fait un paiement par ". - "chèque ou par liquide. Contacte un ". - "administrateur du groupe si tu es sûr de ". - "ne pas venir"); + $page->trigError("Impossible de te désinscrire complètement ". + "parce que tu as fait un paiement par ". + "chèque ou par liquide. Contacte un ". + "administrateur du groupe si tu es sûr de ". + "ne pas venir"); return; } @@ -527,7 +527,7 @@ class XnetEventsModule extends PLModule if (may_update() && Post::v('adm')) { $member = get_infos(Post::v('mail')); if (!$member) { - $page->trig("Membre introuvable"); + $page->trigError("Membre introuvable"); } // change the price paid by a participant diff --git a/modules/xnetevents/xnetevents.inc.php b/modules/xnetevents/xnetevents.inc.php index b1822da..1303d9c 100644 --- a/modules/xnetevents/xnetevents.inc.php +++ b/modules/xnetevents/xnetevents.inc.php @@ -259,7 +259,7 @@ function event_change_shortname(&$page, $eid, $old, $new) } // Quelques vérifications sur l'alias (caractères spéciaux) if ($new && !preg_match( "/^[a-zA-Z0-9\-.]{3,20}$/", $new)) { - $page->trig("Le raccourci demandé n'est pas valide. + $page->trigError("Le raccourci demandé n'est pas valide. Vérifie qu'il comporte entre 3 et 20 caractères et qu'il ne contient que des lettres non accentuées, des chiffres ou les caractères - et ."); @@ -273,7 +273,7 @@ function event_change_shortname(&$page, $eid, $old, $new) WHERE short_name = {?}', $new); if ($res->fetchOneCell() > 0) { - $page->trig("Le raccourci demandé est déjà utilisé. Choisis en un autre."); + $page->trigError("Le raccourci demandé est déjà utilisé. Choisis en un autre."); return $old; } } diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php index bd2fdf2..270a010 100644 --- a/modules/xnetgrp.php +++ b/modules/xnetgrp.php @@ -209,7 +209,7 @@ class XnetGrpModule extends PLModule global $globals; $site = $globals->asso('site'); if (!$site) { - $page->trig('Le groupe n\'a pas de site web.'); + $page->trigError('Le groupe n\'a pas de site web.'); return $this->handler_index($page); } http_redirect($site); @@ -226,7 +226,7 @@ class XnetGrpModule extends PLModule if (S::has_perms()) { if (Post::v('mail_domain') && (strstr(Post::v('mail_domain'), '.') === false)) { - $page->trig("le domaine doit être un FQDN (aucune modif effectuée) !!!"); + $page->trigError("le domaine doit être un FQDN (aucune modif effectuée) !!!"); return; } XDB::execute( @@ -282,7 +282,7 @@ class XnetGrpModule extends PLModule $page->assign('super', true); } if (!$globals->asso('wiki_desc') && $globals->asso('descr')) { - $page->trig("Attention, le format de la description a changé et utilise désormais la syntaxe wiki " + $page->trigWarning("Attention, le format de la description a changé et utilise désormais la syntaxe wiki " . "intégrée au site. Il te faudra probablement adapter le formatage du texte actuel pour " . "qu'il s'affiche correctement avec cette nouvelle syntaxe."); } @@ -691,7 +691,7 @@ class XnetGrpModule extends PLModule WHERE a.alias={?}', $globals->asso('id'), $forlife); pl_redirect("member/$forlife"); } else { - $page->trig($email." n'est pas un alias polytechnique.org valide."); + $page->trigError($email." n'est pas un alias polytechnique.org valide."); } } else { require_once 'xorg.misc.inc.php'; @@ -715,7 +715,7 @@ class XnetGrpModule extends PLModule $uid, $globals->asso('id'), $email); pl_redirect("member/$email"); } - $page->trig("Utilisateur invalide"); + $page->trigError("Utilisateur invalide"); } else { $res = XDB::query('SELECT MAX(uid)+1 FROM groupex.membres'); $uid = max(intval($res->fetchOneCell()), 50001); @@ -725,7 +725,7 @@ class XnetGrpModule extends PLModule pl_redirect("member/$email"); } } else { - $page->trig("« $email » n'est pas une adresse mail valide."); + $page->trigError("« $email » n'est pas une adresse mail valide."); } } } @@ -791,9 +791,9 @@ class XnetGrpModule extends PLModule $mmlist->unsubscribe($liste['list']); } } elseif ($liste['sub']) { - $page->trig("{$user['prenom']} {$user['nom']} a une" - ." demande d'inscription en cours sur la" - ." liste {$liste['list']}@ !"); + $page->trigWarning("{$user['prenom']} {$user['nom']} a une" + ." demande d'inscription en cours sur la" + ." liste {$liste['list']}@ !"); $warning = true; } } @@ -821,9 +821,9 @@ class XnetGrpModule extends PLModule } if ($this->unsubscribe($user)) { - $page->trig('Vous avez été désinscrit du groupe avec succès.'); + $page->trigSuccess('Vous avez été désinscrit du groupe avec succès.'); } else { - $page->trig('Vous avez été désinscrit du groupe, mais des erreurs se sont produites lors des désinscriptions des alias et des mailing-lists.'); + $page->trigWarning('Vous avez été désinscrit du groupe, mais des erreurs se sont produites lors des désinscriptions des alias et des mailing-lists.'); } $page->assign('is_member', is_member(true)); } @@ -842,9 +842,9 @@ class XnetGrpModule extends PLModule } if ($this->unsubscribe($user)) { - $page->trig("{$user['prenom']} {$user['nom']} a été désabonné du groupe !"); + $page->trigSuccess("{$user['prenom']} {$user['nom']} a été désabonné du groupe !"); } else { - $page->trig("{$user['prenom']} {$user['nom']} a été désabonné du groupe, mais des erreurs subsistent !"); + $page->trigWarning("{$user['prenom']} {$user['nom']} a été désabonné du groupe, mais des erreurs subsistent !"); } } @@ -860,10 +860,10 @@ class XnetGrpModule extends PLModule if ($res->numRows() == 0) { $x = get_not_registered_user($login); if (!$x) { - $page->trig("Le login $login ne correspond à aucun X."); + $page->trigError("Le login $login ne correspond à aucun X."); return false; } else if (count($x) > 1) { - $page->trig("Le login $login correspond a plusieurs camarades."); + $page->trigError("Le login $login correspond a plusieurs camarades."); return false; } $uid = $x[0]['user_id']; @@ -965,7 +965,7 @@ class XnetGrpModule extends PLModule $perms ? 'admin' : 'membre', $user['uid'], $globals->asso('id')); $user['perms'] = $perms; - $page->trig('Permissions modifiées !'); + $page->trigSuccess('Permissions modifiées !'); } // Update ML subscriptions @@ -974,24 +974,24 @@ class XnetGrpModule extends PLModule if ($ask == $state) { if ($state && $email_changed) { $mmlist->replace_email($ml, $from_email, $user['email2']); - $page->trig("L'abonnement de {$user['prenom']} {$user['nom']} à $ml@ a été mis à jour."); + $page->trigSuccess("L'abonnement de {$user['prenom']} {$user['nom']} à $ml@ a été mis à jour."); } continue; } if ($state == '1') { - $page->trig("{$user['prenom']} {$user['nom']} a " + $page->trigWarning("{$user['prenom']} {$user['nom']} a " ."actuellement une demande d'inscription en " ."cours sur $ml@ !!!"); } elseif ($ask) { $mmlist->mass_subscribe($ml, Array($user['email2'])); - $page->trig("{$user['prenom']} {$user['nom']} a été abonné à $ml@."); + $page->trigSuccess("{$user['prenom']} {$user['nom']} a été abonné à $ml@."); } else { if ($email_changed) { $mmlist->mass_unsubscribe($ml, Array($from_email)); } else { $mmlist->mass_unsubscribe($ml, Array($user['email2'])); } - $page->trig("{$user['prenom']} {$user['nom']} a été désabonné de $ml@."); + $page->trigSuccess("{$user['prenom']} {$user['nom']} a été désabonné de $ml@."); } } @@ -1003,14 +1003,14 @@ class XnetGrpModule extends PLModule XDB::query("INSERT INTO virtual_redirect (vid,redirect) SELECT vid,{?} FROM virtual WHERE alias={?}", $user['email'], $ml); - $page->trig("{$user['prenom']} {$user['nom']} a été abonné à $ml."); + $page->trigSuccess("{$user['prenom']} {$user['nom']} a été abonné à $ml."); } else { XDB::query("DELETE FROM virtual_redirect USING virtual_redirect INNER JOIN virtual USING(vid) WHERE redirect={?} AND alias={?}", $user['email'], $ml); - $page->trig("{$user['prenom']} {$user['nom']} a été désabonné de $ml."); + $page->trigSuccess("{$user['prenom']} {$user['nom']} a été désabonné de $ml."); } } } @@ -1063,14 +1063,14 @@ class XnetGrpModule extends PLModule return true; } if (!$upload->upload($_FILES['image']) && !$upload->download(Env::v('image_url'))) { - $page->trig('Impossible de télécharger l\'image'); + $page->trigError('Impossible de télécharger l\'image'); return false; } elseif (!$upload->isType('image')) { - $page->trig('Le fichier n\'est pas une image valide au format JPEG, GIF ou PNG.'); + $page->trigError('Le fichier n\'est pas une image valide au format JPEG, GIF ou PNG.'); $upload->rm(); return false; } elseif (!$upload->resizeImage(200, 300, 100, 100, 32284)) { - $page->trig('Impossible de retraiter l\'image'); + $page->trigError('Impossible de retraiter l\'image'); return false; } return true; @@ -1138,12 +1138,12 @@ class XnetGrpModule extends PLModule ($art['promo_min'] != 0 && ($art['promo_min'] <= 1900 || $art['promo_min'] >= 2020)) || ($art['promo_max'] != 0 && ($art['promo_max'] <= 1900 || $art['promo_max'] >= 2020)))) { - $page->trig("L'intervalle de promotions est invalide."); + $page->trigError("L'intervalle de promotions est invalide."); Post::kill('valid'); } if (!trim($art['titre']) || !trim($art['texte'])) { - $page->trig("L'article doit avoir un titre et un contenu."); + $page->trigError("L'article doit avoir un titre et un contenu."); Post::kill('valid'); } @@ -1202,7 +1202,7 @@ class XnetGrpModule extends PLModule $art['promo_min'], $art['promo_max'], $art['peremption'], "", S::v('uid'), $upload); $article->submit(); - $page->trig("L'affichage sur la page d'accueil de Polytechnique.org est en attente de validation."); + $page->trigWarning("L'affichage sur la page d'accueil de Polytechnique.org est en attente de validation."); } else if ($upload && $upload->exists()) { $upload->rm(); } @@ -1211,7 +1211,7 @@ class XnetGrpModule extends PLModule $article = new NLReq(S::v('uid'), $globals->asso('nom') . " : " .$art['titre'], $art['texte'], $art['contact_html']); $article->submit(); - $page->trig("La parution dans la Lettre Mensuelle est en attente de validation."); + $page->trigWarning("La parution dans la Lettre Mensuelle est en attente de validation."); } } else { XDB::query("UPDATE groupex.announces diff --git a/modules/xnetlists.php b/modules/xnetlists.php index 50f8b9d..f442fea 100644 --- a/modules/xnetlists.php +++ b/modules/xnetlists.php @@ -96,7 +96,7 @@ class XnetListsModule extends ListsModule USING x4dat.virtual AS v LEFT JOIN x4dat.virtual_redirect AS r USING(vid) WHERE v.alias={?}', $alias); - $page->trig(Post::v('del_alias')." supprimé !"); + $page->trigSuccess(Post::v('del_alias')." supprimé !"); } $listes = $this->client->get_lists(); @@ -128,14 +128,14 @@ class XnetListsModule extends ListsModule } if (!Post::has('liste')) { - $page->trig('champs «adresse souhaitée» vide'); + $page->trigError('champs «adresse souhaitée» vide'); return; } $liste = strtolower(Post::v('liste')); if (!preg_match("/^[a-zA-Z0-9\-]*$/", $liste)) { - $page->trig('le nom de la liste ne doit contenir que des lettres non accentuées, chiffres et tirets'); + $page->trigError('le nom de la liste ne doit contenir que des lettres non accentuées, chiffres et tirets'); return; } @@ -143,11 +143,11 @@ class XnetListsModule extends ListsModule $res = XDB::query('SELECT alias FROM x4dat.virtual WHERE alias={?}', $new); if ($res->numRows()) { - $page->trig('cet alias est déjà pris'); + $page->trigError('cet alias est déjà pris'); return; } if (!Post::v('desc')) { - $page->trig('le sujet est vide'); + $page->trigError('le sujet est vide'); return; } @@ -254,9 +254,9 @@ class XnetListsModule extends ListsModule SELECT vid, {?} FROM x4dat.virtual WHERE alias={?}", "$alias@m4x.org", $lfull); - $page->trig("$alias@m4x.org ajouté"); + $page->trigSuccess("$alias@m4x.org ajouté"); } else { - $page->trig("$mbox@{$globals->mail->domain} n'existe pas."); + $page->trigError("$mbox@{$globals->mail->domain} n'existe pas."); } } else { XDB::query( @@ -264,7 +264,7 @@ class XnetListsModule extends ListsModule SELECT vid,{?} FROM x4dat.virtual WHERE alias={?}", "$mbox@$dom", $lfull); - $page->trig("$mbox@$dom ajouté"); + $page->trigSuccess("$mbox@$dom ajouté"); } } @@ -311,13 +311,13 @@ class XnetListsModule extends ListsModule } if (!Post::has('liste')) { - $page->trig('champs «adresse souhaitée» vide'); + $page->trigError('champs «adresse souhaitée» vide'); return; } $liste = Post::v('liste'); if (!preg_match("/^[a-zA-Z0-9\-\.]*$/", $liste)) { - $page->trig('le nom de l\'alias ne doit contenir que des lettres,' - .' chiffres, tirets et points'); + $page->trigError('le nom de l\'alias ne doit contenir que des lettres,' + .' chiffres, tirets et points'); return; } @@ -325,7 +325,7 @@ class XnetListsModule extends ListsModule $res = XDB::query('SELECT COUNT(*) FROM x4dat.virtual WHERE alias={?}', $new); $n = $res->fetchOneCell(); if ($n) { - $page->trig('cet alias est déjà pris'); + $page->trigError('cet alias est déjà pris'); return; } -- 2.1.4