X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=modules%2Fxnetgrp.php;h=4b5e9f6055c8413689a96237ddf9c288be8aca91;hb=a208363f3edb12278eb38ff3a4628ca8af54e1e5;hp=dd225acb70bafd11e563975e671716e6ed452639;hpb=c4f03937349bddfad25990239a825f8161615260;p=platal.git diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php index dd225ac..4b5e9f6 100644 --- a/modules/xnetgrp.php +++ b/modules/xnetgrp.php @@ -32,6 +32,7 @@ class XnetGrpModule extends PLModule '%grp/edit' => $this->make_hook('edit', AUTH_MDP, 'groupadmin'), '%grp/mail' => $this->make_hook('mail', AUTH_MDP, 'groupadmin'), '%grp/forum' => $this->make_hook('forum', AUTH_MDP, 'groupmember'), + '%grp/former_users' => $this->make_hook('former_users', AUTH_MDP, 'admin'), '%grp/annuaire' => $this->make_hook('annuaire', AUTH_MDP, 'groupannu'), '%grp/annuaire/vcard' => $this->make_hook('vcard', AUTH_MDP, 'groupmember:groupannu'), '%grp/annuaire/csv' => $this->make_hook('csv', AUTH_MDP, 'groupmember:groupannu'), @@ -324,53 +325,29 @@ class XnetGrpModule extends PLModule { global $globals; - if ($action == 'trombi') { - __autoload('userset'); - if ($action == 'trombi') { - $view = new ProfileSet(new UFC_Group($globals->asso('id'))); - } else { - $view = new UserSet(new UFC_Group($globals->asso('id'))); - } - $view->addMod('trombi', 'Trombinoscope'); - $view->apply('annuaire', $page, $action, $subaction); - $page->changeTpl('xnetgrp/annuaire.tpl'); - $count = XDB::fetchOneCell('SELECT COUNT(*) - FROM group_members - WHERE asso_id = {?}', - $globals->asso('id')); - $page->assign('nb_tot', $count); - return; - } - + __autoload('userset'); + $admins = false; + if ($action == 'admins') { + $admins = true; + $action = $subaction; + } + $view = new UserSet(new UFC_Group($globals->asso('id'), $admins)); + $view->addMod('groupmember', 'Annuaire'); + $view->addMod('trombi', 'Trombinoscope'); + $view->apply('annuaire', $page, $action); + $page->assign('only_admin', $admins); $page->changeTpl('xnetgrp/annuaire.tpl'); - $sort = Env::s('order', 'directory_name'); - $ofs = Env::i('offset'); - if ($ofs < 0) { - $ofs = 0; - } - - $sdesc = $sort{0} == '-'; - $sf = $sdesc ? substr($sort, 1) : $sort; - if ($sf == 'promo') { - $se = new UFO_Promo(null, $sdesc); - } else { - $se = new UFO_Name($sf, null, null, $sdesc); - } + } - if (Env::b('admin')) { - $uf = $globals->asso()->getAdminsFilter(null, $se); - } else { - $uf = $globals->asso()->getMembersFilter(null, $se); - } - $users = $uf->getUsers(new PlLimit(NB_PER_PAGE, $ofs * NB_PER_PAGE)); - $count = $uf->getTotalCount(); + function handler_former_users($page) + { + global $globals; + require_once 'userset.inc.php'; - $page->assign('nb_tot', $count); - $page->assign('pages', floor(($count + NB_PER_PAGE - 1) / NB_PER_PAGE)); - $page->assign('current', $ofs); - $page->assign('order', $sort); - $page->assign('users', $users); - $page->assign('only_admin', Env::b('admin')); + $view = new UserSet(new UFC_GroupFormerMember($globals->asso('id'))); + $view->addMod('groupmember', 'Anciens membres', true, array('noadmin' => true)); + $view->apply('former_users', $page); + $page->changeTpl('xnetgrp/former_users.tpl'); } function handler_trombi($page) @@ -415,8 +392,24 @@ class XnetGrpModule extends PLModule S::assert_xsrf_token(); $users = array_keys(Env::v('add_users')); + $former_users = XDB::fetchColumn('SELECT uid + FROM group_former_members + WHERE remember = TRUE AND uid IN {?}', + $users); + $new_users = array_diff($users, $former_users); + + foreach ($former_users as $uid) { + $user = User::getSilentWithUID($uid); + $page->trigWarning($user->fullName() . ' est un ancien membre du groupe qui ne souhaite pas y revenir.'); + } + if (count($former_users) > 1) { + $page->trigWarning('S\'ils souhaitent revenir dans le groupe, il faut qu\'ils en fassent la demande sur la page d\'accueil du groupe.'); + } elseif (count($former_users)) { + $page->trigWarning('S\'il souhaite revenir dans le groupe, il faut qu\'il en fasse la demande sur la page d\'accueil du groupe.'); + } + $data = array(); - foreach ($users as $uid) { + foreach ($new_users as $uid) { $data[] = XDB::format('({?}, {?})', $globals->asso('id'), $uid); } XDB::rawExecute('INSERT INTO group_members (asso_id, uid) @@ -495,15 +488,19 @@ class XnetGrpModule extends PLModule } $nonusers = array_unique($nonusers); $uids = array_unique($uids); - $uids = XDB::fetchColumn('SELECT a.uid - FROM accounts AS a - WHERE a.uid IN {?} AND NOT EXISTS (SELECT * - FROM group_members AS g - WHERE a.uid = g.uid AND g.asso_id = {?})', - $uids, $globals->asso('id')); - - $users = User::getBulkUsersWithUIDs($uids); - usort($users, 'User::compareDirectoryName'); + if (count($uids)) { + $uids = XDB::fetchColumn('SELECT a.uid + FROM accounts AS a + WHERE a.uid IN {?} AND NOT EXISTS (SELECT * + FROM group_members AS g + WHERE a.uid = g.uid AND g.asso_id = {?})', + $uids, $globals->asso('id')); + + $users = User::getBulkUsersWithUIDs($uids); + usort($users, 'User::compareDirectoryName'); + } else { + $users = array(); + } sort($nonusers); $page->assign('users', $users); @@ -561,9 +558,8 @@ class XnetGrpModule extends PLModule { global $globals; $this->removeSubscriptionRequest($user->id()); - XDB::execute("INSERT IGNORE INTO group_members (asso_id, uid) - VALUES ({?}, {?})", - $globals->asso('id'), $user->id()); + Group::subscribe($globals->asso('id'), $user->id()); + if (XDB::affectedRows() == 1) { $mailer = new PlMailer(); $mailer->addTo($user->forlifeEmail()); @@ -661,6 +657,9 @@ class XnetGrpModule extends PLModule XDB::execute("INSERT INTO group_member_sub_requests (asso_id, uid, ts, reason) VALUES ({?}, {?}, NOW(), {?})", $globals->asso('id'), S::i('uid'), Post::v('message')); + XDB::execute('DELETE FROM group_former_members + WHERE uid = {?} AND asso_id = {?}', + S::i('uid'), $globals->asso('id')); $uf = New UserFilter(New UFC_Group($globals->asso('id'), true)); $admins = $uf->iterUsers(); $admin = $admins->next(); @@ -886,9 +885,23 @@ class XnetGrpModule extends PLModule } if ($user) { - XDB::execute('INSERT IGNORE INTO group_members (uid, asso_id) - VALUES ({?}, {?})', - $user->id(), $globals->asso('id')); + // First check if the user used to be in this group. + XDB::rawExecute('DELETE FROM group_former_members + WHERE remember AND DATE_SUB(NOW(), INTERVAL 1 YEAR) > unsubsciption_date'); + $former_member = XDB::fetchOneCell('SELECT remember + FROM group_former_members + WHERE uid = {?} AND asso_id = {?}', + $user->id(), $globals->asso('id')); + if ($former_member === 1) { + $page->trigError($user->fullName() . ' est un ancien membre du groupe qui ne souhaite pas y revenir. S\'il souhaite revenir dans le groupe, il faut qu\'il en fasse la demande sur la page d\'accueil du groupe.'); + return; + } elseif (!is_null($former_member) && Post::i('force_continue') == 0) { + $page->trigWarning($user->fullName() . ' est un ancien membre du groupe qui s\'est récemment désinscrit. Malgré cela, si tu penses qu\'il souhaite revenir, cliquer sur « Ajouter » l\'ajoutera bien au groupe cette fois.'); + $page->assign('force_continue', 1); + return; + } + + Group::subscribe($globals->asso('id'), $user->id()); $this->removeSubscriptionRequest($user->id()); if ($suggest_account_activation) { pl_redirect('member/suggest/' . $user->login() . '/' . $email . '/' . $globals->asso('nom')); @@ -975,12 +988,10 @@ class XnetGrpModule extends PLModule $page->assign('users', $users); } - function unsubscribe(PlUser $user) + function unsubscribe(PlUser $user, $remember = false) { global $globals; - XDB::execute("DELETE FROM group_members - WHERE uid = {?} AND asso_id = {?}", - $user->id(), $globals->asso('id')); + Group::unsubscribe($globals->asso('id'), $user->id(), $remember); if ($globals->asso('notif_unsub')) { $mailer = new PlMailer('xnetgrp/unsubscription-notif.mail.tpl'); @@ -1047,7 +1058,7 @@ class XnetGrpModule extends PLModule $hasSingleGroup = ($user->groupCount() == 1); - if ($this->unsubscribe($user)) { + if ($this->unsubscribe($user, Post::b('remember'))) { $page->trigSuccess('Tu as été désinscrit du groupe avec succès.'); } else { $page->trigWarning('Tu as été désinscrit du groupe, mais des erreurs se sont produites lors des désinscriptions des alias et des listes de diffusion.'); @@ -1265,16 +1276,39 @@ class XnetGrpModule extends PLModule $page->trigSuccess("{$user->fullName()} a été désabonné de $ml."); } } + + if ($globals->asso('has_nl')) { + // Updates group's newsletter subscription. + if (Post::i('newsletter') == 1) { + XDB::execute('INSERT IGNORE INTO newsletter_ins (uid, nlid) + SELECT {?}, id + FROM newsletters + WHERE group_id = {?}', + $user->id(), $globals->asso('id')); + } else { + XDB::execute('DELETE ni + FROM newsletter_ins AS ni + INNER JOIN newsletters AS n ON (n.id = ni.nlid) + WHERE ni.uid = {?} AND n.group_id = {?}', + $user->id(), $globals->asso('id')); + } + } } $res = XDB::rawFetchAllAssoc('SHOW COLUMNS FROM group_members LIKE \'position\''); $positions = str_replace(array('enum(', ')', '\''), '', $res[0]['Type']); + $nl_registered = XDB::fetchOneCell('SELECT COUNT(ni.uid) + FROM newsletter_ins AS ni + INNER JOIN newsletters AS n ON (n.id = ni.nlid) + WHERE ni.uid = {?} AND n.group_id = {?}', + $user->id(), $globals->asso('id')); $page->assign('user', $user); $page->assign('suggest', $this->suggest($user)); $page->assign('listes', $mmlist->get_lists($user->forlifeEmail())); $page->assign('alias', $user->emailGroupAliases($globals->asso('mail_domain'))); $page->assign('positions', explode(',', $positions)); + $page->assign('nl_registered', $nl_registered); } function handler_rss(PlPage $page, PlUser $user) @@ -1299,7 +1333,7 @@ class XnetGrpModule extends PLModule $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)) { + } elseif (!$upload->resizeImage(80, 100, 100, 100, 32284)) { $page->trigError('Impossible de retraiter l\'image'); return false; }