Project

General

Profile

Anomalie #686

mauvais adhérent

Added by Frédéric PAVIOT over 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
-
Start date:
08/26/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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


Files


Related issues

Precedes Galette - Anomalie #709: Les responsables des groupes ne peuvent modifier la composition mais le bouton est actifFermé09/18/2013

Actions
Precedes Galette - Anomalie #712: les responsables de groupes ne peuvent pas filtrer les adhérents par groupeFermé09/30/2013

Actions

Associated revisions

Revision a2ca5fbb (diff)
Added by Johan Cwiklinski about 6 years ago

Group managers cannot edit members they own; fixes #686

Revision 0f825b97 (diff)
Added by Johan Cwiklinski about 6 years ago

Fix various issues regarding groups management; fixes #686

Revision 758271e7 (diff)
Added by Johan Cwiklinski about 6 years ago

Groups manager rights, refs #499, #686

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

History

#1

Updated by Johan Cwiklinski about 6 years ago

  • Assignee set to Johan Cwiklinski
  • Category set to Core
#2

Updated by Johan Cwiklinski about 6 years ago

  • File 0001-Group-managers-cannot-edit-members-they-own-fixes-68.patch added

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

Updated by Roland Telle about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • % Done changed from 0 to 100
  • Status changed from Nouveau to Résolu
#5

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé
#6

Updated by Johan Cwiklinski about 6 years ago

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

Updated by Fr cero about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Fermé to 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

Updated by Johan Cwiklinski about 6 years ago

  • File admin.patch added

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

Updated by Fr cero about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • File deleted (0001-Group-managers-cannot-edit-members-they-own-fixes-68.patch)
#13

Updated by Johan Cwiklinski about 6 years ago

  • File deleted (admin.patch)
#14

Updated by Fr cero about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

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

Updated by Fr cero about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from In Progress to Résolu
#18

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF