Projet

Général

Profil

Anomalie #1201

Champs dynamiques Permission et Visibilité ne semble pas fonctionner

Ajouté par Alain Paris il y a 4 mois. Mis à jour il y a 2 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
Début:
22/09/2018
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:
Vote:

Description

Bonjour,

Il semble qu'il y est un bug
Les champs dynamiques (Fiche adhérent) sont visibles et modifiables par tous quel que soit la configuration de ceux ci à leur création chez moi.
Pour les contributions et transactions je ne sais pas si le comportement est le même les adhérents hors bureau ne visualisant pas le détail de leur contribution/transaction.

Testé sur 9.1.2 et version nightly

AParis


Fichiers

affichage Champs Dyn galette v8.3.jpg (270 ko) affichage Champs Dyn galette v8.3.jpg affichage en 8.3 Alain Paris, 26/10/2018 16:55
affichage Champs Dyn galette v9.2dev.jpg (353 ko) affichage Champs Dyn galette v9.2dev.jpg affichage en 9.2 dev Alain Paris, 26/10/2018 16:55
dynamic_fields_acls.patch (3,96 ko) dynamic_fields_acls.patch Johan Cwiklinski, 31/10/2018 07:41
erreur adherent.png (54,2 ko) erreur adherent.png Alain Paris, 31/10/2018 10:43
erreur contribution.png (54,5 ko) erreur contribution.png Alain Paris, 31/10/2018 10:43
erreur transac.png (54,7 ko) erreur transac.png Alain Paris, 31/10/2018 10:43
comportement a l_enregistrement fiche adherent .jpg (332 ko) comportement a l_enregistrement fiche adherent .jpg Alain Paris, 31/10/2018 18:41
fix-1201.patch (8,65 ko) fix-1201.patch Johan Cwiklinski, 05/11/2018 06:07

Révisions associées

Révision 2b96ac95 (diff)
Ajouté par Johan Cwiklinski il y a 2 mois

Do not display dynamic fields if current user do not have proper ACLs; closes #1201

Historique

#1

Mis à jour par Alain Paris il y a 3 mois

Bonjour,

Je reviens sur la demande précédente.

Je viens de m'apercevoir que les permissions ne jouent en fait que sur le contenu des champs contrairement a la version 0.8.

Si les permissions sont bureau ou administrateur:

Sur la version 0.8 , le simple adhérent ne voit pas le champ supplémentaire dans sa fiche ni en cas de modification de celle fiche,.

Dans la version 0.9 le simple adhérent voit les champs supplémentaires avec leur nom dans sa fiche seul les contenus sont vides.Dans modification idem les champs existent comme s'ils étaient modifiable ,ils ne s'enregistrent toutefois pas.

Du fait on ne peut avoir comme dans la version galette 0.8 de champs dynamiques réservé au bureau ou administrateur...

Je pense que cela doit être une anomalie.

AParis

#2

Mis à jour par Johan Cwiklinski il y a 3 mois

En effet, je suis d'accord : ces champs ne devraient pas être affichés, ils seront toujours vides de toutes façons. Et c'est une bonne chose que les données soient correctement gérées d'autre part :D

Ci-joint, une proposition de correctif que je n'ai pas eu le temps de tester dans tous les cas de figure (ils sont nombreux... :'( )

#3

Mis à jour par Alain Paris il y a 3 mois

Bonjour,

Je ne sais comment utiliser le "patch" j'ai donc modifié a la main les 4 fichiers ....

les champs apparaissent bien en fonction des statuts des membres mais ...

Les enregistrements des fiches adhérents ,contributions,transactions...provoquent tous une erreur .....(que ce soit une création ou une modification) et que l'on ai modifié ou pas un champ.

Voir fichiers joints...

#4

Mis à jour par Johan Cwiklinski il y a 3 mois

Alain Paris a écrit :

Je ne sais comment utiliser le "patch" j'ai donc modifié a la main les 4 fichiers ....

Il semble exister différentes méthodes sous windows pour ce faire, une rapide recherche devrait répondre à la question.

Ça fonctionne bien aussi, mais le risque est d'oublier quelque chose ou de se tromper... :)

les champs apparaissent bien en fonction des statuts des membres mais ...

Les enregistrements des fiches adhérents ,contributions,transactions...provoquent tous une erreur .....(que ce soit une création ou une modification) et que l'on ai modifié ou pas un champ.

Ha, en effet, j'ai oublié un endroit ; il faut en plus effectuer la modification suivante :

diff --git a/galette/lib/Galette/Entity/DynamicFieldsHandle.php b/galette/lib/Galette/Entity/DynamicFieldsHandle.php
index bdd9af91..881341d1 100644
--- a/galette/lib/Galette/Entity/DynamicFieldsHandle.php
+++ b/galette/lib/Galette/Entity/DynamicFieldsHandle.php
@@ -386,7 +386,7 @@ class DynamicFieldsHandle
      */
     private function handleRemovals()
     {
-        $fields = new DynamicFieldsSet($this->zdb);
+        $fields = new DynamicFieldsSet($this->zdb, $this->login);
         $this->dynamic_fields = $fields->getList($this->form_name, $this->login);

         $select = $this->zdb->select(self::TABLE, 'd');

Voir fichiers joints...

#5

Mis à jour par Alain Paris il y a 3 mois

Après remplacement de la ligne, il n'y a plus d'erreur mais lors de l’enregistrement d'une fiche adhérent, même sans modification il y a suppression des données des champs concernant les autres statuts.

#6

Mis à jour par Johan Cwiklinski il y a 3 mois

Arf :/ Merci pour les retours ; je regarderai tout ça de plus près quand j'en aurai le loisir ; je m'absente quelques jours.

#7

Mis à jour par Johan Cwiklinski il y a 2 mois

Ci-joint, un nouveau patch qui devrait corriger le tout cette fois.

#8

Mis à jour par Alain Paris il y a 2 mois

Bonjour,

merci avec la nouvelle modification cela semble fonctionner correctement....après les quelques essais que j'ai effectué...

Je viens de découvrir par hasard une anomalie sur les versions nightly (  testé depuis celle  du 24 juillet (la nightly du 01 juin étant OK).)

Le fait de choisir la langue allemande que ce soit sur la page d’accueil ,ou les autres pages en appuyant sur le "drapeau" allemand me plante le navigateur et pas de message d'erreur.

A voir si je suis le seul concerné ou s'il faut ouvrir un ticket ?

#9

Mis à jour par Johan Cwiklinski il y a 2 mois

  • Statut changé de In Progress à Résolu
  • % réalisé changé de 0 à 100
#10

Mis à jour par Johan Cwiklinski il y a 2 mois

  • Statut changé de Résolu à Fermé

Merci pour les retours.

Je pense avoir trouvé la problème avec l'allemand (et il y a bien un message d'erreur) ; le problème principal est que la traduction allemande n'est pas suivie :/

Je viens de re-générer une nigthly dans laquelle le problème devrait être corrigé. Si ça persiste, merci d'ouvrir un nouveau ticlket.

Formats disponibles : Atom PDF