Anomalie #686
fermémauvais adhérent
100%
Description
rebonjour version 0.7.5
je suis responsable de groupe, lorsque je sélectionne un adhérent puis je clique sur modification, je tombe sur la page du responsable du groupe au lieu de l'adhérent membre dudit groupe
Fichiers
Mis à jour par Johan Cwiklinski il y a plus de 11 ans
- Assigné à mis à Johan Cwiklinski
- Catégorie mis à Core
Mis à jour par Johan Cwiklinski il y a plus de 11 ans
- Fichier 0001-Group-managers-cannot-edit-members-they-own-fixes-68.patch ajouté
Le patch correctif joint devrait corriger le souci.
La photo de l'adhérent n'était pas non plus visible par le responsable de groupe, c'est corrigé également.
Mis à jour par Roland Telle il y a plus de 11 ans
Johan Cwiklinski a écrit :
Le patch correctif joint devrait corriger le souci.
La photo de l'adhérent n'était pas non plus visible par le responsable de groupe, c'est corrigé également.
Grâce à ce correctif, le responsable de groupe peut désormais consulter et modifier la fiche d'un membre de son groupe.
En revanche, le bouton permettant de modifier la liste des membres est actif et déclenche une page intitulée "Sélection des membres du groupe" ... blanche, ce qui est normal puisque le responsable ne peut pas modifier sa liste.
Peut-être est-il possible d'occulter ledit bouton ?
Mis à jour par Johan Cwiklinski il y a plus de 11 ans
- % réalisé changé de 0 à 100
- Statut changé de Nouveau à Résolu
Appliqué par commit a2ca5fbb5fd6fafa17aa853a4aac895dde2e4916.
Mis à jour par Johan Cwiklinski il y a plus de 11 ans
Roland Telle a écrit :
Grâce à ce correctif, le responsable de groupe peut désormais consulter et modifier la fiche d'un membre de son groupe.
En revanche, le bouton permettant de modifier la liste des membres est actif et déclenche une page intitulée "Sélection des membres du groupe" ... blanche, ce qui est normal puisque le responsable ne peut pas modifier sa liste.
Peut-être est-il possible d'occulter ledit bouton ?
Arf... Je n'avais pas vu ce commentaire :/ Oui, en effet, ce bouton ne devrait apparemment pas apparaître (décidément, ce n'est est pas fini de cette histoire de gestion de groupes... voilà pourquoi je rechignais à le faire ! :p).
Mis à jour par Fr cero il y a environ 11 ans
Bonjour, Et oui c'est pas fini, après avoir appliqué le patch sur la 7.5.1 et sur la 7.5.2 si on crée une fiche d'un nouvel adhérent, ensuite en tant qu'admin si on veut la modifer, on arrive sur la fiche de l'admin au lieu de celle du nouvel adhérent !
D'autre part si cet adhérent est responsable d'un ou plusieurs groupes, s'il se connecte et modifie sa fiche alors tous ses groupes disparaissent !
Il devient simple adhérent et ne peut plus faire grand chose. En rétablissant le fichier d'origine on revient à un comportement plus normal de ce coté.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Statut changé de Fermé à In Progress
Il faut que je teste. C'est chouette, c'est sorti en stable déjà.
Ces histoires de groupes et de responsables commencent à me courir.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Fichier admin.patch ajouté
Le patch joint devrait corriger le problème avec les admins (et les membres du staff). Pas eu le temps encore de voir le reste.
Mis à jour par Fr cero il y a environ 11 ans
Je confirme, ça fonctionne pour l'édition des fiches maintenant on arrive bien sur celle souhaitée. Pour le reste courage mais tu arrives au bout du tunnel ! Sans vouloir te stresser, il y a aussi le fait qu'un groupmember ne peut pas envoyer de mail à son ou ses groupes, l'appui sur le bouton ENVOYER UN MAIL ramène sur sa fiche perso. T'as de la chance, il va pleuvoir demain, tu auras tout loisir de coder et de ne pas avoir à être distrait par une sortie VTT, Kite, Surf, rando ou autre...; ! :)
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Fichier 0002-Groups-manager-rights-refs-499-686.patch 0002-Groups-manager-rights-refs-499-686.patch ajouté
- Fichier 0001-Fix-various-issues-regarding-groups-management-fixes.patch 0001-Fix-various-issues-regarding-groups-management-fixes.patch ajouté
Fr cero a écrit :
[...] l'appui sur le bouton ENVOYER UN MAIL ramène sur sa fiche perso
Le bug ici, c'est que le bouton apparaisse, confer #499. L'évolution #499 est ouverte depuis 8 mois, quasi personne n'y participe, je n'ai pas vraiment eu davantage de retours via la liste, bien qu'ayant soulevé ce problème plusieurs fois (à cahier des charges daubé, fonctionnalité daubée... :/).
Quand j'aurai une liste de fonctionnalités demandées fixe et définitive, je m'y attellerai, la gestion des groupes c'est bien trop compliqué/pénible pour que j'y revienne sans cesse.
Pour l'heure, je vais virer le bouton d'envoi de mail pour les responsables de groupes (c'est tout sauf trivial à mettre en oeuvre).
Le patch 0001 corrige le reste des problèmes, (normalement tous les pépins de l'édition des fiches [en admin, responsable de groupe ou simple membre]). Merci de bien vouloir tester ces points.
Le patch 0002 supprime les boutons problématiques de la liste des adhérents pour les responsables de groupes, et permet la génération des cartes de membres, étiquettes ainsi que la liste d'émargement.
Ces patchs sont basés sur la version 0.7.5.3 de Galette.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Fichier
0001-Group-managers-cannot-edit-members-they-own-fixes-68.patchsupprimé
Mis à jour par Fr cero il y a environ 11 ans
J'ai patché mes fichiers (pas rapide depuis le git de ton dépôt,il faut copier ligne par ligne sinon on récupère les sauts de lignes et la numérotation !). Émargement, pdf, étiquetteé et cartes fonctionnent correctement.
Les édition des fiches fonctionnent apparemment bien aussi.
Les bémols sont:
&) Dans gestion_adherents.php le filtre par groupe ne fonctionne pas correctement, la requête qui est envoyée devrait avoir une clause WHERE sur g.id_group me semble t'il.
SELECT DISTINCT `a`.*, `p`.* FROM `galette_adherents` AS `a`
INNER JOIN `galette_statuts` AS `p` ON a.id_statut=p.id_statut
INNER JOIN `galette_groups_members` AS `g` ON a.id_adh=g.id_adh
INNER JOIN `galette_groups_managers` AS `m` ON g.id_group=m.id_group WHERE (m.id_adh = '259') ORDER BY `nom_adh` ASC, `prenom_adh` ASC
2) Dans gestion_groupes il y a encore le bouton '+ nouveau groupe' comme déjà signalé (pas grave car il ne mène à rien) et aux onglets responsables et adhérents, les icônes de mofif vont sur des pages blanches.
Pour ce qui est de la #499 le cahier des charges me va bien, la fonctionnalité d’envoi de mails qui était demandée me parait indispensable pour les responsables de groupe, à part ça je ne vois pour le moment rien d'autre à ajouter à cette #499.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Fr cero a écrit :
J'ai patché mes fichiers (pas rapide depuis le git de ton dépôt,il faut copier ligne par ligne sinon on récupère les sauts de lignes et la numérotation !). Émargement, pdf, étiquetteé et cartes fonctionnent correctement.
Les fichiers patch sont destinés à être téléchargés et appliqués en local, soit ave cla commande patch
, soit via git :
git am fichier.patch
Les édition des fiches fonctionnent apparemment bien aussi.
Les bémols sont:
&) Dans gestion_adherents.php le filtre par groupe ne fonctionne pas correctement, la requête qui est envoyée devrait avoir une clause WHERE sur g.id_group me semble t'il.
Quand je filtre sur un groupe, j'ai :
WHERE (g.id_group = 3 OR gs.parent_group = NULL OR gs.parent_group = 3)
La requête me semble donc bonne. S'il y a un problème, il est ailleurs, et devra faire l'objet d'un nouveau ticket (ça n'a rienà voir avec le problème initial rapporté ici).
2) Dans gestion_groupes il y a encore le bouton '+ nouveau groupe' comme déjà signalé (pas grave car il ne mène à rien) et aux onglets responsables et adhérents, les icônes de mofif vont sur des pages blanches.
Je n'ai jamais dit que c'était corrigé ça :)
Mis à jour par Fr cero il y a environ 11 ans
Pour le filtre des groupes je suis d'accord, il fonctionne bien pour les admin et staff, il ne fonctionne pas pour un simple responsable de groupe (ni en recherche simple, ni en recherche avancée)
Je veux bien ouvrir un autre ticket pour cette anomalie.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Statut changé de In Progress à Résolu
Appliqué par commit 0f825b97e21902cb2f1c85deab0392e37097ffed.