Project

General

Profile

Actions

Anomalie #1513

closed

Possibilité par un responsable de groupe de modifier un événement créé par un membre de bureau ou administrateur

Added by Alain Paris almost 4 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
11/06/2020
Due date:
% Done:

100%

Estimated time:
Version utilisée:

Description

Bonjour,

testé sur Galette0.9.4.2,plugin-events-hotfix-1.3.1 hotfix.

Je me connecte en tant que responsable de groupe, je ne peux pas modifier un événement créé par un autre que moi sur la page events/events(pas d’icône de modification).

Par contre si je passe par calendrier,Informations de l'évènement j'ai accès a events/event/edit/xyz et j'ai possibilité de modifier et d'enregistrer si je sélectionne mon groupe.

Alain


Files

resas responsable de groupe.jpg (128 KB) resas responsable de groupe.jpg vue du responsable d'un groupe Alain Paris, 12/07/2020 09:01 PM
resas adherent.jpg (109 KB) resas adherent.jpg vue d'un adhérent Alain Paris, 12/07/2020 09:05 PM
Actions #1

Updated by Johan Cwiklinski almost 4 years ago

J'ai ajouté une correction sur https://github.com/galette/plugin-events/pull/4 ; l'accès à la page ne sera plus autorisé - dans tous les cas ; mais il n'aurait pas du voir le lien non plus (ça, je ne suis pas sûr que ce soit corrigé).

Actions #2

Updated by Johan Cwiklinski almost 4 years ago

  • Status changed from Nouveau to In Progress
Actions #3

Updated by Alain Paris almost 4 years ago

Bonsoir,

J'ai appliqué le correctif sur les 4 fichiers ,si je n'ai pas fait d'erreur....le lien me renvoi sur une page vierge de création d'événement (events/event/edit/xyz). Ne pourrait on pas désactiver ce lien dans la pop-up pour les responsables de groupes + simple adhérents.

Les responsables de groupes pourraient toujours modifier a partir de la page événements (events/events)ou réservations (events/bookings/all).

De plus je ne sais si c'est un effet du correctif , sur la page Réservations (events/bookings/all) le nom des événements (concernant tous les adhérents) n'est plus renseigné.

Par contre, cela a peut être été demandé déjà, je pense que sur la page Evénements il serait intéressant d'avoir une fenêtre pop-up (identique a celle du calendrier pour avoir les infos complémentaires) lors de la sélection ou survol d'un événement.

Actions #4

Updated by Johan Cwiklinski almost 4 years ago

Je vais revoir ma copie, je me suis rendu compte que certaines choses étaient corrigées sur la branche de développement, et le hotfix/1.3.1 est lui basé sur la branche stable... Bref, j'ai perdu du temps à refaire des choses existantes, et je me suis emmêlé :(

Concernant le correctif... J'ai fait ça très vite, la seule chose qui est corrigée de manière certaine, c'est la possibilité pour le responsable de groupe d'effectivement modifier ce qu'il ne devrait pas. Ça ne devrait pas forcément renvoyer vers une page de création, mais à vrai dire on s'en fiche ; puisque normalement il ne devrait en effet pas y avoir de lien qui y mène non plus... En testant rapidement je ne l'ai pas vu ; mais je n'ai pas beaucoup d'évènements en base, je n'avais probablement pas le cas en question.

Je n'ai pas testé avec d'autres évènements, il est possible qu'une restriction soit effective et empêche l'affichage du nom, mais je ne crois pas que ça vienne de ce dernier correctif.

Updated by Alain Paris almost 4 years ago

Par contre,

J'ai fais des tests en partant de 2 galettes nightly avec le hotfix 1.3.1 mis a jour avec les derniers commits .
Pour la nightly Galette v0.9.5-git-5ec3ab5a24 (2020-12-06 12:44:04 GMT+0100) j'ai modifié _define.php dans le plugin(0.9.5 au lieu de 0.9.4 ligne 42)?.

l'affichage du nom des événements : ne sont concernés que les adhérents et les responsables de groupe.Il semble que maintenant l’affichage soit dépendant du carré de sélection.

Actions #6

Updated by Alain Paris almost 4 years ago

Bonsoir,

J'ai refait un test ce jour avec "plugin-events-develop" et "plugin-events-release-1.3.1", l'anomalie concernant le non affichage du nom des événements (colonne"événement") n'existe que sur la version 1.3.1.
Peut-être cela viens t'il de la correction concernant le filtrage Anomalie #1494 ?).

Actions #7

Updated by Johan Cwiklinski almost 3 years ago

  • Status changed from In Progress to Résolu
  • Assignee set to Johan Cwiklinski
  • Target version set to 1.3.2
  • % Done changed from 0 to 100
Actions #8

Updated by Johan Cwiklinski almost 3 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF