Arrange templates by module
[platal.git] / modules / xnetgrp.php
index 580716b..5f7a8fe 100644 (file)
@@ -82,7 +82,9 @@ class XnetGrpModule extends PLModule
             '%grp/annuaire/vcard' => $this->make_hook('vcard',     AUTH_MDP),
             '%grp/trombi'         => $this->make_hook('trombi',    AUTH_MDP),
             '%grp/subscribe'      => $this->make_hook('subscribe', AUTH_MDP),
-            '%grp/paiement'       => $this->make_hook('paiement',  AUTH_MDP),
+            '%grp/unsubscribe'    => $this->make_hook('unsubscribe', AUTH_MDP),
+
+            '%grp/change_rights'  => $this->make_hook('change_rights', AUTH_MDP),
 
             '%grp/admin/annuaire'
                  => $this->make_hook('admin_annuaire', AUTH_MDP),
@@ -111,11 +113,7 @@ class XnetGrpModule extends PLModule
             return PL_NOT_FOUND;
         }
 
-        $page->changeTpl('xnet/groupe/asso.tpl');
-        $page->setType($globals->asso('cat'));
-        $page->assign('is_admin', may_update());
-        $page->assign('is_member', is_member());
-        $page->assign('logged', S::logged());
+        new_group_open_page('xnetgrp/asso.tpl');
 
         if (S::logged()) {
             if (Env::has('read')) {
@@ -210,7 +208,7 @@ class XnetGrpModule extends PLModule
     {
         global $globals;
 
-        new_groupadmin_page('xnet/groupe/edit.tpl');
+        new_groupadmin_page('xnetgrp/edit.tpl');
 
         if (Post::has('submit')) {
             if (S::has_perms()) {
@@ -223,7 +221,7 @@ class XnetGrpModule extends PLModule
                         SET  nom={?}, diminutif={?}, cat={?}, dom={?},
                              descr={?}, site={?}, mail={?}, resp={?},
                              forum={?}, mail_domain={?}, ax={?}, pub={?},
-                             sub_url={?}, inscriptible={?}
+                             sub_url={?}, inscriptible={?}, unsub_url={?}
                       WHERE  id={?}",
                       Post::v('nom'), Post::v('diminutif'),
                       Post::v('cat'), Post::i('dom'),
@@ -232,7 +230,7 @@ class XnetGrpModule extends PLModule
                       Post::v('forum'), Post::v('mail_domain'),
                       Post::has('ax'), Post::has('pub')?'private':'public',
                       Post::v('sub_url'), Post::v('inscriptible'),
-                      $globals->asso('id'));
+                      Post::v('unsub_url'),$globals->asso('id'));
                 if (Post::v('mail_domain')) {
                     XDB::execute('INSERT INTO virtual_domains (domain) VALUES({?})',
                                            Post::v('mail_domain'));
@@ -241,13 +239,15 @@ class XnetGrpModule extends PLModule
                 XDB::execute(
                     "UPDATE  groupex.asso
                         SET  descr={?}, site={?}, mail={?}, resp={?},
-                             forum={?}, ax={?}, pub= {?}, sub_url={?}
+                             forum={?}, ax={?}, pub= {?}, sub_url={?},
+                             unsub_url={?}
                       WHERE  id={?}",
                       Post::v('descr'), Post::v('site'),
                       Post::v('mail'), Post::v('resp'),
                       Post::v('forum'), Post::has('ax'),
                       Post::has('pub')?'private':'public',
-                      Post::v('sub_url'), $globals->asso('id'));
+                      Post::v('sub_url'), Post::v('unsub_url'),
+                      $globals->asso('id'));
             }
 
             if ($_FILES['logo']['name']) {
@@ -273,7 +273,7 @@ class XnetGrpModule extends PLModule
     {
         global $globals;
 
-        new_groupadmin_page('xnet/groupe/mail.tpl');
+        new_groupadmin_page('xnetgrp/mail.tpl');
         $mmlist = new MMList(S::v('uid'), S::v('password'),
                            $globals->asso('mail_domain'));
         $page->assign('listes', $mmlist->get_lists());
@@ -284,10 +284,11 @@ class XnetGrpModule extends PLModule
             $body  = Post::v('body');
 
             $mls = array_keys(Env::v('ml', array()));
+            $mbr = array_keys(Env::v('membres', array()));
 
-            require_once 'xnet/mail.inc.php';
-            $tos = get_all_redirects(Post::has('membres'), $mls, $mmlist);
-            send_xnet_mails($from, $sujet, $body, $tos, Post::v('replyto'));
+            require_once dirname(__FILE__) . '/xnetgrp/mail.inc.php';
+            $tos = get_all_redirects($mbr,  $mls, $mmlist);
+            send_xnet_mails($from, $sujet, $body, $tos, Post::v('replyto'), $_FILES['uploaded']);
             $page->kill("Mail envoyé !");
             $page->assign('sent', true);
         }
@@ -296,14 +297,7 @@ class XnetGrpModule extends PLModule
     function handler_annuaire(&$page)
     {
         global $globals;
-
-        if ($globals->asso('pub') == 'public') {
-            new_group_page('xnet/groupe/annuaire.tpl');
-        } else {
-            new_groupadmin_page('xnet/groupe/annuaire.tpl');
-        }
-
-        $page->assign('admin', may_update());
+        new_annu_page('xnetgrp/annuaire.tpl');
 
         $sort = Env::v('order');
         switch (Env::v('order')) {
@@ -314,7 +308,7 @@ class XnetGrpModule extends PLModule
         }
         $page->assign('sort', $sort);
 
-        if ($group == 'initiale')
+        if ($group == 'initiale') {
             $res = XDB::iterRow(
                         'SELECT  UPPER(SUBSTRING(
                                     IF(m.origine="X", IF(u.nom_usage<>"", u.nom_usage, u.nom),m.nom),
@@ -324,16 +318,17 @@ class XnetGrpModule extends PLModule
                           WHERE  asso_id = {?}
                        GROUP BY  letter
                        ORDER BY  letter', $globals->asso('id'));
-        else
+        } else {
             $res = XDB::iterRow(
-                        'SELECT  IF(m.origine="X",u.promo,"extérieur") AS promo,
+                        'SELECT  IF(m.origine="X",u.promo,
+                                    IF(m.origine="ext", "extérieur", "personne morale")) AS promo,
                                  COUNT(*), IF(m.origine="X",u.promo,"") AS promo_o
                            FROM  groupex.membres AS m
                       LEFT JOIN  auth_user_md5   AS u ON ( u.user_id = m.uid )
                           WHERE  asso_id = {?}
                        GROUP BY  promo
                        ORDER BY  promo_o DESC', $globals->asso('id'));
-
+        }
         $alphabet = array();
         $nb_tot = 0;
         while (list($char, $nb) = $res->next()) {
@@ -372,7 +367,7 @@ class XnetGrpModule extends PLModule
                            IF(u.nom_usage<>"", u.nom_usage, u.nom),
                            m.nom) LIKE "'.addslashes(Env::v('initiale')).'%"';
         } elseif (Env::has('promo')) {
-            $ini = 'AND IF(m.origine="X", u.promo, "extérieur") = "'
+            $ini = 'AND IF(m.origine="X", u.promo, IF(m.origine="ext", "extérieur", "personne morale")) = "'
                  .addslashes(Env::v('promo')).'"';
         } elseif (Env::has('admin')) {
             $ini = 'AND m.perms = "admin"';
@@ -381,7 +376,7 @@ class XnetGrpModule extends PLModule
         $ann = XDB::iterator(
                   "SELECT  IF(m.origine='X',IF(u.nom_usage<>'', u.nom_usage, u.nom) ,m.nom) AS nom,
                            IF(m.origine='X',u.prenom,m.prenom) AS prenom,
-                           IF(m.origine='X',u.promo,'extérieur') AS promo,
+                           IF(m.origine='X', u.promo, IF(m.origine='ext', 'extérieur', 'personne morale')) AS promo,
                            IF(m.origine='X',u.promo,'') AS promo_o,
                            IF(m.origine='X' AND u.perms != 'pending',a.alias,m.email) AS email,
                            IF(m.origine='X',FIND_IN_SET('femme', u.flags), m.sexe) AS femme,
@@ -393,7 +388,7 @@ class XnetGrpModule extends PLModule
                 LEFT JOIN  auth_user_md5   AS u ON ( u.user_id = m.uid )
                 LEFT JOIN  aliases         AS a ON ( a.id = m.uid AND a.type='a_vie' )
                     WHERE  m.asso_id = {?} $ini
-                           AND (m.origine = 'ext' OR u.perms != 'pending' OR m.email IS NOT NULL)
+                           AND (m.origine != 'X' OR u.perms != 'pending' OR m.email IS NOT NULL)
                  ORDER BY  $tri
                     LIMIT  {?},{?}", $globals->asso('id'), $ofs*NB_PER_PAGE, NB_PER_PAGE);
         $page->assign('ann', $ann);
@@ -402,14 +397,7 @@ class XnetGrpModule extends PLModule
     function handler_trombi(&$page, $num = 1)
     {
         global $globals;
-        if ($globals->asso('cat') == 'Promotions') {
-            new_group_open_page('xnet/groupe/trombi.tpl');
-        } elseif ($globals->asso('pub') == 'public') {
-            new_group_page('xnet/groupe/trombi.tpl');
-        } else {
-            new_groupadmin_page('xnet/groupe/trombi.tpl');
-        }
-        $page->assign('admin', may_update());
+        new_annu_page('xnetgrp/trombi.tpl');
         
         $page->assign('urlmainsite', "https://www.polytechnique.org/");
         $trombi = new Trombi(array($this, '_trombi_getlist'));
@@ -463,11 +451,7 @@ class XnetGrpModule extends PLModule
     {
         global $globals;
 
-        $page->changeTpl('xnet/groupe/inscrire.tpl');
-
-        $page->setType($globals->asso('cat'));
-        $page->assign('asso', $globals->asso());
-        $page->assign('admin', may_update());
+        new_group_open_page('xnetgrp/inscrire.tpl');
 
         if (!$globals->asso('inscriptible'))
                 $page->kill("Il n'est pas possible de s'inscire en ligne à ce "
@@ -499,8 +483,7 @@ class XnetGrpModule extends PLModule
                     XDB::execute("INSERT INTO groupex.membres
                                             VALUES ({?}, {?}, 'membre', 'X', NULL, NULL, NULL, NULL, NULL)",
                                             $globals->asso('id'), $uid);
-                    require_once 'diogenes/diogenes.hermes.inc.php';
-                    $mailer = new HermesMailer();
+                    $mailer = new PlMailer();
                     $mailer->addTo("$u@polytechnique.org");
                     $mailer->setFrom('"'.S::v('prenom').' '.S::v('nom')
                                      .'" <'.S::v('forlife').'@polytechnique.org>');
@@ -518,8 +501,7 @@ class XnetGrpModule extends PLModule
                 }
                 elseif (Env::has('refuse'))
                 {
-                    require_once 'diogenes/diogenes.hermes.inc.php';
-                    $mailer = new HermesMailer();
+                    $mailer = new PlMailer();
                     $mailer->addTo("$u@polytechnique.org");
                     $mailer->setFrom('"'.S::v('prenom').' '.S::v('nom')
                                      .'" <'.S::v('forlife').'@polytechnique.org>');
@@ -577,8 +559,7 @@ class XnetGrpModule extends PLModule
                         ." résoudre ce problème.\n";
             }
 
-            require_once 'diogenes/diogenes.hermes.inc.php';
-            $mailer = new HermesMailer();
+            $mailer = new PlMailer();
             $mailer->addTo($to);
             $mailer->setFrom('"'.S::v('prenom').' '.S::v('nom')
                              .'" <'.S::v('forlife').'@polytechnique.org>');
@@ -588,72 +569,31 @@ class XnetGrpModule extends PLModule
         }
     }
 
-    function handler_paiement(&$page)
+    function handler_change_rights(&$page)
     {
-        global $globals;
-
-        new_group_page('xnet/groupe/telepaiement.tpl');
-
-        $res = XDB::query(
-                "SELECT id, text
-                  FROM {$globals->money->mpay_tprefix}paiements
-                 WHERE asso_id = {?} AND NOT FIND_IN_SET(flags, 'old')
-              ORDER BY id DESC", $globals->asso('id'));
-        $tit = $res->fetchAllAssoc();
-        $page->assign('titres', $tit);
-
-        $order = Env::v('order', 'timestamp');
-        $orders = array('timestamp', 'nom', 'promo', 'montant');
-        if (!in_array($order, $orders)) {
-            $order = 'timestamp';
-        }
-        $inv_order = Env::v('order_inv', 0);
-        $page->assign('order', $order);
-        $page->assign('order_inv', !$inv_order);
-
-        if ($order == 'timestamp') {
-            $inv_order = !$inv_order;
-        }
-
-        if ($inv_order) {
-            $inv_order = ' DESC';
-        } else {
-            $inv_order = '';
-        }
-        if ($order == 'montant') {
-            $order = 'LENGTH(montant) '.$inv_order.', montant';
-        }
-
-        $orderby = 'ORDER BY '.$order.$inv_order;
-        if ($order != 'nom') {
-            $orderby .= ', nom'; $inv_order = '';
-        }
-        $orderby .= ', prenom'.$inv_order;
-        if ($order != 'timestamp') {
-            $orderby .= ', timestamp DESC';
-        }
-
-        if (may_update()) {
-            $trans = array();
-            foreach($tit as $foo) {
-                $pid = $foo['id'];
-                $res = XDB::query(
-                        "SELECT  IF(u.nom_usage<>'', u.nom_usage, u.nom) AS nom,
-                                 u.prenom, u.promo, a.alias, timestamp AS `date`, montant
-                           FROM  {$globals->money->mpay_tprefix}transactions AS t
-                     INNER JOIN  auth_user_md5  AS u ON ( t.uid = u.user_id )
-                     INNER JOIN  aliases        AS a ON ( t.uid = a.id AND a.type='a_vie' )
-                          WHERE  ref = {?} ".$orderby, $pid);
-                $trans[$pid] = $res->fetchAllAssoc();
-                $sum = 0;
-                foreach ($trans[$pid] as $i => $t) {
-                    $sum += strtr(substr($t['montant'], 0, strpos($t['montant'], 'EUR')), ',', '.');
-                }
-                $trans[$pid][] = array('nom' => 'somme totale',
-                                       'montant' => strtr($sum, '.', ',').' EUR');
+        if (Env::has('right') && (may_update() || S::has('suid'))) {
+            switch (Env::v('right')) {
+              case 'admin':              
+                XnetSession::killSuid();
+                break;
+              case 'anim':
+                XnetSession::doSelfSuid();
+                may_update(true);
+                is_member(true);
+                break;
+              case 'member':
+                XnetSession::doSelfSuid();  
+                may_update(false, true);
+                is_member(true);
+                break;
+              case 'logged':
+                XnetSession::doSelfSuid();
+                may_update(false, true);
+                is_member(false, true);
+                break;
             }
-            $page->assign('trans', $trans);
         }
+        pl_redirect("");
     }
 
     function handler_admin_annuaire(&$page)
@@ -662,7 +602,7 @@ class XnetGrpModule extends PLModule
 
         require_once 'xnet/mail.inc.php';
 
-        new_groupadmin_page('xnet/groupe/annuaire-admin.tpl');
+        new_groupadmin_page('xnetgrp/annuaire-admin.tpl');
         $mmlist = new MMList(S::v('uid'), S::v('password'),
                              $globals->asso('mail_domain'));
         $lists  = $mmlist->get_lists();
@@ -707,7 +647,7 @@ class XnetGrpModule extends PLModule
     {
         global $globals;
 
-        new_groupadmin_page('xnet/groupe/membres-add.tpl');
+        new_groupadmin_page('xnetgrp/membres-add.tpl');
         $page->addJsLink('ajax.js');
 
         if (is_null($email)) {
@@ -738,7 +678,7 @@ class XnetGrpModule extends PLModule
                     $uid = Env::i('userid');
                     $res = XDB::query("SELECT *
                                          FROM auth_user_md5
-                                         WHERE user_id = {?} AND perms = 'pending'", $uid);
+                                        WHERE user_id = {?} AND perms = 'pending'", $uid);
                     if ($res->numRows() == 1) {
                         XDB::execute('INSERT INTO groupex.membres (uid, asso_id, origine, email)
                                            VALUES ({?}, {?}, "X", {?})',
@@ -776,7 +716,7 @@ class XnetGrpModule extends PLModule
     function handler_admin_member_new_ajax(&$page)
     {
         header('Content-Type: text/html; charset="iso-8859-15"');
-        $page->changeTpl('xnet/groupe/membres-new-search.tpl', NO_SKIN);
+        $page->changeTpl('xnetgrp/membres-new-search.tpl', NO_SKIN);
         list($nom, $prenom) = str_replace(array('-', ' ', "'"), '%', array(Env::v('nom'), Env::v('prenom')));
         $where = "perms = 'pending'";
         if (!empty($nom)) {
@@ -798,63 +738,95 @@ class XnetGrpModule extends PLModule
         }
     }
 
-    function handler_admin_member_del(&$page, $user = null)
+    function unsubscribe(&$user)
     {
-        global $globals;
-
-        new_groupadmin_page('xnet/groupe/membres-del.tpl');
-        $user = get_infos($user);
-        if (empty($user)) {
-            return PL_NOT_FOUND;
-        }
-        $page->assign('user', $user);
-
-        if (!Post::has('confirm')) {
-            return;
-        }
-
+        global $globals, $page;
         XDB::execute(
                 "DELETE FROM  groupex.membres WHERE uid={?} AND asso_id={?}",
                 $user['uid'], $globals->asso('id'));
 
-        // don't unsubscribe email from list if other user use same email
         $user_same_email = get_infos($user['email']);
+        $domain = $globals->asso('mail_domain');
 
-        if (($domain = $globals->asso('mail_domain')) && empty($user_same_email)) {
+        if (!$domain || !empty($user_same_email)) {
+            return true;
+        }
 
-            $mmlist = new MMList(S::v('uid'), S::v('password'), $domain);
-            $listes = $mmlist->get_lists($user['email2']);
+        $mmlist = new MMList(S::v('uid'), S::v('password'), $domain);
+        $listes = $mmlist->get_lists($user['email2']);
 
-            foreach ($listes as $liste) {
-                if ($liste['sub'] == 2) {
+        $may_update = may_update();
+        $warning    = false;
+        foreach ($listes as $liste) {
+            if ($liste['sub'] == 2) {
+                if ($may_update) {
                     $mmlist->mass_unsubscribe($liste['list'], Array($user['email2']));
-                    $page->trig("{$user['prenom']} {$user['nom']} a été"
-                                ." désinscrit de {$liste['list']}");
-                } elseif ($liste['sub']) {
-                    $page->trig("{$user['prenom']} {$user['nom']} a une"
-                                ." demande d'inscription en cours sur la"
-                                ." liste {$liste['list']}@ !");
+                } else {
+                    $mmlist->unsubscribe($liste['list']);
                 }
+            } elseif ($liste['sub']) {
+                $page->trig("{$user['prenom']} {$user['nom']} a une"
+                            ." demande d'inscription en cours sur la"
+                            ." liste {$liste['list']}@ !");
+                $warning = true;
             }
+        }
 
-            XDB::execute(
-                    "DELETE FROM  virtual_redirect
-                           USING  virtual_redirect
-                      INNER JOIN  virtual USING(vid)
-                           WHERE  redirect={?} AND alias LIKE {?}", $user['email'], '%@'.$domain);
-            if (mysql_affected_rows()) {
-                $page->trig("{$user['prenom']} {$user['nom']} a été désabonné des alias du groupe !");
-            }
+        XDB::execute(
+                "DELETE FROM  virtual_redirect
+                       USING  virtual_redirect
+                  INNER JOIN  virtual USING(vid)
+                       WHERE  redirect={?} AND alias LIKE {?}", $user['email'], '%@'.$domain);
+        return !$warning;
+    }
+
+    function handler_unsubscribe(&$page)
+    {
+        new_group_page('xnetgrp/membres-del.tpl');
+        $user = get_infos(S::v('forlife'));
+        if (empty($user)) {
+            return PL_NOT_FOUND;
+        }
+        $page->assign('self', true);
+        $page->assign('user', $user);
+
+        if (!Post::has('confirm')) {
+            return;
         }
 
-        $page->trig("{$user['prenom']} {$user['nom']} a été retiré du groupe !");
+        if ($this->unsubscribe($user)) {
+            $page->trig('Vous avez été désinscrit du groupe avec succès');
+        } else {
+            $page->trig('Vous avez été désinscrit du groupe, mais des erreurs se sont produites lors des désinscriptions des alias et des mailing-lists.');
+        }
+        $page->assign('is_member', is_member(true));
+    }
+
+    function handler_admin_member_del(&$page, $user = null)
+    {
+        new_groupadmin_page('xnetgrp/membres-del.tpl');
+        $user = get_infos($user);
+        if (empty($user)) {
+            return PL_NOT_FOUND;
+        }
+        $page->assign('user', $user);
+
+        if (!Post::has('confirm')) {
+            return;
+        }
+
+        if ($this->unsubscribe($user)) {
+            $page->trig("{$user['prenom']} {$user['nom']} a été désabonné du groupe !");
+        } else {
+            $page->trig("{$user['prenom']} {$user['nom']} a été désabonné du groupe, mais des erreurs subsistent !");
+        }
     }
 
     function handler_admin_member(&$page, $user)
     {
         global $globals;
 
-        new_groupadmin_page('xnet/groupe/membres-edit.tpl');
+        new_groupadmin_page('xnetgrp/membres-edit.tpl');
 
         $user = get_infos($user);
         if (empty($user)) {
@@ -868,17 +840,18 @@ class XnetGrpModule extends PLModule
             $email_changed = ($user['origine'] != 'X' && strtolower($user['email']) != strtolower(Post::v('email')));
             $from_email = $user['email'];
             if ($user['origine'] != 'X') {
+                $user['nom']     = Post::v('nom');
+                $user['prenom']  = (Post::v('origine') == 'ext') ? Post::v('prenom') : '';
+                $user['sexe']    = (Post::v('origine') == 'ext') ? Post::v('sexe') : 0;
+                $user['origine'] = Post::v('origine');
                 XDB::query('UPDATE groupex.membres
-                               SET prenom={?}, nom={?}, email={?}, sexe={?}
+                               SET prenom={?}, nom={?}, email={?}, sexe={?}, origine={?}
                              WHERE uid={?} AND asso_id={?}',
-                           Post::v('prenom'), Post::v('nom'),
-                           Post::v('email'), Post::v('sexe'),
-                           $user['uid'], $globals->asso('id'));
-                $user['nom']    = Post::v('nom');
-                $user['prenom'] = Post::v('prenom');
-                $user['sexe']   = Post::v('sexe');
-                $user['email']  = Post::v('email');
-                $user['email2'] = Post::v('email');
+                           $user['prenom'], $user['nom'], Post::v('email'),
+                           $user['sexe'], $user['origine'], $user['uid'],
+                           $globals->asso('id'));
+                $user['email']   = Post::v('email');
+                $user['email2']  = Post::v('email');
             }
 
             $perms = Post::i('is_admin');
@@ -894,7 +867,7 @@ class XnetGrpModule extends PLModule
             foreach (Env::v('ml1', array()) as $ml => $state) {
                 $ask = empty($_REQUEST['ml2'][$ml]) ? 0 : 2;
                 if ($ask == $state) {
-                    if ($state) {
+                    if ($state && $email_changed) {
                         $mmlist->replace_email($ml, $from_email, $user['email2']);
                         $page->trig("L'abonnement de {$user['prenom']} {$user['nom']} à $ml@ a été mis à jour");
                     }
@@ -954,7 +927,7 @@ class XnetGrpModule extends PLModule
         global $globals;
         require_once('rss.inc.php');
         require_once('url_catcher.inc.php');
-        $uid = init_rss('xnet/groupe/announce-rss.tpl', $user, $hash, false);
+        $uid = init_rss('xnetgrp/announce-rss.tpl', $user, $hash, false);
         $page->register_modifier('url_catcher', 'url_catcher');
 
         if ($uid) {
@@ -964,13 +937,15 @@ class XnetGrpModule extends PLModule
                              INNER JOIN groupex.announces AS a ON ( (a.promo_min = 0 OR a.promo_min <= u.promo)
                                                                   AND (a.promo_max = 0 OR a.promo_max <= u.promo))
                              INNER JOIN auth_user_md5 AS u2 ON (u2.user_id = a.user_id)
-                                  WHERE u.user_id = {?} AND peremption >= NOW()", $uid);
+                             WHERE u.user_id = {?} AND peremption >= NOW() AND a.asso_id = {?}",
+                                   $uid, $globals->asso('id'));
         } else {
             $rss = XDB::iterator("SELECT a.id, a.titre, a.texte, a.create_date,
                                          IF(u.nom_usage != '', u.nom_usage, u.nom) AS nom, u.prenom, u.promo
                                     FROM groupex.announces AS a
                              INNER JOIN auth_user_md5 AS u USING(user_id)
-                                   WHERE FIND_IN_SET(a.flags, 'public') AND peremption >= NOW()");
+                             WHERE FIND_IN_SET(a.flags, 'public') AND peremption >= NOW() AND a.asso_id = {?}",
+                                  $globals->asso('id'));
         }
         $page->assign('asso', $globals->asso());
         $page->assign('rss', $rss);
@@ -979,7 +954,7 @@ class XnetGrpModule extends PLModule
     function handler_edit_announce(&$page, $aid = null)
     {
         global $globals, $platal;
-        new_groupadmin_page('xnet/groupe/announce-edit.tpl');
+        new_groupadmin_page('xnetgrp/announce-edit.tpl');
         $page->assign('new', is_null($aid));
         $art = array();
 
@@ -1008,7 +983,7 @@ class XnetGrpModule extends PLModule
             }
 
             if (!$art['public'] &&
-                ($art['promo_min'] > $art['promo_max'] ||
+                (($art['promo_min'] > $art['promo_max'] && $art['promo_max'] != 0) ||
                  ($art['promo_min'] != 0 && ($art['promo_min'] <= 1900 || $art['promo_min'] >= 2020)) ||
                  ($art['promo_max'] != 0 && ($art['promo_max'] <= 1900 || $art['promo_max'] >= 2020))))
             {
@@ -1108,7 +1083,7 @@ class XnetGrpModule extends PLModule
     function handler_admin_announce(&$page)
     {
         global $globals;
-        new_groupadmin_page('xnet/groupe/announce-admin.tpl');
+        new_groupadmin_page('xnetgrp/announce-admin.tpl');
 
         if (Env::has('del')) {
             XDB::execute("DELETE FROM groupex.announces