Project

General

Profile

Actions

Anomalie #767

closed

Droits champs dynamiques

Added by Pierre-Yves DESAULLE over 10 years ago. Updated over 10 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
01/07/2014
Due date:
% Done:

100%

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


Files

display_dynamic_fields.tpl (2.07 KB) display_dynamic_fields.tpl Fichier modifié Pierre-Yves DESAULLE, 01/07/2014 11:05 AM
Actions #1

Updated by Johan Cwiklinski over 10 years ago

  • Assignee set to Johan Cwiklinski
  • Target version set to 0.8.0
  • Category set to Core
  • Status changed from Nouveau to 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>

Actions #2

Updated by Johan Cwiklinski over 10 years ago

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.

Actions #3

Updated by Johan Cwiklinski over 10 years ago

  • Subject changed from Anolamie Champs Dynamique to Droits champs dynamiques
Actions #4

Updated by Pierre-Yves DESAULLE over 10 years ago

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

Actions #5

Updated by Pierre-Yves DESAULLE over 10 years ago

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.

Actions #6

Updated by Johan Cwiklinski over 10 years ago

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.

Actions #7

Updated by Johan Cwiklinski over 10 years ago

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

Updated by Johan Cwiklinski over 10 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF