From: Vincent Zanotti Date: Sat, 23 Aug 2008 18:01:40 +0000 (+0200) Subject: Switches marketing/public and marketing/private pages to hruid based urls. X-Git-Tag: xorg/0.10.0~86^2~35 X-Git-Url: http://git.polytechnique.org/?a=commitdiff_plain;h=2455266e47619d52a3ffd66685490d22c456d337;p=platal.git Switches marketing/public and marketing/private pages to hruid based urls. Signed-off-by: Vincent Zanotti --- diff --git a/modules/marketing.php b/modules/marketing.php index 41219af..69160fa 100644 --- a/modules/marketing.php +++ b/modules/marketing.php @@ -76,41 +76,43 @@ class MarketingModule extends PLModule $page->assign('nbInsMarkOK', $res->fetchOneCell()); } - function handler_private(&$page, $uid = null, + function handler_private(&$page, $hruid = null, $action = null, $value = null) { global $globals; $page->changeTpl('marketing/private.tpl'); - if (is_null($uid)) { + $user = User::getSilent($hruid); + if (!$user) { return PL_NOT_FOUND; } - $page->assign('path', 'marketing/private/'.$uid); - - $res = XDB::query("SELECT nom, prenom, promo, matricule - FROM auth_user_md5 - WHERE user_id={?} AND perms='pending'", $uid); - - if (list($nom, $prenom, $promo, $matricule) = $res->fetchOneRow()) { - require_once('user.func.inc.php'); - $matricule_X = get_X_mat($matricule); - $page->assign('nom', $nom); - $page->assign('prenom', $prenom); - $page->assign('promo', $promo); - $page->assign('matricule', $matricule); - $page->assign('matricule_X',$matricule_X); - } else { - $page->kill('uid invalide'); + // Retrieves marketed user details. + $res = XDB::query( + "SELECT matricule + FROM auth_user_md5 + WHERE user_id = {?} AND perms = 'pending'", $user->id()); + if (!($matricule = $res->fetchOneCell())) { + $page->kill("Cet utilisateur est déjà inscrit au site."); } + require_once('user.func.inc.php'); + $matricule = $res->fetchOneCell(); + $matricule_X = get_X_mat($matricule); + + $page->assign('full_name', $user->fullName()); + $page->assign('promo', $user->promo()); + $page->assign('matricule', $matricule); + $page->assign('matricule_X',$matricule_X); + + // Applies in-parameter action to the user. if ($action == 'del') { S::assert_xsrf_token(); - Marketing::clear($uid, $value); + Marketing::clear($user->id(), $value); } if ($action == 'rel') { - $market = Marketing::get($uid, $value); + $market = Marketing::get($user->id(), $value); if ($market == null) { $page->trigWarning("Aucun marketing n'a été effectué vers $value"); } else { @@ -131,9 +133,9 @@ class MarketingModule extends PLModule if ($action == 'relforce') { S::assert_xsrf_token(); - $market = Marketing::get($uid, Post::v('to')); + $market = Marketing::get($user->id(), Post::v('to')); if (is_null($market)) { - $market = new Marketing($uid, Post::v('to'), 'default', null, 'staff'); + $market = new Marketing($user->id(), Post::v('to'), 'default', null, 'staff'); } $market->send(Post::v('title'), Post::v('message')); $page->trigSuccess("Email envoyé"); @@ -141,30 +143,33 @@ class MarketingModule extends PLModule if ($action == 'insrel') { S::assert_xsrf_token(); - if (Marketing::relance($uid)) { + if (Marketing::relance($user->id())) { $page->trigSuccess('relance faite'); } } if ($action == 'add' && Post::has('email') && Post::has('type')) { - $market = new Marketing($uid, Post::v('email'), 'default', null, Post::v('type'), S::v('uid')); + $market = new Marketing($user->id(), Post::v('email'), 'default', null, Post::v('type'), S::v('uid')); $market->add(false); } + // Retrieves and display the existing marketing attempts. $res = XDB::iterator( "SELECT r.*, a.alias FROM register_marketing AS r INNER JOIN aliases AS a ON (r.sender=a.id AND a.type = 'a_vie') WHERE uid={?} - ORDER BY date", $uid); + ORDER BY date", $user->id()); $page->assign('addr', $res); $res = XDB::query("SELECT date, relance FROM register_pending - WHERE uid = {?}", $uid); + WHERE uid = {?}", $user->id()); if (list($pending, $relance) = $res->fetchOneRow()) { $page->assign('pending', $pending); $page->assign('relance', $relance); } + + $page->assign('path', 'marketing/private/' . $user->login()); } function handler_broken(&$page, $uid = null) @@ -249,41 +254,46 @@ class MarketingModule extends PLModule $page->assign('nonins', XDB::iterator($sql, $promo)); } - function handler_public(&$page, $uid = null) + function handler_public(&$page, $hruid = null) { $page->changeTpl('marketing/public.tpl'); - if (is_null($uid)) { + // Retrieves the user info, and checks the user is not yet registered. + $user = User::getSilent($hruid); + if (!$user) { return PL_NOT_FOUND; } - $res = XDB::query("SELECT nom, prenom, promo FROM auth_user_md5 - WHERE user_id={?} AND perms='pending'", $uid); + $res = XDB::query( + "SELECT COUNT(*) + FROM auth_user_md5 + WHERE user_id = {?} AND perms = 'pending'", $user->id()); + if (!$res->fetchOneCell()) { + $page->kill("Cet utilisateur est déjà inscrit au site."); + } + + // Displays the page, and handles the eventual user actions. + $page->assign('full_name', $user->fullName()); + $page->assign('promo', $user->promo()); - if (list($nom, $prenom, $promo) = $res->fetchOneRow()) { - $page->assign('prenom', $prenom); - $page->assign('nom', $nom); - $page->assign('promo', $promo); + if (Post::has('valide')) { + S::assert_xsrf_token(); + $email = trim(Post::v('mail')); - if (Post::has('valide')) { - S::assert_xsrf_token(); - $email = trim(Post::v('mail')); + require_once 'emails.inc.php'; + if (!isvalid_email_redirection($email)) { + $page->trigError("Email invalide !"); + } else { + // On cherche les marketings précédents sur cette adresse + // email, en se restreignant au dernier mois - require_once 'emails.inc.php'; - if (!isvalid_email_redirection($email)) { - $page->trigError("Email invalide !"); + if (Marketing::get($user->id(), $email, true)) { + $page->assign('already', true); } else { - // On cherche les marketings précédents sur cette adresse - // email, en se restreignant au dernier mois - - if (Marketing::get($uid, $email, true)) { - $page->assign('already', true); - } else { - $page->assign('ok', true); - check_email($email, "Une adresse surveillée est proposée au marketing par " . S::v('forlife')); - $market = new Marketing($uid, $email, 'default', null, Post::v('origine'), S::v('uid')); - $market->add(); - } + $page->assign('ok', true); + check_email($email, "Une adresse surveillée est proposée au marketing par " . S::user()->login()); + $market = new Marketing($user->id(), $email, 'default', null, Post::v('origine'), S::v('uid')); + $market->add(); } } } diff --git a/templates/marketing/private.tpl b/templates/marketing/private.tpl index aee32ff..0ca3bba 100644 --- a/templates/marketing/private.tpl +++ b/templates/marketing/private.tpl @@ -20,7 +20,7 @@ {* *} {**************************************************************************} -

