Projet

Général

Profil

Actions

Anomalie #767

fermé

Droits champs dynamiques

Ajouté par Pierre-Yves DESAULLE il y a environ 10 ans. Mis à jour il y a environ 10 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
Début:
07/01/2014
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:

Description

Lors de l'affichage des champs dynamique la visibilité n'est pas bien pris en compte. Voici le fichier modifié pour prendre en compte la visibilité.
Je suis parti du principe que :
1 Champs avec visibilité TOUS est visible par tout le monde
1 Champs avec visibilité STAFF est visible par les membres du STAFF et ADMIN
1 Champs avec visibilité ADMIN est visible par les membres ADMIN

Le fichier modifié est "display_dynamic_fields.tpl"


Fichiers

display_dynamic_fields.tpl (2,07 ko) display_dynamic_fields.tpl Fichier modifié Pierre-Yves DESAULLE, 07/01/2014 11:05

Mis à jour par Johan Cwiklinski il y a environ 10 ans

  • Assigné à mis à Johan Cwiklinski
  • Version cible mis à 0.8.0
  • Catégorie mis à Core
  • Statut changé de Nouveau à In Progress

À priori, l'admin a toujours le droit de voir les champs, pas la peine de vérifier quel est la permission enregistrée dans ce cas là. Le diff final devrait être :

--- a/galette/templates/default/display_dynamic_fields.tpl
+++ b/galette/templates/default/display_dynamic_fields.tpl
@@ -2,7 +2,7 @@
 <table class="details">
     <caption class="ui-state-active ui-corner-top">{_T string="Additionnal fields:"}</caption>
     {foreach from=$dynamic_fields item=field}
-        {if $field.field_perm ne 1 || $login->isAdmin() || $login->isStaff()}
+        {if $field.field_perm eq 0 || $login->isAdmin() || $login->isStaff() && $field.field_perm eq 2}
             {if $field.field_type eq 0}
     <tr>
         <th class="separator" colspan="2">{$field.field_name|escape}</th>

Mis à jour par Johan Cwiklinski il y a environ 10 ans

Et il faut faire la même chose lors de l'édition :

diff --git a/galette/templates/default/edit_dynamic_fields.tpl b/galette/templates/default/edit_dynamic_fields.tpl
index c7980a7..2f3b314 100644
--- a/galette/templates/default/edit_dynamic_fields.tpl
+++ b/galette/templates/default/edit_dynamic_fields.tpl
@@ -3,7 +3,7 @@
     <legend class="ui-state-active ui-corner-top">{_T string="Additionnal fields:"}</legend>
     <div>
 {foreach from=$dynamic_fields item=field}
-{if $field.field_perm ne 1 || $login->isAdmin() || $login->isStaff()}
+{if $field.field_perm eq 0 || $login->isAdmin() || $login->isStaff() && $field.field_perm eq 2}
     {if $field.field_type eq 0}
         <div class="separator">{$field.field_name|escape}</div>
     {else}

J'attends une validation avant d'intégrer ces modifications dans Galette.

Mis à jour par Johan Cwiklinski il y a environ 10 ans

  • Sujet changé de Anolamie Champs Dynamique à Droits champs dynamiques

Mis à jour par Pierre-Yves DESAULLE il y a environ 10 ans

J'ai fais exprès de laisser le teste sur Admin et la permission du champs dans le cas ou tu n'étais pas d'accord avec mon point de vus expliqué ci-dessus :
"
Je suis parti du principe que :
1 Champs avec visibilité TOUS est visible par tout le monde
1 Champs avec visibilité STAFF est visible par les membres du STAFF et ADMIN
1 Champs avec visibilité ADMIN est visible par les membres ADMIN
"

J'ai voulu tester le patch et pour cela (et rendre plus facile la mise a dispo de patch etc .. ) je viens de cloner la branche master en local pour pouvoir le tester mais dès l’installation je suis bloqué il manque des "includes" :
../includes/password_compat-1.0.3
../includes/phpMailer-5.2.6
../includes/Smarty-3.1.14

Idem pour la branche de Développement (En comparant avec ma version 0.7.8 en production, il y a d'autre "includes" qui manque).

Je ne sais pas si c'est une erreur de ma part ou une anomalie. Dans le doute je le poste ici (Cela m'étonnerait que ce soit une anomalie ... mais plutôt une erreur de ma part).

Pierre-Yves

Mis à jour par Pierre-Yves DESAULLE il y a environ 10 ans

Pierre-Yves DESAULLE a écrit :

J'ai fais exprès de laisser le teste sur Admin et la permission du champs dans le cas ou tu n'étais pas d'accord avec mon point de vus expliqué ci-dessus :
"
Je suis parti du principe que :
1 Champs avec visibilité TOUS est visible par tout le monde
1 Champs avec visibilité STAFF est visible par les membres du STAFF et ADMIN
1 Champs avec visibilité ADMIN est visible par les membres ADMIN
"

J'ai voulu tester le patch et pour cela (et rendre plus facile la mise a dispo de patch etc .. ) je viens de cloner la branche master en local pour pouvoir le tester mais dès l’installation je suis bloqué il manque des "includes" :
../includes/password_compat-1.0.3
../includes/phpMailer-5.2.6
../includes/Smarty-3.1.14

Idem pour la branche de Développement (En comparant avec ma version 0.7.8 en production, il y a d'autre "includes" qui manque).

Je ne sais pas si c'est une erreur de ma part ou une anomalie. Dans le doute je le poste ici (Cela m'étonnerait que ce soit une anomalie ... mais plutôt une erreur de ma part).

Pierre-Yves

Je confirme les modifications, je vient de les tester sur ma version 0.7.7 en local.

Mis à jour par Johan Cwiklinski il y a environ 10 ans

Pierre-Yves DESAULLE a écrit :

J'ai voulu tester le patch et pour cela (et rendre plus facile la mise a dispo de patch etc .. ) je viens de cloner la branche master en local pour pouvoir le tester mais dès l’installation je suis bloqué il manque des "includes" :
[...]
Je ne sais pas si c'est une erreur de ma part ou une anomalie. Dans le doute je le poste ici (Cela m'étonnerait que ce soit une anomalie ... mais plutôt une erreur de ma part).

C'est marqué dans la doc :p

Les bibliothèques utilisées ne sont pas intégrées dans le dépôt, il faut les récupérer à part. L'archive fournit uniquement les dépendances pour le branche develop.

Pierre-Yves DESAULLE a écrit :

Je confirme les modifications, je vient de les tester sur ma version 0.7.7 en local.

Ok, merci. Je vais donc intégrer ces modifications, qui feront partie de la prochaine release.

Mis à jour par Johan Cwiklinski il y a environ 10 ans

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

Mis à jour par Johan Cwiklinski il y a environ 10 ans

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

Formats disponibles : Atom PDF