Project

General

Profile

Anomalie #1201

Champs dynamiques Permission et Visibilité ne semble pas fonctionner

Added by Alain Paris 10 months ago. Updated 9 months ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
09/22/2018
Due date:
% Done:

100%

Estimated time:
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


Files

affichage Champs Dyn galette v8.3.jpg (270 KB) affichage Champs Dyn galette v8.3.jpg affichage en 8.3 Alain Paris, 10/26/2018 04:55 PM
affichage Champs Dyn galette v9.2dev.jpg (353 KB) affichage Champs Dyn galette v9.2dev.jpg affichage en 9.2 dev Alain Paris, 10/26/2018 04:55 PM
dynamic_fields_acls.patch (3.96 KB) dynamic_fields_acls.patch Johan Cwiklinski, 10/31/2018 07:41 AM
erreur adherent.png (54.2 KB) erreur adherent.png Alain Paris, 10/31/2018 10:43 AM
erreur contribution.png (54.5 KB) erreur contribution.png Alain Paris, 10/31/2018 10:43 AM
erreur transac.png (54.7 KB) erreur transac.png Alain Paris, 10/31/2018 10:43 AM
comportement a l_enregistrement fiche adherent .jpg (332 KB) comportement a l_enregistrement fiche adherent .jpg Alain Paris, 10/31/2018 06:41 PM
fix-1201.patch (8.65 KB) fix-1201.patch Johan Cwiklinski, 11/05/2018 06:07 AM

Associated revisions

Revision 2b96ac95 (diff)
Added by Johan Cwiklinski 9 months ago

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

History

#1

Updated by Alain Paris 9 months ago

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

Updated by Johan Cwiklinski 9 months ago

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

Updated by Alain Paris 9 months ago

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

Updated by Johan Cwiklinski 9 months ago

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

Updated by Alain Paris 9 months ago

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

Updated by Johan Cwiklinski 9 months ago

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

Updated by Johan Cwiklinski 9 months ago

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

#8

Updated by Alain Paris 9 months ago

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

Updated by Johan Cwiklinski 9 months ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
#10

Updated by Johan Cwiklinski 9 months ago

  • Status changed from Résolu to 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.

Also available in: Atom PDF