X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;ds=sidebyside;f=modules%2Fmarketing.php;h=ac5cff5fbb35a3a28f7a6e300197bb2609b4b37b;hb=c9566e01c2add8f12a858abe0fbb0bc6b8e3c0d4;hp=aad71640655f3254d956d3db8f46536f7c2199cd;hpb=20eca0a987511ea8cfdc57c2e4cb7d00e23002d8;p=platal.git
diff --git a/modules/marketing.php b/modules/marketing.php
index aad7164..ac5cff5 100644
--- a/modules/marketing.php
+++ b/modules/marketing.php
@@ -1,6 +1,6 @@
$this->make_hook('marketing', AUTH_MDP, 'admin'),
- 'marketing/promo' => $this->make_hook('promo', AUTH_MDP, 'admin'),
- 'marketing/relance' => $this->make_hook('relance', AUTH_MDP, 'admin'),
- 'marketing/this_week' => $this->make_hook('week', AUTH_MDP, 'admin'),
- 'marketing/volontaire' => $this->make_hook('volontaire', AUTH_MDP, 'admin'),
-
- 'marketing/private' => $this->make_hook('private', AUTH_MDP, 'admin'),
- 'marketing/public' => $this->make_hook('public', AUTH_COOKIE),
- 'marketing/broken' => $this->make_hook('broken', AUTH_COOKIE),
+ 'marketing' => $this->make_hook('marketing', AUTH_PASSWD, 'admin'),
+ 'marketing/promo' => $this->make_hook('promo', AUTH_PASSWD, 'admin'),
+ 'marketing/relance' => $this->make_hook('relance', AUTH_PASSWD, 'admin'),
+ 'marketing/this_week' => $this->make_hook('week', AUTH_PASSWD, 'admin'),
+ 'marketing/volontaire' => $this->make_hook('volontaire', AUTH_PASSWD, 'admin'),
+
+ 'marketing/private' => $this->make_hook('private', AUTH_PASSWD, 'admin'),
+ 'marketing/public' => $this->make_hook('public', AUTH_COOKIE, 'user'),
+ 'marketing/broken' => $this->make_hook('broken', AUTH_COOKIE, 'user'),
);
}
- function handler_marketing(&$page)
+ function handler_marketing($page)
{
$page->changeTpl('marketing/index.tpl');
$page->setTitle('Marketing');
- $alive = new UserFilter(new PFC_Not(new UFC_Dead()));
- $registered = new UserFilter(new PFC_And(new UFC_Registered(), new PFC_Not(new UFC_Dead())));
- $alive72 = new UserFilter(new PFC_And(new UFC_Promo('>=', UserFilter::GRADE_ING, 1972), new PFC_Not(new UFC_Dead())));
- $registered72 = new UserFilter(new PFC_And(new UFC_Registered(), new UFC_Promo('>=', UserFilter::GRADE_ING, 1972), new PFC_Not(new UFC_Dead())));
- $aliveWomen = new UserFilter(new PFC_And(new UFC_Sex(User::GENDER_FEMALE) , new PFC_Not(new UFC_Dead())));
- $registeredWomen = new UserFilter(new PFC_And(new UFC_Registered(), new UFC_Sex(User::GENDER_FEMALE), new PFC_Not(new UFC_Dead())));
+ $alive = array(
+ 'all' => new ProfileFilter(new PFC_Not(new UFC_Dead())),
+ 'women' => new ProfileFilter(new PFC_And(new UFC_Sex(User::GENDER_FEMALE) , new PFC_Not(new UFC_Dead()))),
+ 'x' => new ProfileFilter(new PFC_And(new PFC_Not(new UFC_Dead()), new UFC_AccountType('x'))),
+ '72' => new ProfileFilter(new PFC_And(new UFC_Promo('>=', UserFilter::GRADE_ING, 1972), new PFC_Not(new UFC_Dead()))),
+ 'master' => new ProfileFilter(new PFC_And(new PFC_Not(new UFC_Dead()), new UFC_AccountType('master'))),
+ 'phd' => new ProfileFilter(new PFC_And(new PFC_Not(new UFC_Dead()), new UFC_AccountType('phd')))
+ );
+ $registered = array(
+ 'all' => new ProfileFilter(new PFC_And(new UFC_Registered(true), new PFC_Not(new UFC_Dead()))),
+ 'women' => new ProfileFilter(new PFC_And(new UFC_Registered(true), new UFC_Sex(User::GENDER_FEMALE), new PFC_Not(new UFC_Dead()))),
+ 'x' => new ProfileFilter(new PFC_And(new UFC_Registered(true), new PFC_Not(new UFC_Dead()), new UFC_AccountType('x'))),
+ '72' => new ProfileFilter(new PFC_And(new UFC_Registered(true), new UFC_Promo('>=', UserFilter::GRADE_ING, 1972), new PFC_Not(new UFC_Dead()))),
+ 'master' => new ProfileFilter(new PFC_And(new UFC_Registered(true), new PFC_Not(new UFC_Dead()), new UFC_AccountType('master'))),
+ 'phd' => new ProfileFilter(new PFC_And(new UFC_Registered(true), new PFC_Not(new UFC_Dead()), new UFC_AccountType('phd')))
+ );
$statistics = array(
- 'alive' => $alive->getTotalCount(),
- 'registered' => $registered->getTotalCount(),
- 'alive72' => $alive72->getTotalCount(),
- 'registered72' => $registered72->getTotalCount(),
- 'womenAlive' => $aliveWomen->getTotalCount(),
- 'womenRegistered' => $registeredWomen->getTotalCount(),
+ 'all' => array('description' => "Ãtudiants et anciens de l'X"),
+ 'women' => array('description' => "Ãtudiantes et anciennes de l'X"),
+ 'x' => array('description' => 'X'),
+ '72' => array('description' => 'X vivants depuis la promo 1972'),
+ 'master' => array('description' => "Masters de l'X"),
+ 'phd' => array('description' => "Docteurs de l'X")
);
- $statistics['registeredRate'] = $statistics['registered'] / $statistics['alive'] * 100;
- $statistics['registeredRate72'] = $statistics['registered72'] / $statistics['alive72'] * 100;
- $statistics['womenRegisteredRate'] = $statistics['womenRegistered'] / $statistics['womenAlive'] * 100;
+ foreach ($statistics as $key => &$data) {
+ $data['alive'] = $alive[$key]->getTotalCount();
+ $data['registered'] = $registered[$key]->getTotalCount();
+ $data['rate'] = round($data['registered'] / $data['alive'] * 100, 2);
+ }
- $registeredWeek = new UserFilter(new PFC_And(new UFC_Registered(false, '>=', strtotime('1 week ago')), new PFC_Not(new UFC_Dead())));
+ $registeredWeek = new ProfileFilter(new PFC_And(new UFC_Registered(true, '>=', strtotime('1 week ago')), new PFC_Not(new UFC_Dead())));
$registrationPending = XDB::fetchOneCell('SELECT COUNT(*)
FROM register_pending');
$registrations = array(
@@ -97,7 +109,7 @@ class MarketingModule extends PLModule
$page->assign('marketings', $marketings);
}
- function handler_private(&$page, $hruid = null,
+ function handler_private($page, $hruid = null,
$action = null, $value = null)
{
global $globals;
@@ -174,9 +186,9 @@ class MarketingModule extends PLModule
// Retrieves and display the existing marketing attempts.
$res = XDB::iterator(
- "SELECT r.*, a.alias
- FROM register_marketing AS r
- LEFT JOIN aliases AS a ON (r.sender = a.uid AND a.type = 'a_vie')
+ "SELECT r.*, s.email AS alias
+ FROM register_marketing AS r
+ LEFT JOIN email_source_account AS s ON (r.sender = s.uid AND s.type = 'forlife')
WHERE r.uid = {?}
ORDER BY date", $user->id());
$page->assign('addr', $res);
@@ -191,7 +203,7 @@ class MarketingModule extends PLModule
$page->assign('path', 'marketing/private/' . $user->login());
}
- function handler_broken(&$page, $uid = null)
+ function handler_broken($page, $uid = null)
{
$page->changeTpl('marketing/broken.tpl');
@@ -206,15 +218,14 @@ class MarketingModule extends PLModule
pl_redirect('emails/redirect');
}
- $res = XDB::query('SELECT p.deathdate IS NULL AS alive, e.last,
- IF(e.email IS NOT NULL, e.email,
- IF(FIND_IN_SET(\'googleapps\', eo.storage), \'googleapps\', NULL)) AS email
- FROM email_options AS eo
- LEFT JOIN account_profiles AS ap ON (ap.uid = eo.uid AND FIND_IN_SET(\'owner\', ap.perms))
- LEFT JOIN profiles AS p ON (p.pid = ap.pid)
- LEFT JOIN emails AS e ON (e.flags = \'active\' AND e.uid = eo.uid)
- WHERE eo.uid = {?}
- ORDER BY e.panne_level, e.last', $user->id());
+ $res = XDB::query('SELECT p.deathdate IS NULL AS alive, r.last, IF(r.type = \'googleapps\', \'googleapps\', r.redirect) AS active_email
+ FROM accounts AS a
+ LEFT JOIN email_redirect_account AS r ON (a.uid = r.uid AND r.type IN (\'smtp\', \'googleapps\') AND r.flags = \'active\')
+ LEFT JOIN account_profiles AS ap ON (ap.uid = r.uid AND FIND_IN_SET(\'owner\', ap.perms))
+ LEFT JOIN profiles AS p ON (p.pid = ap.pid)
+ WHERE a.uid = {?}
+ ORDER BY r.broken_level, r.last',
+ $user->id());
if (!$res->numRows()) {
return PL_NOT_FOUND;
}
@@ -226,34 +237,33 @@ class MarketingModule extends PLModule
if (Post::has('mail')) {
$email = valide_email(Post::v('mail'));
}
- if (Post::has('valide') && isvalid_email_redirection($email)) {
+ if (Post::has('valide') && isvalid_email_redirection($email, $user)) {
S::assert_xsrf_token();
// security stuff
check_email($email, "Proposition d'une adresse surveillee pour " . $user->login() . " par " . S::user()->login());
- $res = XDB::query("SELECT flags
- FROM emails
- WHERE email = {?} AND uid = {?}", $email, $user->id());
- $state = $res->numRows() ? $res->fetchOneCell() : null;
- if ($state == 'panne') {
+ $state = XDB::fetchOneCell('SELECT flags
+ FROM email_redirect_account
+ WHERE redirect = {?} AND uid = {?}',
+ $email, $user->id());
+ if ($state == 'broken') {
$page->trigWarning("L'adresse que tu as fournie est l'adresse actuelle de {$user->fullName()} et est en panne.");
} elseif ($state == 'active') {
$page->trigWarning("L'adresse que tu as fournie est l'adresse actuelle de {$user->fullName()}");
- } elseif ($user->email && !trim(Post::v('comment'))) {
+ } elseif ($user->email && !Post::t('comment')) {
$page->trigError("Il faut que tu ajoutes un commentaire à ta proposition pour justifier le "
. "besoin de changer la redirection de {$user->fullName()}.");
} else {
- require_once 'validations.inc.php';
$valid = new BrokenReq(S::user(), $user, $email, trim(Post::v('comment')));
$valid->submit();
$page->assign('sent', true);
}
} elseif ($email) {
- $page->trigError("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.");
}
}
- function handler_promo(&$page, $promo = null)
+ function handler_promo($page, $promo = null)
{
$page->changeTpl('marketing/promo.tpl');
@@ -263,13 +273,14 @@ class MarketingModule extends PLModule
$page->assign('promo', $promo);
$uf = new UserFilter(new PFC_And(new UFC_Promo('=', UserFilter::DISPLAY, $promo),
- new PFC_Not(new UFC_Registered())),
- array(new UFO_Name(Profile::LASTNAME), new UFO_Name(Profile::FIRSTNAME)));
+ new PFC_Not(new UFC_Registered()),
+ new PFC_Not(new UFC_Dead())),
+ array(new UFO_Name()));
$users = $uf->getUsers();
$page->assign('nonins', $users);
}
- function handler_public(&$page, $hruid = null)
+ function handler_public($page, $hruid = null)
{
$page->changeTpl('marketing/public.tpl');
@@ -292,7 +303,7 @@ class MarketingModule extends PLModule
$email = trim(Post::v('mail'));
require_once 'emails.inc.php';
- if (!isvalid_email_redirection($email)) {
+ if (!isvalid_email_redirection($email, $user)) {
$page->trigError('Email invalide !');
} else {
// On cherche les marketings précédents sur cette adresse
@@ -313,24 +324,24 @@ class MarketingModule extends PLModule
require_once 'marketing.inc.php';
$sender = User::getSilent(S::v('uid'));
+ $perso_signature = 'Cordialement,
--
' . $sender->fullName();
$market = new AnnuaireMarketing(null, true);
$text = $market->getText(array(
'sexe' => $user->isFemale(),
- 'forlife_email' => $user->login() . '@' . $globals->mail->domain,
- 'forlife_email2' => $user->login() . '@' . $globals->mail->domain2
+ 'forlife_email' => $user->forlifeEmail(),
+ 'forlife_email2' => $user->forlifeEmailAlternate()
));
$text = str_replace('%%hash%%', '', $text);
$text = str_replace('%%personal_notes%%', '', $text);
$text = str_replace('%%sender%%',
- "" . $sender->fullName() . '', $text);
+ '' . $perso_signature . '', $text);
$page->assign('text', nl2br($text));
- // TODO (JAC): define a unique Xorg signature for all the emails we send.
- $page->assign('xorg_signature', "L'équipe de Polytechnique.org,
Le portail des élèves & anciens élèves de l'Ãcole polytechnique");
- $page->assign('perso_signature', $sender->fullName());
+ $page->assign('perso_signature', $perso_signature);
+ $page->assign('mail_part', 'escaped_html');
}
}
- function handler_week(&$page, $sorting = 'per_promo')
+ function handler_week($page, $sorting = 'per_promo')
{
$page->changeTpl('marketing/this_week.tpl');
@@ -340,7 +351,7 @@ class MarketingModule extends PLModule
$page->assign('users', $uf->getUsers());
}
- function handler_volontaire(&$page, $promo = null)
+ function handler_volontaire($page, $promo = null)
{
$page->changeTpl('marketing/volontaire.tpl');
@@ -354,18 +365,18 @@ class MarketingModule extends PLModule
if (!is_null($promo)) {
- $it = XDB::iterator('SELECT m.uid, m.email, s.alias AS forlife
+ $it = XDB::iterator('SELECT m.uid, m.email, s.email AS forlife
FROM register_marketing AS m
INNER JOIN account_profiles AS ap ON (m.uid = ap.uid AND FIND_IN_SET(\'owner\', ap.perms))
INNER JOIN profile_display AS pd ON (pd.pid = ap.pid)
- LEFT JOIN aliases AS s ON (m.sender = s.uid AND s.type = \'a_vie\')
+ LEFT JOIN email_source_account AS s ON (m.sender = s.uid AND s.type = \'forlife\')
WHERE pd.promo = {?}
ORDER BY pd.sort_name', $promo);
$page->assign('addr', $it);
}
}
- function handler_relance(&$page)
+ function handler_relance($page)
{
$page->changeTpl('marketing/relance.tpl');