Projet

Général

Profil

Anomalie #686

mauvais adhérent

Ajouté par Frédéric PAVIOT il y a environ 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
-
Début:
26/08/2013
Echéance:
% réalisé:

100%

Version utilisée:

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

0001-Fix-various-issues-regarding-groups-management-fixes.patch Magnifier (6,17 ko) Johan Cwiklinski, 29/09/2013 12:17

0002-Groups-manager-rights-refs-499-686.patch Magnifier (3,74 ko) Johan Cwiklinski, 29/09/2013 12:17


Demandes liées

Précède Galette - Anomalie #709: Les responsables des groupes ne peuvent modifier la composition mais le bouton est actif Fermé 18/09/2013
Précède Galette - Anomalie #712: les responsables de groupes ne peuvent pas filtrer les adhérents par groupe Fermé 30/09/2013

Révisions associées

Révision a2ca5fbb
Ajouté par Johan Cwiklinski il y a environ 4 ans

Group managers cannot edit members they own; fixes #686

Révision 0f825b97
Ajouté par Johan Cwiklinski il y a presque 4 ans

Fix various issues regarding groups management; fixes #686

Révision 758271e7
Ajouté par Johan Cwiklinski il y a presque 4 ans

Groups manager rights, refs #499, #686

Disable delete, mailings and csv exports buttons on members list.
Allow attendance sheet, labels and card generation.

Historique

#1 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Assigné à mis à Johan Cwiklinski
  • Catégorie mis à Core

#2 Mis à jour par Johan Cwiklinski il y a environ 4 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.

#3 Mis à jour par Roland Telle il y a environ 4 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 ?

#4 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • % réalisé changé de 0 à 100
  • Statut changé de Nouveau à Résolu

#5 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Statut changé de Résolu à Fermé

#6 Mis à jour par Johan Cwiklinski il y a environ 4 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).

#7 Mis à jour par Fr cero il y a presque 4 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é.

#8 Mis à jour par Johan Cwiklinski il y a presque 4 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.

#9 Mis à jour par Johan Cwiklinski il y a presque 4 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.

#10 Mis à jour par Fr cero il y a presque 4 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...; ! :)

#11 Mis à jour par Johan Cwiklinski il y a presque 4 ans

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.

#12 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Fichier 0001-Group-managers-cannot-edit-members-they-own-fixes-68.patch supprimé

#13 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Fichier admin.patch supprimé

#14 Mis à jour par Fr cero il y a presque 4 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.

#15 Mis à jour par Johan Cwiklinski il y a presque 4 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 :)

#16 Mis à jour par Fr cero il y a presque 4 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.

#17 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de In Progress à Résolu

#18 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF