X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=modules%2Fcarnet.php;h=74dd23fc64996a8177b41e0646b76c34cb305841;hb=6eeb600e8d56aa77188d2618928109a876afb289;hp=194d866de756027df1bbdc75a983ac53f414ab33;hpb=e6bf921624e50f7abdd9a1862d0b7ef4f1053090;p=platal.git diff --git a/modules/carnet.php b/modules/carnet.php index 194d866..74dd23f 100644 --- a/modules/carnet.php +++ b/modules/carnet.php @@ -56,7 +56,7 @@ class CarnetModule extends PLModule function handler_index(&$page) { $page->changeTpl('carnet/index.tpl'); - $page->assign('xorg_title','Polytechnique.org - Mon carnet'); + $page->setTitle('Polytechnique.org - Mon carnet'); $this->_add_rss_link($page); } @@ -65,8 +65,8 @@ class CarnetModule extends PLModule $page->changeTpl('carnet/panel.tpl'); if (Get::has('read')) { - $_SESSION['watch_last'] = Get::v('read'); - update_NbNotifs(); + S::set('watch_last', Get::v('read')); + Platal::session()->updateNbNotifs(); pl_redirect('carnet/panel'); } @@ -85,7 +85,7 @@ class CarnetModule extends PLModule if(preg_match('!^ *(\d{4}) *$!', $arg, $matches)) { $p = intval($matches[1]); if($p<1900 || $p>2100) { - $page->trig("la promo entrée est invalide"); + $page->trigError("la promo entrée est invalide"); } else { if ($action == 'add_promo') { $watch->_promos->add($p); @@ -97,9 +97,9 @@ class CarnetModule extends PLModule $p1 = intval($matches[1]); $p2 = intval($matches[2]); if($p1<1900 || $p1>2100) { - $page->trig('la première promo de la plage entrée est invalide'); + $page->trigError('la première promo de la plage entrée est invalide'); } elseif($p2<1900 || $p2>2100) { - $page->trig('la seconde promo de la plage entrée est invalide'); + $page->trigError('la seconde promo de la plage entrée est invalide'); } else { if ($action == 'add_promo') { $watch->_promos->addRange($p1, $p2); @@ -108,7 +108,7 @@ class CarnetModule extends PLModule } } } else { - $page->trig("La promo (ou la plage de promo) entrée est dans un format incorrect."); + $page->trigError("La promo (ou la plage de promo) entrée est dans un format incorrect."); } } @@ -127,9 +127,8 @@ class CarnetModule extends PLModule $promo_sortie = $res->fetchOneCell(); $page->assign('promo_sortie', $promo_sortie); - if ($action && !S::has_xsrf_token()) { - $page->trig("La mise à jour des notifications a échouée, merci de réessayer."); - $action = false; + if ($action) { + S::assert_xsrf_token(); } switch ($action) { case 'add_promo': @@ -146,24 +145,21 @@ class CarnetModule extends PLModule break; } - if (Env::has('subs') && S::has_xsrf_token()) { + if (Env::has('subs')) { + S::assert_xsrf_token(); $watch->_subs->update('sub'); - } elseif (Env::has('subs')) { - $page->trig("La mise à jour des notifications a échouée, merci de réessayer."); } - if (Env::has('flags_contacts') && S::has_xsrf_token()) { + if (Env::has('flags_contacts')) { + S::assert_xsrf_token(); $watch->watch_contacts = Env::b('contacts'); $watch->saveFlags(); - } elseif (Env::has('flags_contacts')) { - $page->trig("La mise à jour des notifications a échouée, merci de réessayer."); } - if (Env::has('flags_mail') && S::has_xsrf_token()) { + if (Env::has('flags_mail')) { + S::assert_xsrf_token(); $watch->watch_mail = Env::b('mail'); $watch->saveFlags(); - } elseif (Env::has('flags_mail')) { - $page->trig("La mise à jour des notifications a échouée, merci de réessayer."); } $page->assign_by_ref('watch', $watch); @@ -199,14 +195,14 @@ class CarnetModule extends PLModule } function searchErrorHandler($explain) { - global $page; - $page->trig($explain); + $page =& Platal::page(); + $page->trigError($explain); $this->handler_contacts($page); } function handler_contacts(&$page, $action = null, $subaction = null, $ssaction = null) { - $page->assign('xorg_title','Polytechnique.org - Mes contacts'); + $page->setTitle('Polytechnique.org - Mes contacts'); $this->_add_rss_link($page); $uid = S::v('uid'); @@ -214,15 +210,17 @@ class CarnetModule extends PLModule // For XSRF protection, checks both the normal xsrf token, and the special RSS token. // It allows direct linking to contact adding in the RSS feed. - if (Env::v('action') && (S::has_xsrf_token() || Env::v('token') === S::v('core_rss_hash'))) { - switch (Env::v('action')) { - case 'retirer': + if (Env::v('action') && Env::v('token') !== S::v('core_rss_hash')) { + S::assert_xsrf_token(); + } + switch (Env::v('action')) { + case 'retirer': if (is_numeric($user)) { if (XDB::execute('DELETE FROM contacts WHERE uid = {?} AND contact = {?}', $uid, $user)) { - $page->trig("Contact retiré !"); + $page->trigSuccess("Contact retiré !"); } } else { if (XDB::execute( @@ -231,12 +229,12 @@ class CarnetModule extends PLModule INNER JOIN aliases AS a ON (c.contact=a.id and a.type!="homonyme") WHERE c.uid = {?} AND a.alias={?}', $uid, $user)) { - $page->trig("Contact retiré !"); + $page->trigSuccess("Contact retiré !"); } } break; - case 'ajouter': + case 'ajouter': require_once('user.func.inc.php'); if (($login = get_user_login($user)) !== false) { if (XDB::execute( @@ -245,14 +243,11 @@ class CarnetModule extends PLModule FROM aliases WHERE alias = {?}', $uid, $login)) { - $page->trig('Contact ajouté !'); + $page->trigSuccess('Contact ajouté !'); } else { - $page->trig('Contact déjà dans la liste !'); + $page->trigWarning('Contact déjà dans la liste !'); } } - } - } elseif (Env::v('action')) { - $page->trig("La modification du contact a échouée, merci de réessayer."); } $search = false; @@ -272,7 +267,7 @@ class CarnetModule extends PLModule $base = 'carnet/contacts'; $view = new UserSet("INNER JOIN contacts AS c2 ON (u.user_id = c2.contact)", " c2.uid = $uid "); } - $view->addMod('minifiche', 'Mini-Fiches', true); + $view->addMod('minifiche', 'Mini-fiches', true); $view->addMod('trombi', 'Trombinoscope', false, array('with_admin' => false, 'with_promo' => true)); $view->addMod('geoloc', 'Planisphère', false, array('with_annu' => 'carnet/contacts/search')); $view->apply($base, $page, $action, $subaction); @@ -286,7 +281,7 @@ class CarnetModule extends PLModule require_once dirname(__FILE__).'/carnet/contacts.pdf.inc.php'; require_once 'user.func.inc.php'; - session_write_close(); + Platal::session()->close(); $sql = "SELECT a.alias FROM aliases AS a @@ -334,7 +329,6 @@ class CarnetModule extends PLModule if (!$uid) { $uid = S::i('uid'); } else if ($uid != S::i('uid')) { - require_once 'xorg.misc.inc.php'; send_warning_email("Récupération d\'un autre utilisateur ($uid)"); } } else if (!$uid) {