From 23ba40c466e05d369cd9e2a96107f38e309b767a Mon Sep 17 00:00:00 2001
From: Pascal Corpet <pascal.corpet@m4x.org>
Date: Sun, 6 Jun 2010 23:06:59 +0200
Subject: [PATCH] Fixes unsubscription from group

---
 classes/group.php                 |  4 ++--
 modules/xnetgrp.php               | 24 +++++++++++++-----------
 templates/xnetgrp/membres-del.tpl |  4 ++--
 3 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/classes/group.php b/classes/group.php
index ce93e92..f58ff6b 100644
--- a/classes/group.php
+++ b/classes/group.php
@@ -73,13 +73,13 @@ class Group
 
     public function iterMembers($extra_cond = null, $sort = null, $limit = null)
     {
-        $uf = getMembersFilter($extra_cond, $sort);
+        $uf = $this->getMembersFilter($extra_cond, $sort);
         return $uf->iterUsers($limit);
     }
 
     public function iterAdmins($extra_cond = null, $sort = null, $limit = null)
     {
-        $uf = getAdminsFilter($extra_cond, $sort);
+        $uf = $this->getAdminsFilter($extra_cond, $sort);
         return $uf->iterUsers($limit);
     }
 
diff --git a/modules/xnetgrp.php b/modules/xnetgrp.php
index fa6c111..215f45b 100644
--- a/modules/xnetgrp.php
+++ b/modules/xnetgrp.php
@@ -731,18 +731,20 @@ class XnetGrpModule extends PLModule
 
         $may_update = may_update();
         $warning    = false;
-        foreach ($listes as $liste) {
-            if ($liste['sub'] == 2) {
-                if ($may_update) {
-                    $mmlist->mass_unsubscribe($liste['list'], Array($user->forlifeEmail()));
-                } else {
-                    $mmlist->unsubscribe($liste['list']);
+        if (is_array($listes)) {
+            foreach ($listes as $liste) {
+                if ($liste['sub'] == 2) {
+                    if ($may_update) {
+                        $mmlist->mass_unsubscribe($liste['list'], Array($user->forlifeEmail()));
+                    } else {
+                        $mmlist->unsubscribe($liste['list']);
+                    }
+                } elseif ($liste['sub']) {
+                    Platal::page()->trigWarning($user->fullName() . " a une"
+                                               ." demande d'inscription en cours sur la"
+                                               ." liste {$liste['list']}@ !");
+                    $warning = true;
                 }
-            } elseif ($liste['sub']) {
-                Platal::page()->trigWarning($user->fullName() . " a une"
-                                           ." demande d'inscription en cours sur la"
-                                           ." liste {$liste['list']}@ !");
-                $warning = true;
             }
         }
 
diff --git a/templates/xnetgrp/membres-del.tpl b/templates/xnetgrp/membres-del.tpl
index 51e0b71..30ca337 100644
--- a/templates/xnetgrp/membres-del.tpl
+++ b/templates/xnetgrp/membres-del.tpl
@@ -35,7 +35,7 @@
 <h1>{$asso->nom}&nbsp;: gestion des membres</h1>
 
 <h2>
-  Suppression du membre&nbsp;: {$user.prenom} {$user.nom}
+  Suppression du membre&nbsp;: {profile user=$user groupperms=false sex=false promo=true}
 </h2>
 
 
@@ -47,7 +47,7 @@
     Êtes-vous sûr de vouloir vous désinscrire du groupe {$asso->nom} et de toutes
     les listes de diffusion associées&nbsp;?
     {else}
-    Êtes-vous sûr de vouloir supprimer {$user.prenom} {$user.nom} du groupe,
+    Êtes-vous sûr de vouloir supprimer {$user->fullName()} du groupe,
     lui retirer tous les droits associés à son statut de membre
     et le désabonner de toutes les listes de diffusion du groupe&nbsp;?
     {/if}
-- 
2.1.4