X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=modules%2Fxnetgrp.php;h=08342a085580f2ee977625c16fc094cee22ce0d8;hb=911075226546c6b058eca1e0b889097f79fa6232;hp=9274953705b2d6472a260656fd58e5ded782f0e8;hpb=12262f1306059765d8625a6752364679c8625d31;p=platal.git diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php index 9274953..08342a0 100644 --- a/modules/xnetgrp.php +++ b/modules/xnetgrp.php @@ -29,34 +29,36 @@ class XnetGrpModule extends PLModule '%grp/asso.php' => $this->make_hook('index', AUTH_PUBLIC), '%grp/logo' => $this->make_hook('logo', AUTH_PUBLIC), '%grp/site' => $this->make_hook('site', AUTH_PUBLIC), - '%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/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'), - '%grp/trombi' => $this->make_hook('trombi', AUTH_MDP, 'groupannu'), - '%grp/geoloc' => $this->make_hook('geoloc', AUTH_MDP, 'groupannu'), + '%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/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'), + '%grp/directory/sync' => $this->make_hook('directory_sync', AUTH_MDP, 'groupadmin'), + '%grp/trombi' => $this->make_hook('trombi', AUTH_MDP, 'groupannu'), + '%grp/geoloc' => $this->make_hook('geoloc', AUTH_MDP, 'groupannu'), '%grp/subscribe' => $this->make_hook('subscribe', AUTH_MDP), - '%grp/subscribe/valid' => $this->make_hook('subscribe_valid', AUTH_MDP, 'groupadmin'), - '%grp/unsubscribe' => $this->make_hook('unsubscribe', AUTH_MDP, 'groupmember'), + '%grp/subscribe/valid' => $this->make_hook('subscribe_valid', AUTH_MDP, 'groupadmin'), + '%grp/unsubscribe' => $this->make_hook('unsubscribe', AUTH_MDP, 'groupmember'), '%grp/change_rights' => $this->make_hook('change_rights', AUTH_MDP), - '%grp/admin/annuaire' => $this->make_hook('admin_annuaire', AUTH_MDP, 'groupadmin'), - '%grp/member' => $this->make_hook('admin_member', AUTH_MDP, 'groupadmin'), - '%grp/member/new' => $this->make_hook('admin_member_new', AUTH_MDP, 'groupadmin'), - '%grp/member/new/ajax' => $this->make_hook('admin_member_new_ajax', AUTH_MDP, 'user', NO_AUTH), - '%grp/member/del' => $this->make_hook('admin_member_del', AUTH_MDP, 'groupadmin'), - - '%grp/rss' => $this->make_hook('rss', AUTH_PUBLIC, 'user', NO_HTTPS), - '%grp/announce/new' => $this->make_hook('edit_announce', AUTH_MDP, 'groupadmin'), - '%grp/announce/edit' => $this->make_hook('edit_announce', AUTH_MDP, 'groupadmin'), + '%grp/admin/annuaire' => $this->make_hook('admin_annuaire', AUTH_MDP, 'groupadmin'), + '%grp/member' => $this->make_hook('admin_member', AUTH_MDP, 'groupadmin'), + '%grp/member/new' => $this->make_hook('admin_member_new', AUTH_MDP, 'groupadmin'), + '%grp/member/new/ajax' => $this->make_hook('admin_member_new_ajax', AUTH_MDP, 'user', NO_AUTH), + '%grp/member/del' => $this->make_hook('admin_member_del', AUTH_MDP, 'groupadmin'), + '%grp/member/suggest' => $this->make_hook('admin_member_suggest', AUTH_MDP, 'groupadmin'), + + '%grp/rss' => $this->make_token_hook('rss', AUTH_PUBLIC), + '%grp/announce/new' => $this->make_hook('edit_announce', AUTH_MDP, 'groupadmin'), + '%grp/announce/edit' => $this->make_hook('edit_announce', AUTH_MDP, 'groupadmin'), '%grp/announce/photo' => $this->make_hook('photo_announce', AUTH_PUBLIC), - '%grp/admin/announces' => $this->make_hook('admin_announce', AUTH_MDP, 'groupadmin'), + '%grp/admin/announces' => $this->make_hook('admin_announce', AUTH_MDP, 'groupadmin'), ); } - function handler_index(&$page, $arg = null) + function handler_index($page, $arg = null) { global $globals, $platal; if (!is_null($arg)) { @@ -136,13 +138,13 @@ class XnetGrpModule extends PLModule $page->assign('articles', $arts); } - function handler_logo(&$page) + function handler_logo($page) { global $globals; $globals->asso()->getLogo()->send(); } - function handler_site(&$page) + function handler_site($page) { global $globals; $site = $globals->asso('site'); @@ -154,7 +156,7 @@ class XnetGrpModule extends PLModule exit; } - function handler_edit(&$page) + function handler_edit($page) { global $globals; $page->changeTpl('xnetgrp/edit.tpl'); @@ -184,40 +186,63 @@ class XnetGrpModule extends PLModule $page->trigError('Ni le nom ni le diminutif du groupe ne peuvent être vide.'); return; } + $axDate = make_datetime(Post::v('axDate')); + if (Post::t('axDate') != '') { + $axDate = make_datetime(Post::v('axDate'))->format('Y-m-d'); + } else { + $axDate = null; + } XDB::execute( "UPDATE groups SET nom={?}, diminutif={?}, cat={?}, dom={?}, descr={?}, site={?}, mail={?}, resp={?}, - forum={?}, mail_domain={?}, ax={?}, pub={?}, + forum={?}, mail_domain={?}, ax={?}, axDate = {?}, pub={?}, sub_url={?}, inscriptible={?}, unsub_url={?}, - flags={?} + flags = {?}, welcome_msg = {?} WHERE id={?}", Post::v('nom'), Post::v('diminutif'), Post::v('cat'), (Post::i('dom') == 0) ? null : Post::i('dom'), Post::v('descr'), $site, Post::v('mail'), Post::v('resp'), Post::v('forum'), Post::v('mail_domain'), - Post::has('ax'), Post::v('pub'), + Post::has('ax'), $axDate, Post::v('pub'), Post::v('sub_url'), Post::v('inscriptible'), - Post::v('unsub_url'), $flags, $globals->asso('id')); + Post::v('unsub_url'), $flags, Post::t('welcome_msg'), + $globals->asso('id')); if (Post::v('mail_domain')) { - XDB::execute('INSERT IGNORE INTO virtual_domains (domain) VALUES({?})', - Post::v('mail_domain')); + XDB::execute('INSERT IGNORE INTO email_virtual_domains (name) + VALUES ({?})', + Post::t('mail_domain')); + XDB::execute('UPDATE email_virtual_domains + SET aliasing = id + WHERE name = {?}', + Post::t('mail_domain')); } } else { XDB::execute( "UPDATE groups SET descr={?}, site={?}, mail={?}, resp={?}, forum={?}, pub= {?}, sub_url={?}, - unsub_url={?},flags={?} + unsub_url = {?}, flags = {?}, welcome_msg = {?} WHERE id={?}", Post::v('descr'), $site, Post::v('mail'), Post::v('resp'), Post::v('forum'), Post::v('pub'), Post::v('sub_url'), Post::v('unsub_url'), - $flags, $globals->asso('id')); + $flags, Post::t('welcome_msg'), + $globals->asso('id')); } + Phone::deletePhones(0, Phone::LINK_GROUP, $globals->asso('id')); + $phone = new Phone(array('link_type' => 'group', 'link_id' => $globals->asso('id'), 'id' => 0, + 'type' => 'fixed', 'display' => Post::v('phone'), 'pub' => 'public')); + $fax = new Phone(array('link_type' => 'group', 'link_id' => $globals->asso('id'), 'id' => 1, + 'type' => 'fax', 'display' => Post::v('fax'), 'pub' => 'public')); + $phone->save(); + $fax->save(); + Address::deleteAddresses(null, Address::LINK_GROUP, null, $globals->asso('id')); + $address = new Address(array('groupid' => $globals->asso('id'), 'type' => Address::LINK_GROUP, 'text' => Post::v('address'))); + $address->save(); if ($_FILES['logo']['name']) { $upload = PlUpload::get($_FILES['logo'], $globals->asso('id'), 'asso.logo', true); @@ -244,7 +269,7 @@ class XnetGrpModule extends PLModule } } - function handler_mail(&$page) + function handler_mail($page) { global $globals; @@ -281,7 +306,7 @@ class XnetGrpModule extends PLModule } } - function handler_forum(&$page, $group = null, $artid = null) + function handler_forum($page, $group = null, $artid = null) { global $globals; $page->changeTpl('xnetgrp/forum.tpl'); @@ -294,7 +319,7 @@ class XnetGrpModule extends PLModule run_banana($page, 'ForumsBanana', $get); } - function handler_annuaire(&$page, $action = null, $subaction = null) + function handler_annuaire($page, $action = null, $subaction = null) { global $globals; @@ -347,25 +372,25 @@ class XnetGrpModule extends PLModule $page->assign('only_admin', Env::b('admin')); } - function handler_trombi(&$page) + function handler_trombi($page) { pl_redirect('annuaire/trombi'); } - function handler_geoloc(&$page) + function handler_geoloc($page) { pl_redirect('annuaire/geoloc'); } - function handler_vcard(&$page, $photos = null) + function handler_vcard($page, $photos = null) { global $globals; $vcard = new VCard($photos == 'photos', 'Membre du groupe ' . $globals->asso('nom')); - $vcard->addProfiles($globals->asso()->getMembersFilter()->getProfiles()); + $vcard->addProfiles($globals->asso()->getMembersFilter()->getProfiles(null, Profile::FETCH_ALL)); $vcard->show(); } - function handler_csv(&$page, $filename = null) + function handler_csv($page, $filename = null) { global $globals; if (is_null($filename)) { @@ -377,6 +402,113 @@ class XnetGrpModule extends PLModule $page->assign('users', $users); } + function handler_directory_sync($page) + { + global $globals; + require_once 'emails.inc.php'; + + $page->changeTpl('xnetgrp/sync.tpl'); + Platal::load('lists', 'lists.inc.php'); + + if (Env::has('add_users')) { + S::assert_xsrf_token(); + + $users = array_keys(Env::v('add_users')); + $data = array(); + foreach ($users as $uid) { + $data[] = XDB::format('({?}, {?})', $globals->asso('id'), $uid); + } + XDB::rawExecute('INSERT INTO group_members (asso_id, uid) + VALUES ' . implode(',', $data)); + } + + if (Env::has('add_nonusers')) { + S::assert_xsrf_token(); + + $nonusers = array_keys(Env::v('add_nonusers')); + foreach ($nonusers as $email) { + if ($user = User::getSilent($email) || !isvalid_email($email)) { + continue; + } + + list($local_part, $domain) = explode('@', strtolower($email)); + $hruid = User::makeHrid($local_part, $domain, 'ext'); + if ($user = User::getSilent($hruid)) { + continue; + } + + $parts = explode('.', $local_part); + if (count($parts) == 1) { + $lastname = $display_name = $full_name = $directory_name = ucfirst($local_part); + $firstname = ''; + } else { + $firstname = ucfirst($parts[0]); + $lastname = ucwords(implode(' ', array_slice($parts, 1))); + $display_name = $firstname; + $full_name = $firstname . ' ' . $lastname; + $directory_name = strtoupper($lastname) . ' ' . $firstname; + } + XDB::execute('INSERT INTO accounts (hruid, display_name, full_name, directory_name, firstname, lastname, email, type, state) + VALUES ({?}, {?}, {?}, {?}, {?}, {?}, {?}, \'xnet\', \'disabled\')', + $hruid, $display_name, $full_name, $directory_name, $firstname, $lastname, $email); + $uid = XDB::insertId(); + XDB::execute('INSERT INTO group_members (asso_id, uid) + VALUES ({?}, {?})', + $globals->asso('id'), $uid); + } + } + + if (Env::has('add_users') || Env::has('add_nonusers')) { + $page->trigSuccess('Ajouts réalisés avec succès.'); + } + + $user = S::user(); + $client = new MMList($user, $globals->asso('mail_domain')); + $lists = $client->get_lists(); + $members = array(); + foreach ($lists as $list) { + $details = $client->get_members($list['list']); + $members = array_merge($members, list_extract_members($details[1])); + } + $members = array_unique($members); + $uids = array(); + $users = array(); + $nonusers = array(); + foreach ($members as $email) { + if ($user = User::getSilent($email)) { + $uids[] = $user->id(); + } else { + $nonusers[] = $email; + } + } + + $aliases = iterate_list_alias($globals->asso('mail_domain')); + foreach ($aliases as $alias) { + list($local_part, $domain) = explode('@', $alias); + $aliases_members = list_alias_members($local_part, $domain); + $users = array_merge($users, $aliases_members['users']); + $nonusers = array_merge($nonusers, $aliases_members['nonusers']); + } + foreach ($users as $user) { + $uids[] = $user->id(); + } + $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'); + sort($nonusers); + + $page->assign('users', $users); + $page->assign('nonusers', $nonusers); + } + private function removeSubscriptionRequest($uid) { global $globals; @@ -385,7 +517,7 @@ class XnetGrpModule extends PLModule $globals->asso('id'), $uid); } - private function validSubscription(User &$user) + private function validSubscription(User $user) { global $globals; $this->removeSubscriptionRequest($user->id()); @@ -401,6 +533,7 @@ class XnetGrpModule extends PLModule . "\n" . " Suite à ta demande d'adhésion à " . $globals->asso('nom') . ", j'ai le plaisir de t'annoncer que ton inscription a été validée !\n" + . (is_null($globals->asso('welcome_msg')) ? '' : "\n" . $globals->asso('welcome_msg') . "\n") . "\n" . "Bien cordialement,\n" . "-- \n" @@ -410,7 +543,7 @@ class XnetGrpModule extends PLModule } } - function handler_subscribe(&$page, $u = null) + function handler_subscribe($page, $u = null) { global $globals; $page->changeTpl('xnetgrp/inscrire.tpl'); @@ -491,9 +624,9 @@ class XnetGrpModule extends PLModule $uf = New UserFilter(New UFC_Group($globals->asso('id'), true)); $admins = $uf->iterUsers(); $admin = $admins->next(); - $to = $admin->bestalias; + $to = $admin->bestEmail(); while ($admin = $admins->next()) { - $to .= ', ' . $admin->bestalias; + $to .= ', ' . $admin->bestEmail(); } $append = "\n" @@ -525,7 +658,7 @@ class XnetGrpModule extends PLModule } } - function handler_subscribe_valid(&$page) + function handler_subscribe_valid($page) { global $globals; @@ -554,7 +687,7 @@ class XnetGrpModule extends PLModule $page->assign('valid', $it); } - function handler_change_rights(&$page) + function handler_change_rights($page) { if (Env::has('right') && (may_update() || S::suid())) { switch (Env::v('right')) { @@ -581,7 +714,7 @@ class XnetGrpModule extends PLModule http_redirect($_SERVER['HTTP_REFERER']); } - function handler_admin_annuaire(&$page) + function handler_admin_annuaire($page) { global $globals; @@ -621,7 +754,7 @@ class XnetGrpModule extends PLModule $page->assign('lists', $lists); } - function handler_admin_member_new(&$page, $email = null) + function handler_admin_member_new($page, $email = null) { global $globals; @@ -632,21 +765,27 @@ class XnetGrpModule extends PLModule } S::assert_xsrf_token(); + $suggest_account_activation = false; + + // FS#703 : $_GET is urldecoded twice, hence + // + (the data) => %2B (in the url) => + (first decoding) => ' ' (second decoding) + // Since there can be no spaces in emails, we can fix this with : + $email = str_replace(' ', '+', $email); // Finds or creates account: first cases are for users with an account. if (!User::isForeignEmailAddress($email)) { // Standard account - $user = User::get($email); + $user = User::getSilent($email); } else if (!isvalid_email($email)) { // email might not be a regular email but an alias or a hruid - $user = User::get($email); + $user = User::getSilent($email); if (!$user) { // need a valid email address $page->trigError('« ' . $email . ' Â» n\'est pas une adresse email valide.'); return; } } else if (Env::v('x') && Env::i('userid')) { - $user = User::getWithUID(Env::i('userid')); + $user = User::getSilentWithUID(Env::i('userid')); if (!$user) { $page->trigError('Utilisateur invalide.'); return; @@ -670,17 +809,26 @@ class XnetGrpModule extends PLModule } } } else { - // User is of type xnet. + // User is of type xnet. There are 3 possible cases: + // * the email is not known yet: we create a new account and + // propose to send an email to the user so he can activate + // his account, + // * the email is known but the user was not contacted in order to + // activate yet: we propose to send an email to the user so he + // can activate his account, + // * the email is known and the user was already contacted or has + // an active account: nothing to be done. list($mbox, $domain) = explode('@', strtolower($email)); $hruid = User::makeHrid($mbox, $domain, 'ext'); // User might already have an account (in another group for example). - $user = User::get($hruid); + $user = User::getSilent($hruid); // If the user has no account yet, creates new account: build names from email address. if (empty($user)) { $parts = explode('.', $mbox); if (count($parts) == 1) { - $display_name = $full_name = $directory_name = ucfirst($mbox); + $lastname = $display_name = $full_name = $directory_name = ucfirst($mbox); + $firstname = ''; } else { $firstname = ucfirst($parts[0]); $lastname = ucwords(implode(' ', array_slice($parts, 1))); @@ -688,12 +836,13 @@ class XnetGrpModule extends PLModule $full_name = "$firstname $lastname"; $directory_name = strtoupper($lastname) . " " . $firstname; } - XDB::execute('INSERT INTO accounts (hruid, display_name, full_name, directory_name, - email, type) - VALUES ({?}, {?}, {?}, {?}, {?}, \'xnet\')', - $hruid, $display_name, $full_name, $directory_name, $email); - $user = User::get($hruid); + XDB::execute('INSERT INTO accounts (hruid, display_name, full_name, directory_name, firstname, lastname, email, type, state) + VALUES ({?}, {?}, {?}, {?}, {?}, {?}, {?}, \'xnet\', \'disabled\')', + $hruid, $display_name, $full_name, $directory_name, $firstname, $lastname, $email); + $user = User::getSilent($hruid); } + + $suggest_account_activation = $this->suggest($user); } if ($user) { @@ -701,11 +850,59 @@ class XnetGrpModule extends PLModule VALUES ({?}, {?})', $user->id(), $globals->asso('id')); $this->removeSubscriptionRequest($user->id()); - pl_redirect('member/' . $user->login()); + if ($suggest_account_activation) { + pl_redirect('member/suggest/' . $user->login() . '/' . $email . '/' . $globals->asso('nom')); + } else { + pl_redirect('member/' . $user->login()); + } + } + } + + // Check if the user has a pending or active account, and thus if we should her account's activation. + private function suggest(PlUser $user) + { + $active = XDB::fetchOneCell('SELECT state = \'active\' + FROM accounts + WHERE uid = {?}', + $user->id()); + $pending = XDB::fetchOneCell('SELECT uid + FROM register_pending_xnet + WHERE uid = {?}', + $user->id()); + $requested = AccountReq::isPending($user->id()); + + if ($active || $pending || $requested) { + return false; + } + return true; + } + + function handler_admin_member_suggest($page, $hruid, $email) + { + $page->changeTpl('xnetgrp/membres-suggest.tpl'); + + // FS#703 : $_GET is urldecoded twice, hence + // + (the data) => %2B (in the url) => + (first decoding) => ' ' (second decoding) + // Since there can be no spaces in emails, we can fix this with : + $email = str_replace(' ', '+', $email); + + if (Post::has('suggest')) { + if (Post::t('suggest') == 'yes') { + $user = S::user(); + $group = Platal::globals()->asso('nom'); + $request = new AccountReq($user, $hruid, $email, $group); + $request->submit(); + $page->trigSuccessRedirect('Un email va bien être envoyé à ' . $email . ' pour l\'activation de son compte.', + $group . '/member/' . $hruid); + } else { + pl_redirect('member/' . $hruid); + } } + $page->assign('email', $email); + $page->assign('hruid', $hruid); } - function handler_admin_member_new_ajax(&$page) + function handler_admin_member_new_ajax($page) { pl_content_headers("text/html"); $page->changeTpl('xnetgrp/membres-new-search.tpl', NO_SKIN); @@ -738,7 +935,7 @@ class XnetGrpModule extends PLModule $page->assign('users', $users); } - function unsubscribe(PlUser &$user) + function unsubscribe(PlUser $user) { global $globals; XDB::execute("DELETE FROM group_members @@ -784,24 +981,23 @@ class XnetGrpModule extends PLModule } } - XDB::execute("DELETE FROM virtual_redirect - USING virtual_redirect - INNER JOIN virtual USING(vid) - WHERE redirect={?} AND alias LIKE {?}", - $user->forlifeEmail(), '%@'.$domain); + XDB::execute('DELETE v + FROM email_virtual AS v + INNER JOIN email_virtual_domains AS d ON (v.domain = d.id) + WHERE v.redirect = {?} AND d.name = {?}', + $user->forlifeEmail(), $domain); return !$warning; } - function handler_unsubscribe(&$page) + function handler_unsubscribe($page) { $page->changeTpl('xnetgrp/membres-del.tpl'); $user = S::user(); - $uid = S::user()->id(); - if (empty($uid)) { + if (empty($user)) { return PL_NOT_FOUND; } $page->assign('self', true); - $page->assign('user', $uid); + $page->assign('user', $user); if (!Post::has('confirm')) { return; @@ -809,21 +1005,36 @@ class XnetGrpModule extends PLModule S::assert_xsrf_token(); } + $hasSingleGroup = ($user->groupCount() == 1); + if ($this->unsubscribe($user)) { - $page->trigSuccess('Vous avez été désinscrit du groupe avec succès.'); + $page->trigSuccess('Tu as été désinscrit du groupe avec succès.'); } else { - $page->trigWarning('Vous avez été désinscrit du groupe, mais des erreurs se sont produites lors des désinscriptions des alias et des listes de diffusion.'); + $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.'); + } + + // If user is of type xnet account and this was her last group, disable the account. + if ($user->type == 'xnet' && $hasSingleGroup) { + $user->clear(true); } $page->assign('is_member', is_member(true)); } - function handler_admin_member_del(&$page, $user = null) + function handler_admin_member_del($page, $user = null) { $page->changeTpl('xnetgrp/membres-del.tpl'); $user = User::getSilent($user); if (empty($user)) { return PL_NOT_FOUND; } + + global $globals; + + if (!$user->inGroup($globals->asso('id'))) { + pl_redirect('annuaire'); + } + + $page->assign('self', false); $page->assign('user', $user); if (!Post::has('confirm')) { @@ -832,14 +1043,21 @@ class XnetGrpModule extends PLModule S::assert_xsrf_token(); } + $hasSingleGroup = ($user->groupCount() == 1); + if ($this->unsubscribe($user)) { $page->trigSuccess("{$user->fullName()} a été désinscrit du groupe !"); } else { $page->trigWarning("{$user->fullName()} a été désinscrit du groupe, mais des erreurs subsistent !"); } + + // If user is of type xnet account and this was her last group, disable the account. + if ($user->type == 'xnet' && $hasSingleGroup) { + $user->clear(true); + } } - private function changeLogin(PlPage &$page, PlUser &$user, MMList &$mmlist, $login) + private function changeLogin(PlPage $page, PlUser $user, $login) { // Search the user's uid. $xuser = User::getSilent($login); @@ -865,25 +1083,30 @@ class XnetGrpModule extends PLModule return $user->login(); } - function handler_admin_member(&$page, $user) + function handler_admin_member($page, $user) { global $globals; - $page->changeTpl('xnetgrp/membres-edit.tpl'); - $user = User::getSilent($user); if (empty($user)) { return PL_NOT_FOUND; } + if (!$user->inGroup($globals->asso('id'))) { + pl_redirect('annuaire'); + } + + $page->changeTpl('xnetgrp/membres-edit.tpl'); + $mmlist = new MMList(S::user(), $globals->asso('mail_domain')); if (Post::has('change')) { + require_once 'emails.inc.php'; S::assert_xsrf_token(); // Convert user status to X if (!Post::blank('login_X')) { - $forlife = $this->changeLogin($page, $user, $mmlist, Post::t('login_X')); + $forlife = $this->changeLogin($page, $user, Post::t('login_X')); if ($forlife) { pl_redirect('member/' . $forlife); } @@ -892,12 +1115,22 @@ class XnetGrpModule extends PLModule // Update user info $email_changed = (!$user->profile() && strtolower($user->forlifeEmail()) != strtolower(Post::v('email'))); $from_email = $user->forlifeEmail(); - if (!$user->profile()) { + if ($user->type == 'virtual' || ($user->type == 'xnet' && !$user->perms)) { + $lastname = Post::s('lastname'); + if (Post::s('type') != 'virtual') { + $firstname = Post::s('firstname'); + $full_name = $firstname . ' ' . $lastname; + $directory_name = mb_strtoupper($lastname) . ' ' . $firstname; + } else { + $firstname = ''; + $full_name = $lastname; + $directory_name = mb_strtoupper($lastname); + } XDB::query('UPDATE accounts SET full_name = {?}, directory_name = {?}, display_name = {?}, - sex = {?}, email = {?}, type = {?} + firstname = {?}, lastname = {?}, sex = {?}, email = {?}, type = {?} WHERE uid = {?}', - Post::t('full_name'), Post::t('directory_name'), Post::t('display_name'), + $full_name, $directory_name, Post::t('display_name'), $firstname, $lastname, (Post::t('sex') == 'male') ? 'male' : 'female', Post::t('email'), (Post::t('type') == 'xnet') ? 'xnet' : 'virtual', $user->id()); } else if (!$user->perms) { @@ -906,18 +1139,30 @@ class XnetGrpModule extends PLModule WHERE uid = {?}', Post::t('email'), $user->id()); } + if (require_email_update($user, Post::t('email'))) { + $listClient = new MMList(S::user()); + $listClient->change_user_email($user->forlifeEmail(), Post::t('email')); + update_alias_user($user->forlifeEmail(), Post::t('email')); + } if (XDB::affectedRows()) { - $page->trigSuccess('Données de l\'utilisateur mise à jour.'); + $page->trigSuccess('Données de l\'utilisateur mises à jour.'); + } + + if (($user->type == 'xnet' && !$user->perms) && Post::b('suggest')) { + $request = new AccountReq(S::user(), $user->hruid, Post::t('email'), $globals->asso('nom')); + $request->submit(); + $page->trigSuccess('Le compte va bientôt être activé.'); } // Update group params for user $perms = Post::v('group_perms'); $comm = Post::t('comm'); - if ($user->group_perms != $perms || $user->group_comm != $comm) { + $position = (Post::t('group_position') == '') ? null : Post::v('group_position'); + if ($user->group_perms != $perms || $user->group_comm != $comm || $user->group_position != $position) { XDB::query('UPDATE group_members - SET perms = {?}, comm = {?} + SET perms = {?}, comm = {?}, position = {?} WHERE uid = {?} AND asso_id = {?}', - ($perms == 'admin') ? 'admin' : 'membre', $comm, + ($perms == 'admin') ? 'admin' : 'membre', $comm, $position, $user->id(), $globals->asso('id')); if (XDB::affectedRows()) { if ($perms != $user->group_perms) { @@ -926,6 +1171,9 @@ class XnetGrpModule extends PLModule if ($comm != $user->group_comm) { $page->trigSuccess('Commentaire mis à jour.'); } + if ($position != $user->group_position) { + $page->trigSuccess('Poste mis à jour.'); + } } } @@ -961,47 +1209,45 @@ class XnetGrpModule extends PLModule // Change subscriptioin to aliases foreach (Env::v('ml3', array()) as $ml => $state) { + require_once 'emails.inc.php'; $ask = !empty($_REQUEST['ml4'][$ml]); + list($local_part, ) = explode('@', $ml); if($state == $ask) { if ($state && $email_changed) { - XDB::query("UPDATE virtual_redirect AS vr, virtual AS v - SET vr.redirect = {?} - WHERE vr.vid = v.vid AND v.alias = {?} AND vr.redirect = {?}", - $user->forlifeEmail(), $ml, $from_email); + update_list_alias($user->id(), $from_email, $local_part, $globals->asso('mail_domain')); $page->trigSuccess("L'abonnement de {$user->fullName()} à $ml a été mis à jour."); } } else if($ask) { - XDB::query("INSERT INTO virtual_redirect (vid,redirect) - SELECT vid,{?} FROM virtual WHERE alias={?}", - $user->forlifeEmail(), $ml); + add_to_list_alias($user->id(), $local_part, $globals->asso('mail_domain')); $page->trigSuccess("{$user->fullName()} a été abonné à $ml."); } else { - XDB::query("DELETE FROM virtual_redirect - USING virtual_redirect - INNER JOIN virtual USING(vid) - WHERE redirect={?} AND alias={?}", - $from_email, $ml); + delete_from_list_alias($user->id(), $local_part, $globals->asso('mail_domain')); $page->trigSuccess("{$user->fullName()} a été désabonné de $ml."); } } } + $res = XDB::rawFetchAllAssoc('SHOW COLUMNS FROM group_members LIKE \'position\''); + $positions = str_replace(array('enum(', ')', '\''), '', $res[0]['Type']); + $page->assign('user', $user); + $page->assign('suggest', $this->suggest($user)); $page->assign('listes', $mmlist->get_lists($user->forlifeEmail())); - $page->assign('alias', $user->emailAliases($globals->asso('mail_domain'), 'user', true)); + $page->assign('alias', $user->emailGroupAliases($globals->asso('mail_domain'))); + $page->assign('positions', explode(',', $positions)); } - function handler_rss(&$page, $user = null, $hash = null) + function handler_rss(PlPage $page, PlUser $user) { global $globals; $page->assign('asso', $globals->asso()); $this->load('feed.inc.php'); $feed = new XnetGrpEventFeed(); - return $feed->run($page, $user, $hash, false); + return $feed->run($page, $user, false); } - private function upload_image(PlPage &$page, PlUpload &$upload) + private function upload_image(PlPage $page, PlUpload $upload) { if (@!$_FILES['image']['tmp_name'] && !Env::v('image_url')) { return true; @@ -1020,7 +1266,7 @@ class XnetGrpModule extends PLModule return true; } - function handler_photo_announce(&$page, $eid = null) { + function handler_photo_announce($page, $eid = null) { if ($eid) { $res = XDB::query('SELECT * FROM group_announces_photo @@ -1045,7 +1291,7 @@ class XnetGrpModule extends PLModule exit; } - function handler_edit_announce(&$page, $aid = null) + function handler_edit_announce($page, $aid = null) { global $globals, $platal; $page->changeTpl('xnetgrp/announce-edit.tpl'); @@ -1125,13 +1371,13 @@ class XnetGrpModule extends PLModule if (!empty($art['contact_html'])) { $fulltext .= "\n\n'''Contacts :'''\\\\\n" . $art['contact_html']; } - $post = null;/* + $post = null; if ($globals->asso('forum')) { require_once 'banana/forum.inc.php'; $banana = new ForumsBanana(S::user()); $post = $banana->post($globals->asso('forum'), null, $art['titre'], MiniWiki::wikiToText($fulltext, false, 0, 80)); - }*/ + } XDB::query('INSERT INTO group_announces (uid, asso_id, create_date, titre, texte, contacts, expiration, promo_min, promo_max, flags, post_id) VALUES ({?}, {?}, NOW(), {?}, {?}, {?}, {?}, {?}, {?}, {?}, {?})', @@ -1210,7 +1456,7 @@ class XnetGrpModule extends PLModule $page->assign_by_ref('upload', $upload); } - function handler_admin_announce(&$page) + function handler_admin_announce($page) { global $globals; $page->changeTpl('xnetgrp/announce-admin.tpl');