Projet

Général

Profil

Actions

Anomalie #686

fermé

mauvais adhérent

Ajouté par Frédéric PAVIOT il y a plus de 10 ans. Mis à jour il y a plus de 10 ans.

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

100%

Temps estimé:
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


Fichiers


Demandes liées 2 (0 ouverte2 fermées)

Précède Galette - Anomalie #709: Les responsables des groupes ne peuvent modifier la composition mais le bouton est actifFerméJohan Cwiklinski18/09/2013

Actions
Précède Galette - Anomalie #712: les responsables de groupes ne peuvent pas filtrer les adhérents par groupeFerméJohan Cwiklinski30/09/2013

Actions

Mis à jour par Johan Cwiklinski il y a plus de 10 ans

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

Mis à jour par Johan Cwiklinski il y a plus de 10 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 10 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 10 ans

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

Mis à jour par Johan Cwiklinski il y a plus de 10 ans

  • Statut changé de Résolu à Fermé

Mis à jour par Johan Cwiklinski il y a plus de 10 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 plus de 10 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 plus de 10 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 plus de 10 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 plus de 10 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 plus de 10 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.

Mis à jour par Johan Cwiklinski il y a plus de 10 ans

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

Mis à jour par Johan Cwiklinski il y a plus de 10 ans

  • Fichier admin.patch supprimé

Mis à jour par Fr cero il y a plus de 10 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 plus de 10 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 plus de 10 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 plus de 10 ans

  • Statut changé de In Progress à Résolu

Mis à jour par Johan Cwiklinski il y a plus de 10 ans

  • Statut changé de Résolu à Fermé
Actions

Formats disponibles : Atom PDF