Galette

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.

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

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.

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.