Forum

Salut,
j'utilise la classe group dans mes modules additionels,
j'ai regardé dans la class Groups, et je ne trouve pas de fonction permettant d'ajouter un utilisateur à un groupe.
J'ai aussi regardé dans la base de données, et j'ai remarqué que dans la table group, chaque ligne (représentant un groupe) contient un champs ou il y a tous les membres du groupe (séparé par un |). Et dans la tables member, chaque ligne (représentant un membre) contient un champ correspondant à tous les groupes ou appartient le membre.
Ca me parait bizarre qu'il y ait deux fois la même information dans la table, mais bon il doit bien y avoir une raison...
Ma question était : existe t'il déjà des fonction (ou des bouts de code à réutilier) permettant de gérer l'insertion et la supression de membre dans un groupe ou faut il que je le programme moi même en jonglant avec les deux tables mentionnées ci dessus ?

MKL

Effectivement dans la classe groups il n'y a pas de quoi ajouter un membre dans un groupe.
Pourquoi veux-tu ajouter des membres à un groupe, l'interface de l'administration ne suffit-elle pas ?
Sinon dans la base de données il y a une redondance d'informations, ce n'est pas forcément politiquement correct mais ça permet un gain assez conséquent de temps et donc de performance.

Ce n'est pas une mauvaise idée, je vais l'ajouter dans la classe des groupes

ok merci beaucoup, je m'y met. A moins que CrowkaiT soit plus rapide que moi.

Voici ce que j'ai ajouté à la class Group, ça à l'air de marcher mais je ne mettrais pas ma main à couper qu'il n'y a pas un bug quelque part , donc à vérifier. J'ai essentielement repris le code qui était dans la page admin_groups.php :

j'ai rajouté une variable à la classe :
[code=php]
var $user_id; // id du membre (utile pour les ajouts et suppressions de groupes)
[/code]

Ceci donc dans le constructeur :
[code=php]
global $session;
$this->user_id=$session->data['user_id'];
[/code]

Puis les deux nouvelles methodes :
[code=php]
//Ajout du membre au groupe
function add_member($idgroup)
{
$sql=new Sql;

//On insère le groupe au champ membre.

$user_groups_key = array_search($idgroup, explode('|', $this->user_groups));



if( !is_numeric($user_groups_key) ) //Le membre n'appartient pas déjà au groupe.

{

$sql->query_inject("UPDATE ".PREFIX."member SET user_groups = CONCAT(user_groups, '" . $idgroup . "|') WHERE user_id = '" . numeric($this->user_id) . "'", __LINE__, __FILE__);

}



//On insère le membre dans le groupe.

$members = $sql->query("SELECT members FROM ".PREFIX."group WHERE id = '" . $idgroup . "'", __LINE__, __FILE__);

$members_key = array_search($this->user_id, explode('|', $members));

if( !is_numeric($members_key) ) //Le membre n'appartient pas déjà au groupe.

{

$sql->query_inject("UPDATE ".PREFIX."group SET members = CONCAT(members, '" . $this->user_id . "|') WHERE id = '" . numeric($idgroup) . "'", __LINE__, __FILE__);

}

}

//Suppression le membre d'un groupe
function del_member($idgroup)
{
$sql=new Sql;
$user_groups_key = array_search($idgroup, $this->user_groups);
if( is_numeric($user_groups_key) ){ // le membre est bien dans le groupe

unset($this->user_groups[$user_groups_key]);
$user_groups_bis=$this->user_groups; // on travaille sur une autre varaible pour enlever le level
array_pop($this->user_groups); // on vire la dernière case qui est le level

$sql->query_inject("UPDATE ".PREFIX."member SET user_groups = '" . implode('|', $this->user_groups) . "' WHERE user_id = '" . $this->user_id . "'", __LINE__, __FILE__);

}

$members_group = $sql->query("SELECT members FROM ".PREFIX."group WHERE id = '" . $idgroup . "'", __LINE__, __FILE__);

$members_group = explode('|', $members_group);

$members_group_key = array_search($this->user_id, $members_group);

if( is_numeric($members_group_key) ){ // le membre est bien dans le groupe
unset($members_group[$members_group_key]);

$sql->query_inject("UPDATE ".PREFIX."group SET members = '" . implode('|', $members_group) . "' WHERE id = '" . $idgroup . "'", __LINE__, __FILE__);
}
}
[/code]

J'obtient tout de même le warning suivant quand je lance la methode add_member('1') :
Notice: Array to string conversion in /home/a/aumoinscher.eu/public_html/phpboost/includes/groups.class.php on line 58
ligne 58 : $user_groups_key = array_search($idgroup, explode('|', $this->user_groups));

Je pense qu'il faut aussi regénrer le cache après avoir appelé une des deux méthodes car lorsque je fait deux fois add_member('1'), je me retrouve avec 1|1| dans la base de données, preuve qu'il ne savait pas que le membre était déjà dans le groupe. Ou alors c'est tout simplement mon code qui est défaillant ? Une idée ?

MKL

On est entrain de réorganiser le code de façon Objet mais de la bonne façon donc ça a bouger sur la 2.1. Donc je l'intègrerai différemment de toi.

Pour ta question il faut régénérer la cache
[code=php]$cache->generate_file('groups');[/code]