X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=modules%2Fxnetlists.php;h=3f0e79248234754dd7e05203903aa3eb60a8e227;hb=fbf66f5e730def1c20bf36004f02f46c2c96bcfc;hp=1c59e10d5487a36f9c9f5c259f40f1e6e1c56b48;hpb=89cb2025f5f2aed0be3994c89ac473aa11535853;p=platal.git diff --git a/modules/xnetlists.php b/modules/xnetlists.php index 1c59e10..3f0e792 100644 --- a/modules/xnetlists.php +++ b/modules/xnetlists.php @@ -1,6 +1,6 @@ $this->make_hook('lists', AUTH_MDP, 'groupmember'), - '%grp/lists/create' => $this->make_hook('create', AUTH_MDP, 'groupmember'), + '%grp/lists' => $this->make_hook('lists', AUTH_MDP, 'groupmember'), + '%grp/lists/create' => $this->make_hook('create', AUTH_MDP, 'groupmember'), - '%grp/lists/members' => $this->make_hook('members', AUTH_COOKIE), - '%grp/lists/csv' => $this->make_hook('csv', AUTH_COOKIE), - '%grp/lists/annu' => $this->make_hook('annu', AUTH_COOKIE), - '%grp/lists/archives' => $this->make_hook('archives', AUTH_COOKIE), - '%grp/lists/archives/rss' => $this->make_hook('rss', AUTH_PUBLIC), + '%grp/lists/members' => $this->make_hook('members', AUTH_COOKIE), + '%grp/lists/csv' => $this->make_hook('csv', AUTH_COOKIE), + '%grp/lists/annu' => $this->make_hook('annu', AUTH_COOKIE), + '%grp/lists/archives' => $this->make_hook('archives', AUTH_COOKIE), + '%grp/lists/archives/rss' => $this->make_hook('rss', AUTH_PUBLIC), - '%grp/lists/moderate' => $this->make_hook('moderate', AUTH_MDP), - '%grp/lists/admin' => $this->make_hook('admin', AUTH_MDP), - '%grp/lists/options' => $this->make_hook('options', AUTH_MDP), - '%grp/lists/delete' => $this->make_hook('delete', AUTH_MDP), + '%grp/lists/moderate' => $this->make_hook('moderate', AUTH_MDP), + '%grp/lists/admin' => $this->make_hook('admin', AUTH_MDP), + '%grp/lists/options' => $this->make_hook('options', AUTH_MDP), + '%grp/lists/delete' => $this->make_hook('delete', AUTH_MDP), - '%grp/lists/soptions' => $this->make_hook('soptions', AUTH_MDP), - '%grp/lists/check' => $this->make_hook('check', AUTH_MDP), - '%grp/lists/sync' => $this->make_hook('sync', AUTH_MDP), + '%grp/lists/soptions' => $this->make_hook('soptions', AUTH_MDP), + '%grp/lists/check' => $this->make_hook('check', AUTH_MDP), + '%grp/lists/sync' => $this->make_hook('sync', AUTH_MDP), - '%grp/alias/admin' => $this->make_hook('aadmin', AUTH_MDP, 'groupadmin'), - '%grp/alias/create' => $this->make_hook('acreate', AUTH_MDP, 'groupadmin'), + '%grp/alias/admin' => $this->make_hook('aadmin', AUTH_MDP, 'groupadmin'), + '%grp/alias/create' => $this->make_hook('acreate', AUTH_MDP, 'groupadmin'), /* hack: lists uses that */ - 'profile' => $this->make_hook('profile', AUTH_PUBLIC), + 'profile' => $this->make_hook('profile', AUTH_PUBLIC), ); } - function prepare_client(&$page, $user = null) + function prepare_client($page, $user = null) { global $globals; Platal::load('lists', 'lists.inc.php'); @@ -70,9 +70,10 @@ class XnetListsModule extends ListsModule return $globals->asso('mail_domain'); } - function handler_lists(&$page) + function handler_lists($page) { global $globals; + require_once 'emails.inc.php'; if (!$globals->asso('mail_domain')) { return PL_NOT_FOUND; @@ -94,32 +95,26 @@ class XnetListsModule extends ListsModule if (Post::has('del_alias') && may_update()) { S::assert_xsrf_token(); - $alias = Post::v('del_alias'); - // prevent group admin from erasing aliases from other groups - $alias = substr($alias, 0, strpos($alias, '@')).'@'.$globals->asso('mail_domain'); - XDB::query( - 'DELETE FROM r, v - USING x4dat.virtual AS v - LEFT JOIN x4dat.virtual_redirect AS r USING(vid) - WHERE v.alias={?}', $alias); - $page->trigSuccess(Post::v('del_alias')." supprimé !"); + $alias = Post::t('del_alias'); + list($local_part, ) = explode('@', $alias); + delete_list_alias($local_part, $globals->asso('mail_domain')); + $page->trigSuccess($alias . ' supprimé !'); } $listes = $this->client->get_lists(); - $page->assign('listes',$listes); - - $alias = XDB::iterator( - 'SELECT alias,type - FROM x4dat.virtual - WHERE alias - LIKE {?} AND type="user" - ORDER BY alias', '%@'.$globals->asso('mail_domain')); - $page->assign('alias', $alias); - + $page->assign('listes', $listes); + $page->assign('aliases', iterate_list_alias($globals->asso('mail_domain'))); $page->assign('may_update', may_update()); + + if (count($listes) > 0 && !$globals->asso('has_ml')) { + XDB::execute("UPDATE groups + SET flags = CONCAT_WS(',', IF(flags = '', NULL, flags), 'has_ml') + WHERE id = {?}", + $globals->asso('id')); + } } - function handler_create(&$page) + function handler_create($page) { global $globals; @@ -135,58 +130,47 @@ class XnetListsModule extends ListsModule S::assert_xsrf_token(); } - if (!Post::has('liste')) { - $page->trigError('champs «adresse souhaitée» vide'); + if (!Post::has('liste') || !Post::t('liste')) { + $page->trigError('Le champs « adresse souhaitée Â» est vide.'); return; } - $liste = strtolower(Post::v('liste')); - - if (!preg_match("/^[a-zA-Z0-9\-]*$/", $liste)) { + $list = strtolower(Post::t('liste')); + if (!preg_match("/^[a-zA-Z0-9\-]*$/", $list)) { $page->trigError('le nom de la liste ne doit contenir que des lettres non accentuées, chiffres et tirets'); return; } - $new = $liste.'@'.$globals->asso('mail_domain'); - $res = XDB::query('SELECT alias FROM x4dat.virtual WHERE alias={?}', $new); - - if ($res->numRows()) { - $page->trigError('cet alias est déjà pris'); + require_once 'emails.inc.php'; + if (list_exist($list, $globals->asso('mail_domain'))) { + $page->trigError('Cet alias est déjà pris.'); return; } - if (!Post::v('desc')) { - $page->trigError('le sujet est vide'); + if (!Post::t('desc')) { + $page->trigError('Le sujet est vide.'); return; } - $ret = $this->client->create_list( - $liste, utf8_decode(Post::v('desc')), Post::v('advertise'), - Post::v('modlevel'), Post::v('inslevel'), - array(S::user()->forlifeEmail()), array(S::user()->forlifeEmail())); - - $dom = strtolower($globals->asso("mail_domain")); - $red = $dom.'_'.$liste; + $success = $this->client->create_list($list, utf8_decode(Post::t('desc')), Post::t('advertise'), + Post::t('modlevel'), Post::t('inslevel'), + array(S::user()->forlifeEmail()), array(S::user()->forlifeEmail())); - if (!$ret) { + if (!$success) { $page->kill("Un problème est survenu, contacter " ."support@m4x.org"); return; } - foreach (array('', 'owner', 'admin', 'bounces', 'unsubscribe') as $app) { - $mdir = $app == '' ? '+post' : '+' . $app; - if (!empty($app)) { - $app = '-' . $app; - } - XDB::execute('INSERT INTO x4dat.virtual (alias,type) - VALUES({?},{?})', $liste. $app . '@'.$dom, 'list'); - XDB::execute('INSERT INTO x4dat.virtual_redirect (vid,redirect) - VALUES ({?}, {?})', XDB::insertId(), - $red . $mdir . '@listes.polytechnique.org'); - } - pl_redirect('lists/admin/'.$liste); + create_list($list, $globals->asso('mail_domain')); + + XDB::execute("UPDATE groups + SET flags = CONCAT_WS(',', IF(flags = '', NULL, flags), 'has_ml') + WHERE id = {?}", + $globals->asso('id')); + + pl_redirect('lists/admin/' . $list); } - function handler_sync(&$page, $liste = null) + function handler_sync($page, $liste = null) { global $globals; @@ -205,13 +189,11 @@ class XnetListsModule extends ListsModule $mails = array_map(create_function('$arr', 'return $arr[1];'), $members); $subscribers = array_unique($mails); - $not_in_group_x = array(); - $not_in_group_ext = array(); - $ann = XDB::fetchColumn('SELECT uid - FROM groupex.membres + FROM group_members WHERE asso_id = {?}', $globals->asso('id')); - $users = User::getBuildUsersWithUIDs($ann, 'promo,full_name'); + $users = User::getBulkUsersWithUIDs($ann); + $not_in_list = array(); foreach ($users as $user) { if (!in_array(strtolower($user->forlifeEmail()), $subscribers)) { @@ -222,7 +204,7 @@ class XnetListsModule extends ListsModule $page->assign('not_in_list', $not_in_list); } - function handler_aadmin(&$page, $lfull = null) + function handler_aadmin($page, $lfull = null) { global $globals; @@ -231,56 +213,33 @@ class XnetListsModule extends ListsModule } $page->changeTpl('xnetlists/alias-admin.tpl'); + require_once 'emails.inc.php'; + list($local_part, $domain) = explode('@', $lfull); if (Env::has('add_member')) { S::assert_xsrf_token(); - $add = Env::t('add_member'); - $user = User::getSilent($add); + $email = Env::t('add_member'); + $user = User::getSilent($email); if ($user) { - $add = $user->forlifeEmail(); - } else if (!User::isForeignEmailAddress($add)) { - $add = null; - } - if (!empty($add)) { - XDB::execute('INSERT INTO x4dat.virtual_redirect (vid, redirect) - SELECT vid, {?}, - FROM x4dat.virtual - WHERE alias = {?}', strtolower($add), $lfull); - $page->trigSuccess($add . ' ajouté.'); + add_to_list_alias($user, $local_part, $domain); + $page->trigSuccess($email . ' ajouté.'); } else { - $page->trigError($add . ' n\'existe pas.'); + $page->trigError($email . " n'existe pas."); } } if (Env::has('del_member')) { S::assert_xsrf_token(); - XDB::query( - "DELETE FROM x4dat.virtual_redirect - USING x4dat.virtual_redirect - INNER JOIN x4dat.virtual USING(vid) - WHERE redirect={?} AND alias={?}", Env::v('del_member'), $lfull); - pl_redirect('alias/admin/'.$lfull); - } - global $globals; - $emails = XDB::fetchColumn('SELECT redirect - FROM virtual_redirect AS vr - INNER JOIN virtual AS v USING(vid) - WHERE v.alias = {?} - ORDER BY redirect', $lfull); - $mem = array(); - foreach ($emails as $email) { - $user = User::getSilent($email); - if ($user) { - $mem[] = array('user' => $user, 'email' => $email); - } else { - $mem[] = array('email' => $email); - } + $user = User::getSilent(Env::t('del_member')); + delete_from_list_alias($user, $local_part, $domain); + $page->trigSuccess($user->fullName() . ' supprimé.'); } - $page->assign('mem', $mem); + + $page->assign('members', list_alias_members($local_part, $domain)); } - function handler_acreate(&$page) + function handler_acreate($page) { global $globals; @@ -296,30 +255,27 @@ class XnetListsModule extends ListsModule } if (!Post::has('liste')) { - $page->trigError('champs «adresse souhaitée» vide'); + $page->trigError('Le champs « adresse souhaitée Â» est vide.'); return; } - $liste = Post::v('liste'); - if (!preg_match("/^[a-zA-Z0-9\-\.]*$/", $liste)) { - $page->trigError('le nom de l\'alias ne doit contenir que des lettres,' - .' chiffres, tirets et points'); + $list = Post::v('liste'); + if (!preg_match("/^[a-zA-Z0-9\-\.]*$/", $list)) { + $page->trigError('Le nom de l\'alias ne doit contenir que des lettres,' + .' chiffres, tirets et points.'); return; } - $new = $liste.'@'.$globals->asso('mail_domain'); - $res = XDB::query('SELECT COUNT(*) FROM x4dat.virtual WHERE alias={?}', $new); - $n = $res->fetchOneCell(); - if ($n) { - $page->trigError('cet alias est déjà pris'); + require_once 'emails.inc.php'; + if (list_exist($list, $globals->asso('mail_domain'))) { + $page->trigError('Cet alias est déjà pris.'); return; } - XDB::query('INSERT INTO x4dat.virtual (alias,type) VALUES({?}, "user")', $new); - - pl_redirect("alias/admin/$new"); + add_to_list_alias(S::user(), $list, $globals->asso('mail_domain')); + pl_redirect('alias/admin/' . $list . '@' . $globals->asso('mail_domain')); } - function handler_profile(&$page, $user = null) + function handler_profile($page, $user = null) { http_redirect('https://www.polytechnique.org/profile/'.$user); }