Marketing de {$prenom} {$nom}

+

Marketing de {$full_name}

Matricules

diff --git a/templates/marketing/public.tpl b/templates/marketing/public.tpl index a9a3e98..88e38b6 100644 --- a/templates/marketing/public.tpl +++ b/templates/marketing/public.tpl @@ -27,7 +27,7 @@ Merci de nous avoir communiqué cette information !

-Nous avions déjà connaissance de cette adresse, nous espérons donc comme toi que {$prenom} va s'inscrire au plus vite. +Nous avions déjà connaissance de cette adresse, nous espérons donc comme toi que {$full_name} va s'inscrire au plus vite.

Si tu le connais personnellement, un petit email pour lui expliquer les atouts de Polytechnique.org @@ -38,21 +38,21 @@ peut sans aucun doute l'aider à se décider !

Merci de nous avoir communiqué cette information ! Un administrateur de Polytechnique.org va - envoyer un email de proposition d'inscription à Polytechnique.org à {$prenom} {$nom} dans les + envoyer un email de proposition d'inscription à Polytechnique.org à {$full_name} dans les toutes prochaines heures (ceci est fait à la main pour vérifier qu'aucun utilisateur malveillant ne fasse mauvais usage de cette fonctionnalité...).

Merci de ton aide à la reconnaissance de notre site ! Tu seras informé par email de - l'inscription de {$prenom} {$nom} si notre camarade accepte de rejoindre la communauté des X sur + l'inscription de {$full_name} si notre camarade accepte de rejoindre la communauté des X sur le web !

{else} -{if $prenom} +{if $full_name}

- Et si nous proposions à {$prenom} {$nom} de s'inscrire à Polytechnique.org ? + Et si nous proposions à {$full_name} de s'inscrire à Polytechnique.org ?

@@ -61,7 +61,7 @@ peut sans aucun doute l'aider à se décider ! proposition d'inscription.

- Si tu es d'accord, merci d'indiquer ci-dessous l'adresse email de {$prenom} {$nom} si tu la + Si tu es d'accord, merci d'indiquer ci-dessous l'adresse email de {$full_name} si tu la connais. Nous nous permettons d'attirer ton attention sur le fait que nous avons besoin d'être sûrs que cette adresse est bien la sienne, afin que la partie privée du site reste uniquement accessible aux seuls polytechniciens. Merci donc de ne nous donner ce renseignement uniquement si @@ -69,15 +69,14 @@ peut sans aucun doute l'aider à se décider !

Nous pouvons au choix lui écrire au nom de l'équipe Polytechnique.org, ou bien, si tu le veux - bien, en ton nom. A toi de choisir la solution qui te paraît la plus adaptée !! Une fois {$prenom} - {$nom} inscrit, nous t'enverrons un email pour te prévenir que son inscription a réussi. + bien, en ton nom. A toi de choisir la solution qui te paraît la plus adaptée !! Une fois + {$full_name} inscrit, nous t'enverrons un email pour te prévenir que son inscription a réussi.

{xsrf_token_field} - - + diff --git a/templates/profile/profile.tpl b/templates/profile/profile.tpl index 138fca4..0b35dd7 100644 --- a/templates/profile/profile.tpl +++ b/templates/profile/profile.tpl @@ -99,7 +99,7 @@ function chgMainWinLoc(strPage) clique ici si tu connais son adresse email ! {elseif !$x.inscrit} Cette personne n'est pas inscrite à Polytechnique.org,
- clique ici si tu connais son adresse email ! + clique ici si tu connais son adresse email ! {else} {if $virtualalias} {$virtualalias}
Nom :{$nom}
Prénom :{$prenom}
Nom :{$full_name}
Promo :{$promo}
Adresse email :