pending commit, finished during MQ/S download ...
[platal.git] / modules / profile.php
index 5abe009..fc92dba 100644 (file)
@@ -29,10 +29,13 @@ class ProfileModule extends PLModule
 
             'fiche.php'        => $this->make_hook('fiche',      AUTH_PUBLIC),
             'profile'          => $this->make_hook('profile',    AUTH_PUBLIC),
+            'profile/edit'     => $this->make_hook('p_edit',     AUTH_MDP),
             'profile/orange'   => $this->make_hook('p_orange',   AUTH_MDP),
-            'profile/referent' => $this->make_hook('p_referent', AUTH_MDP),
             'profile/usage'    => $this->make_hook('p_usage',    AUTH_MDP),
 
+            'referent'         => $this->make_hook('referent',   AUTH_COOKIE),
+            'referent/search'  => $this->make_hook('ref_search', AUTH_COOKIE),
+
             'trombi'  => $this->make_hook('trombi', AUTH_COOKIE),
 
             'vcard'   => $this->make_hook('vcard',  AUTH_COOKIE),
@@ -48,24 +51,23 @@ class ProfileModule extends PLModule
 
     function _trombi_getlist($offset, $limit)
     {
-        global $globals;
-
         $where  = ( $this->promo > 0 ? "WHERE promo='{$this->promo}'" : "" );
 
-        $res = $globals->xdb->query(
+        $res = XDB::query(
                 "SELECT  COUNT(*)
                    FROM  auth_user_md5 AS u
              RIGHT JOIN  photo         AS p ON u.user_id=p.uid
              $where");
         $pnb = $res->fetchOneCell();
 
-        $res = $globals->xdb->query(
-                "SELECT  promo,user_id,a.alias AS forlife,IF(nom_usage='', nom, nom_usage) AS nom,prenom
+        $res = XDB::query(
+                "SELECT  promo, user_id, a.alias AS forlife,
+                         IF (nom_usage='', nom, nom_usage) AS nom, prenom
                    FROM  photo         AS p
              INNER JOIN  auth_user_md5 AS u ON u.user_id=p.uid
              INNER JOIN  aliases       AS a ON ( u.user_id=a.id AND a.type='a_vie' )
                   $where
-               ORDER BY  promo,nom,prenom LIMIT {?}, {?}", $offset*$limit, $limit);
+               ORDER BY  promo, nom, prenom LIMIT {?}, {?}", $offset*$limit, $limit);
 
         return array($pnb, $res->fetchAllAssoc());
     }
@@ -76,25 +78,24 @@ class ProfileModule extends PLModule
             return PL_NOT_FOUND;
         }
 
-        global $globals;
-
-        $res = $globals->xdb->query("SELECT id, pub FROM aliases
+        $res = XDB::query("SELECT id, pub FROM aliases
                                   LEFT JOIN photo ON(id = uid)
                                       WHERE alias = {?}", $x);
         list($uid, $photo_pub) = $res->fetchOneRow();
 
-        if ($req && logged()) {
+        if ($req && S::logged()) {
             include 'validations.inc.php';
             $myphoto = PhotoReq::get_request($uid);
             Header('Content-type: image/'.$myphoto->mimetype);
             echo $myphoto->data;
         } else {
-            $res = $globals->xdb->query(
+            $res = XDB::query(
                     "SELECT  attachmime, attach
                        FROM  photo
                       WHERE  uid={?}", $uid);
 
-            if ((list($type,$data) = $res->fetchOneRow()) && ($photo_pub == 'public' || logged())) {
+            if ((list($type, $data) = $res->fetchOneRow())
+            &&  ($photo_pub == 'public' || S::logged())) {
                 Header("Content-type: image/$type");
                 echo $data;
             } else {
@@ -107,52 +108,48 @@ class ProfileModule extends PLModule
 
     function handler_photo_change(&$page)
     {
-        global $globals;
-
         $page->changeTpl('trombino.tpl');
 
         require_once('validations.inc.php');
 
-        $trombi_x = '/home/web/trombino/photos'.Session::get('promo')
-                    .'/'.Session::get('forlife').'.jpg';
+        $trombi_x = '/home/web/trombino/photos'.S::v('promo')
+                    .'/'.S::v('forlife').'.jpg';
 
         if (Env::has('upload')) {
             $file = isset($_FILES['userfile']['tmp_name'])
                     ? $_FILES['userfile']['tmp_name']
                     : Env::get('photo');
             if ($data = file_get_contents($file)) {
-                if ($myphoto = new PhotoReq(Session::getInt('uid'), $data)) {
+                if ($myphoto = new PhotoReq(S::v('uid'), $data)) {
                     $myphoto->submit();
                 }
             } else {
                 $page->trig('Fichier inexistant ou vide');
             }
         } elseif (Env::has('trombi')) {
-            $myphoto = new PhotoReq(Session::getInt('uid'),
+            $myphoto = new PhotoReq(S::v('uid'),
                                     file_get_contents($trombi_x));
             if ($myphoto) {
                 $myphoto->commit();
                 $myphoto->clean();
             }
         } elseif (Env::get('suppr')) {
-            $globals->xdb->execute('DELETE FROM photo WHERE uid = {?}',
-                                   Session::getInt('uid'));
-            $globals->xdb->execute('DELETE FROM requests
+            XDB::execute('DELETE FROM photo WHERE uid = {?}',
+                                   S::v('uid'));
+            XDB::execute('DELETE FROM requests
                                      WHERE user_id = {?} AND type="photo"',
-                                   Session::getInt('uid'));
+                                   S::v('uid'));
         } elseif (Env::get('cancel')) {
-            $sql = $globals->xdb->query('DELETE FROM requests 
+            $sql = XDB::query('DELETE FROM requests 
                                         WHERE user_id={?} AND type="photo"',
-                                        Session::getInt('uid'));
+                                        S::v('uid'));
         }
 
-        $sql = $globals->xdb->query('SELECT COUNT(*) FROM requests
+        $sql = XDB::query('SELECT COUNT(*) FROM requests
                                       WHERE user_id={?} AND type="photo"',
-                                    Session::getInt('uid'));
+                                    S::v('uid'));
         $page->assign('submited', $sql->fetchOneCell());
         $page->assign('has_trombi_x', file_exists($trombi_x));
-
-        return PL_OK;
     }
 
     function handler_profile(&$page, $x = null)
@@ -168,11 +165,11 @@ class ProfileModule extends PLModule
         $page->assign('simple', true);
 
         $view = 'private';
-        if (!logged() || Env::get('view') == 'public') $view = 'public';
-        if (logged() && Env::get('view') == 'ax')      $view = 'ax';
+        if (!S::logged() || Env::get('view') == 'public') $view = 'public';
+        if (S::logged() && Env::get('view') == 'ax')      $view = 'ax';
 
         if (is_numeric($x)) {
-            $res = $globals->xdb->query(
+            $res = XDB::query(
                     "SELECT  alias 
                        FROM  aliases       AS a
                  INNER JOIN  auth_user_md5 AS u ON (a.id=u.user_id AND a.type='a_vie')
@@ -187,7 +184,7 @@ class ProfileModule extends PLModule
         }
 
         $new   = Env::get('modif') == 'new';
-        $user  = get_user_details($login, Session::getInt('uid'), $view);
+        $user  = get_user_details($login, S::v('uid'), $view);
         $title = $user['prenom'] . ' ' . empty($user['nom_usage']) ? $user['nom'] : $user['nom_usage'];
         $page->assign('xorg_title', $title);
 
@@ -195,20 +192,20 @@ class ProfileModule extends PLModule
 
         $photo = $globals->baseurl.'/photo/'.$user['forlife'].($new ? '/req' : '');
 
-        if(!isset($user['y']) and !isset($user['x'])) {
+        if (!isset($user['y']) and !isset($user['x'])) {
             list($user['x'], $user['y']) = getimagesize("images/none.png");
         }
-        if(!isset($user['y']) or $user['y'] < 1) $user['y']=1;
-        if(!isset($user['x']) or $user['x'] < 1) $user['x']=1;
-        if($user['x'] > 240){
+        if (!isset($user['y']) or $user['y'] < 1) $user['y']=1;
+        if (!isset($user['x']) or $user['x'] < 1) $user['x']=1;
+        if ($user['x'] > 240) {
             $user['y'] = (integer)($user['y']*240/$user['x']);
             $user['x'] = 240;
         }
-        if($user['y'] > 300){
+        if ($user['y'] > 300) {
             $user['x'] = (integer)($user['x']*300/$user['y']);
             $user['y'] = 300;
         }
-        if($user['x'] < 160){
+        if ($user['x'] < 160) {
             $user['y'] = (integer)($user['y']*160/$user['x']);
             $user['x'] = 160;
         }
@@ -221,43 +218,152 @@ class ProfileModule extends PLModule
         $page->assign_by_ref('x', $user);
         $page->assign('photo_url', $photo);
         // alias virtual
-        $res = $globals->xdb->query(
+        $res = XDB::query(
                 "SELECT alias
                    FROM virtual
              INNER JOIN virtual_redirect USING(vid)
              INNER JOIN auth_user_quick  ON ( user_id = {?} AND emails_alias_pub = 'public' )
                   WHERE ( redirect={?} OR redirect={?} )
                         AND alias LIKE '%@{$globals->mail->alias_dom}'",
-                Session::getInt('uid'),
+                S::v('uid'),
                 $user['forlife'].'@'.$globals->mail->domain,
                 $user['forlife'].'@'.$globals->mail->domain2);
         $page->assign('virtualalias', $res->fetchOneCell());
 
         $page->addJsLink('javascript/close_on_esc.js');
-
-        return PL_OK;
     }
 
-    function handler_p_orange(&$page)
+    function handler_p_edit(&$page, $opened_tab = 'general')
     {
         global $globals;
 
+        $page->changeTpl('profil.tpl');
+
+        $page->addCssLink('css/profil.css');
+        $page->assign('xorg_title', 'Polytechnique.org - Mon Profil');
+
+        require_once 'tabs.inc.php';
+        require_once 'profil.func.inc.php';
+        require_once 'synchro_ax.inc.php';
+
+        if (Post::has('register_from_ax_question')) {
+            XDB::query('UPDATE auth_user_quick
+                                     SET profile_from_ax = 1
+                                   WHERE user_id = {?}',
+                                 S::v('uid'));
+        }
+
+        if (is_ax_key_missing()) {
+            $page->assign('no_private_key', true);
+        }
+
+        if (Env::get('synchro_ax') == 'confirm' && !is_ax_key_missing()) {
+            ax_synchronize(S::v('bestalias'), S::v('uid'));
+            $page->trig('Ton profil a été synchronisé avec celui du site polytechniciens.com');
+        }
+
+        // pour tous les tabs, la date de naissance pour verifier
+        // quelle est bien rentree et la date.
+        $res = XDB::query(
+                "SELECT  naissance, DATE_FORMAT(date, '%d.%m.%Y')
+                   FROM  auth_user_md5
+                  WHERE  user_id={?}", S::v('uid'));
+        list($naissance, $date_modif_profil) = $res->fetchOneRow();
+
+        // lorsqu'on n'a pas la date de naissance en base de données
+        if (!$naissance)  {
+            // la date de naissance n'existait pas et vient d'être soumise dans la variable
+            if (Env::has('birth')) {
+                //en cas d'erreur :
+                if (!ereg('[0-3][0-9][0-1][0-9][1][9]([0-9]{2})', Env::get('birth'))) {
+                    $page->assign('etat_naissance', 'query');
+                    $page->trig('Date de naissance incorrecte ou incohérente.');
+                    return;
+                }
+
+                //sinon
+                $birth = sprintf("%s-%s-%s", substr(Env::get('birth'), 4, 4),
+                                 substr(Env::get('birth'), 2, 2),
+                                 substr(Env::get('birth'), 0, 2));
+                XDB::execute("UPDATE auth_user_md5
+                                           SET naissance={?}
+                                         WHERE user_id={?}", $birth,
+                                       S::v('uid'));
+                $page->assign('etat_naissance', 'ok');
+                return;
+            }
+
+            $page->assign('etat_naissance', 'query');
+            return; // on affiche le formulaire pour naissance
+        }
+
+        //doit-on faire un update ?
+        if (Env::has('modifier') || Env::has('suivant')) {
+            require_once "profil/get_{$opened_tab}.inc.php";
+            require_once "profil/verif_{$opened_tab}.inc.php";
+
+            if($page->nb_errs()) {
+                require_once "profil/assign_{$opened_tab}.inc.php";
+                $page->assign('onglet', $opened_tab);
+                $page->assign('onglet_tpl', "profil/$opened_tab.tpl");
+                return;
+            }
+
+            $date=date("Y-m-j");//nouvelle date de mise a jour
+
+            //On sauvegarde l'uid pour l'AX
+            /* on sauvegarde les changements dans user_changes :
+            * on a juste besoin d'insérer le user_id de la personne dans la table
+            */
+            XDB::execute('REPLACE INTO user_changes SET user_id={?}',
+                                   S::v('uid'));
+
+            if (!S::has('suid')) {
+                require_once 'notifs.inc.php';
+                register_watch_op(S::v('uid'), WATCH_FICHE);
+            }
+
+            // mise a jour des champs relatifs au tab ouvert
+            require_once "profil/update_{$opened_tab}.inc.php";
+
+            $log =& S::v('log');
+            $log->log('profil', $opened_tab);
+            $page->assign('etat_update', 'ok');
+        }
+
+        if (Env::has('suivant')) {
+            redirect($globals->baseurl . '/profile/edit/' .
+                     get_next_tab($opened_tab));
+        }
+
+        require_once "profil/get_{$opened_tab}.inc.php";
+        require_once "profil/verif_{$opened_tab}.inc.php";
+        require_once "profil/assign_{$opened_tab}.inc.php";
+
+        $page->assign('onglet', $opened_tab);
+        $page->assign('onglet_tpl', "profil/$opened_tab.tpl");
+
+        return;
+    }
+
+    function handler_p_orange(&$page)
+    {
         $page->changeTpl('orange.tpl');
 
         require_once 'validations.inc.php';
         require_once 'xorg.misc.inc.php';
 
-        $res = $globals->xdb->query(
-                "SELECT  u.promo,u.promo_sortie
+        $res = XDB::query(
+                "SELECT  u.promo, u.promo_sortie
                    FROM  auth_user_md5  AS u
-                  WHERE  user_id={?}", Session::getInt('uid'));
+                  WHERE  user_id={?}", S::v('uid'));
 
-        list($promo,$promo_sortie_old) = $res->fetchOneRow();
+        list($promo, $promo_sortie_old) = $res->fetchOneRow();
         $page->assign('promo_sortie_old', $promo_sortie_old);
         $page->assign('promo',  $promo);
 
         if (!Env::has('promo_sortie')) {
-            return PL_OK;
+            return;
         }
 
         $promo_sortie = Env::getInt('promo_sortie');
@@ -272,9 +378,9 @@ class ProfileModule extends PLModule
             $page->trig('Tu appartiens déjà à la promotion correspondante à cette année de sortie.');
         }
         elseif ($promo_sortie == $promo + 3) {
-            $globals->xdb->execute(
+            XDB::execute(
                 "UPDATE  auth_user_md5 set promo_sortie={?} 
-                  WHERE  user_id={?}",$promo_sortie,Session::getInt('uid'));
+                  WHERE  user_id={?}", $promo_sortie, S::v('uid'));
                 $page->trig('Ton statut "orange" a été supprimé.');
                 $page->assign('promo_sortie_old', $promo_sortie);
         }
@@ -282,20 +388,16 @@ class ProfileModule extends PLModule
             $page->assign('promo_sortie', $promo_sortie);
 
             if (Env::has('submit')) {
-                $myorange = new OrangeReq(Session::getInt('uid'),
+                $myorange = new OrangeReq(S::v('uid'),
                                           $promo_sortie);
                 $myorange->submit();
                 $page->assign('myorange', $myorange);
             }
         }
-
-        return PL_OK;
     }
 
-    function handler_p_referent(&$page, $x = null)
+    function handler_referent(&$page, $x = null)
     {
-        global $globals;
-
         require_once 'user.func.inc.php';
 
         if (is_null($x)) {
@@ -305,10 +407,11 @@ class ProfileModule extends PLModule
         $page->changeTpl('fiche_referent.tpl');
         $page->assign('simple', true);
 
-        $res = $globals->xdb->query(
+        $res = XDB::query(
                 "SELECT  prenom, nom, user_id, promo, cv, a.alias AS bestalias
                   FROM  auth_user_md5 AS u
-            INNER JOIN  aliases       AS a ON (u.user_id=a.id AND FIND_IN_SET('bestalias',a.flags))
+            INNER JOIN  aliases       AS a ON (u.user_id=a.id
+                                               AND FIND_IN_SET('bestalias', a.flags))
             INNER JOIN  aliases       AS a1 ON (u.user_id=a1.id
                                                 AND a1.alias = {?}
                                                 AND a1.type!='homonyme')", $x);
@@ -329,12 +432,12 @@ class ProfileModule extends PLModule
         /////  recuperations infos referent
 
         //expertise
-        $res = $globals->xdb->query("SELECT expertise FROM mentor WHERE uid = {?}", $user_id);
+        $res = XDB::query("SELECT expertise FROM mentor WHERE uid = {?}", $user_id);
         $page->assign('expertise', $res->fetchOneCell());
 
         //secteurs
         $secteurs = $ss_secteurs = Array();
-        $res = $globals->xdb->iterRow(
+        $res = XDB::iterRow(
                 "SELECT  s.label, ss.label
                    FROM  mentor_secteurs AS m
               LEFT JOIN  emploi_secteur AS s ON(m.secteur = s.id)
@@ -348,7 +451,7 @@ class ProfileModule extends PLModule
         $page->assign_by_ref('ss_secteurs', $ss_secteurs);
 
         //pays
-        $res = $globals->xdb->query(
+        $res = XDB::query(
                 "SELECT  gp.pays
                    FROM  mentor_pays AS m
               LEFT JOIN  geoloc_pays AS gp ON(m.pid = gp.a2)
@@ -356,25 +459,135 @@ class ProfileModule extends PLModule
         $page->assign('pays', $res->fetchColumn());
 
         $page->addJsLink('javascript/close_on_esc.js');
-        return PL_OK;
     }
 
-    function handler_p_usage(&$page)
+    function handler_ref_search(&$page)
     {
-        global $globals;
+        $page->changeTpl('referent.tpl');
+
+        $page->assign('xorg_title', 'Polytechnique.org - Conseil Pro');
+
+        $secteur_sel     = Post::get('secteur');
+        $ss_secteur_sel  = Post::get('ss_secteur');
+        $pays_sel        = Post::get('pays', '00');
+        $expertise_champ = Post::get('expertise');
+
+        $page->assign('pays_sel', $pays_sel);
+        $page->assign('expertise_champ', $expertise_champ);
+        $page->assign('secteur_sel', $secteur_sel);
+        $page->assign('ss_secteur_sel', $ss_secteur_sel);
+
+        //recuperation des noms de secteurs
+        $res = XDB::iterRow("SELECT id, label FROM emploi_secteur");
+        $secteurs[''] = '';
+        while (list($tmp_id, $tmp_label) = $res->next()) {
+            $secteurs[$tmp_id] = $tmp_label;
+        }
+        $page->assign_by_ref('secteurs', $secteurs);
+
+        //on recupere les sous-secteurs si necessaire
+        $ss_secteurs[''] = '';
+        if (!empty($secteur_sel)) {
+            $res = XDB::iterRow("SELECT id, label FROM emploi_ss_secteur
+                                          WHERE secteur = {?}", $secteur_sel);
+            while (list($tmp_id, $tmp_label) = $res->next()) {
+                $ss_secteurs[$tmp_id] = $tmp_label;
+            }
+        }
+        $page->assign_by_ref('ss_secteurs', $ss_secteurs);
+
+        //recuperation des noms de pays
+        $res = XDB::iterRow("SELECT a2, pays FROM geoloc_pays
+                                      WHERE pays <> '' ORDER BY pays");
+        $pays['00'] = '';
+        while (list($tmp_id, $tmp_label) = $res->next()) {
+            $pays[$tmp_id] = $tmp_label;
+        }
+        $page->assign_by_ref('pays', $pays);
+
+        // nb de mentors
+        $res = XDB::query("SELECT count(*) FROM mentor");
+        $page->assign('mentors_number', $res->fetchOneCell());
+
+        if (!Env::has('Chercher')) {
+            return;
+        }
 
+        // On vient d'un formulaire
+        $where = array();
+
+        if ($pays_sel != '00') {
+            $where[] = "mp.pid = '".addslashes($pays_sel)."'";
+        }
+        if ($secteur_sel) {
+            $where[] = "ms.secteur = '".addslashes($secteur_sel)."'";
+            if ($ss_secteur_sel) {
+                $where[] = "ms.ss_secteur = '".addslashes($ss_secteur_sel)."'";
+            }
+        }
+        if ($expertise_champ) {
+            $where[] = "MATCH(m.expertise) AGAINST('".addslashes($expertise_champ)."')";
+        }
+
+        if ($where) {
+            $where = join(' AND ', $where);
+
+            $sql = "SELECT  m.uid, a.prenom, a.nom, a.promo,
+                            l.alias AS bestalias, m.expertise, mp.pid,
+                            ms.secteur, ms.ss_secteur
+                      FROM  mentor        AS m
+                 LEFT JOIN  auth_user_md5 AS a ON(m.uid = a.user_id)
+                INNER JOIN  aliases       AS l ON (a.user_id=l.id AND
+                                                   FIND_IN_SET('bestalias', l.flags))
+                 LEFT JOIN  mentor_pays   AS mp ON(m.uid = mp.uid)
+                 LEFT JOIN  mentor_secteurs AS ms ON(m.uid = ms.uid)
+                     WHERE  $where
+                  GROUP BY  uid
+                  ORDER BY  RAND({?})";
+            $res = XDB::iterator($sql, S::v('uid'));
+
+            if ($res->total() == 0) {
+                $page->assign('recherche_trop_large', true);
+                return;
+            }
+
+            $nb_max_res_total = 100;
+            $nb_max_res_ppage = 10;
+
+            $curpage   = Env::getInt('curpage', 1);
+            $personnes = array();
+            $i         = 0;
+
+            while (($pers = $res->next()) && count($personnes) < $nb_max_res_total) {
+                $the_page = intval($i / $nb_max_res_ppage) + 1;
+                if ($the_page == $curpage) {
+                    $personnes[] = $pers;
+                }
+                $i ++;
+            }
+
+            $page->assign('personnes', $personnes);
+            $page->assign('curpage',   $curpage);
+            $page->assign('nb_pages_total',
+                          intval($res->total() / $nb_max_res_ppage) + 1);
+        }
+    }
+
+    function handler_p_usage(&$page)
+    {
         $page->changeTpl('nomusage.tpl');
 
         require_once 'validations.inc.php';
         require_once 'xorg.misc.inc.php';
 
-        $res = $globals->xdb->query(
-                "SELECT  u.nom,u.nom_usage,u.flags,e.alias
+        $res = XDB::query(
+                "SELECT  u.nom, u.nom_usage, u.flags, e.alias
                    FROM  auth_user_md5  AS u
-              LEFT JOIN  aliases        AS e ON(u.user_id = e.id AND FIND_IN_SET('usage',e.flags))
-                  WHERE  user_id={?}", Session::getInt('uid'));
+              LEFT JOIN  aliases        AS e ON(u.user_id = e.id
+                                                AND FIND_IN_SET('usage', e.flags))
+                  WHERE  user_id={?}", S::v('uid'));
 
-        list($nom,$usage_old,$flags,$alias_old) = $res->fetchOneRow();
+        list($nom, $usage_old, $flags, $alias_old) = $res->fetchOneRow();
         $flags = new flagset($flags);
         $page->assign('usage_old', $usage_old);
         $page->assign('alias_old',  $alias_old);
@@ -393,13 +606,11 @@ class ProfileModule extends PLModule
                 if ($reason == 'other') {
                     $reason = Env::get('other_reason');
                 }
-                $myusage = new UsageReq(Session::getInt('uid'), $nom_usage, $reason);
+                $myusage = new UsageReq(S::v('uid'), $nom_usage, $reason);
                 $myusage->submit();
                 $page->assign('myusage', $myusage);
             }
         }
-
-        return PL_OK;
     }
 
     function handler_trombi(&$page, $promo = null)
@@ -407,16 +618,16 @@ class ProfileModule extends PLModule
         require_once 'trombi.inc.php';
 
         $page->changeTpl('trombipromo.tpl');
-        $page->assign('xorg_title','Polytechnique.org - Trombi Promo');
+        $page->assign('xorg_title', 'Polytechnique.org - Trombi Promo');
 
         if (is_null($promo)) {
-            return PL_OK;
+            return;
         }
 
         $this->promo = $promo = intval($promo);
 
         if ($promo >= 1900 && $promo < intval(date('Y'))
-        || ($promo == -1 && has_perms()))
+        || ($promo == -1 && S::has_perms()))
         {
             $trombi = new Trombi(array($this, '_trombi_getlist'));
             $trombi->hidePromo();
@@ -425,8 +636,6 @@ class ProfileModule extends PLModule
         } else {
             $page->trig('Promotion incorrecte (saisir au format YYYY). Recommence.');
         }
-
-        return PL_OK;
     }
 
     function format_adr($params, &$smarty)
@@ -462,14 +671,14 @@ class ProfileModule extends PLModule
         $user  = get_user_details($login);
 
         // alias virtual
-        $res = $globals->xdb->query(
+        $res = XDB::query(
                 "SELECT alias
                    FROM virtual
              INNER JOIN virtual_redirect USING(vid)
              INNER JOIN auth_user_quick  ON ( user_id = {?} AND emails_alias_pub = 'public' )
                   WHERE ( redirect={?} OR redirect={?} )
                         AND alias LIKE '%@{$globals->mail->alias_dom}'",
-                Session::getInt('uid'),
+                S::v('uid'),
                 $user['forlife'].'@'.$globals->mail->domain,
                 $user['forlife'].'@'.$globals->mail->domain2);
 
@@ -481,8 +690,6 @@ class ProfileModule extends PLModule
         header("Cache-Control: ");
         header("Content-type: text/x-vcard\n");
         header("Content-Transfer-Encoding: Quoted-Printable\n");
-
-        return PL_OK;
     }
 }