Development #56788
ajouter à la boite "limiter à un seul formulaire" l'info comme quoi ça concerne les usagers connectés
0%
Description
Quelques mots, peut-être en niveau warning, quand l'accès anonyme au formulaire est permis.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Lié à Documentation #56782: Préciser que l'option "limiter à un seul formulaire" n'est effective que lorsque le formulaire est restreint aux utilisateurs authentifiés ou aux utilisateurs possédant le rôle truc ajouté
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0001-admin-warning-on-only_allow_one-field-56788.patch 0001-admin-warning-on-only_allow_one-field-56788.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 2 ans
L'option « always_advertise » va juste faire afficher un lien vers le formulaire y compris si celui-ci n'est pas accessible (typiquement : user anonyme et formulaire reservé au role xx), mais ça ne permet pas l'accès au formulaire, il faudra se logguer et avoir le bon rôle. Donc je pense que ce n'est pas la peine de prendre en compte cette option always_advertise.
Aussi, je me dis que si on supprime le rôle expéditeur par la suite, et que donc le formulaire redevient "anonyme", ce warning n’apparaîtra pas.
J'en serais presque à dire que l'option « limiter à un seul formulaire » ne devrait fonctionner que pour les formulaires avec un rôle expéditeur (ie nécessitant un login). Et comme ça, voilà, même pas besoin de warning, l'option ne sera pas là. Mais c'est sans doute un peu violent ? (faudra regarder si on a besoin de formulaires "anonyme" + "limités à 1 demande" sur notre SaaS, j'ai dans l'idée que c'est négligeable car c'est un bogue d'usage, en fait)
Ou alors je rate un truc, aussi :)
Mis à jour par Frédéric Péters il y a plus de 2 ans
J'en serais presque à dire que l'option « limiter à un seul formulaire » ne devrait fonctionner que pour les formulaires avec un rôle expéditeur (ie nécessitant un login). Et comme ça, voilà, même pas besoin de warning, l'option ne sera pas là ? Mais c'est sans doute un peu violent ?
À un moment au niveau de GNOME/etc. il y avait le propos comme quoi ça restait mieux de présenter une option, avec explication, plutôt que la cacher, parce qu'une fois cachée l'usager n'a aucune indication quand il se trouve à la chercher ("je suis sûr qu'il y avait cette option, pourquoi n'apparait-elle pas ?"). (on a ce problème à certains endroits d'ailleurs je devrais faire des tickets).
Aussi ça passe à côté de ton commentaire dix mots plus haut "Aussi, je me dis que si on supprime le rôle expéditeur par la suite, et que donc le formulaire redevient "anonyme", ce warning n’apparaîtra pas.".
Voilà, mon idée ici était d'offrir l'information à un endroit directement lisible, plutôt qu'après erreur de configuration pouvoir dire "c'était pourtant dans la doc" (à peu près #56782).
Mis à jour par Thomas Noël il y a plus de 2 ans
Plutôt d'accord sur l'argument contre le fait de cacher des options.
Et donc, autre idée : ajouter un petit panneau de warning /!\ à côté de l'option "Limiter à un seul formulaire" (qui s'affiche sur la page de config du formulaire), quand elle est à "Oui" mais quel le formulaire est ouvert aux anonymes. Et quand on passe sur le petit panneau ou qu'on clique dessus, tooltip qui dit "Attention, la limitation à un seul formulaire n'est valable que pour les usagers connectés". On n'a encore nulle part je crois ce genre de petit warning, mais c'est devenu un truc assez classique (quoique j'en ai jamais vu dans Gnome je crois bien ;) )
Mis à jour par Frédéric Péters il y a plus de 2 ans
On n'a encore nulle part je crois ce genre de petit warning, mais c'est devenu un truc assez classique.
Je veux bien qu'on fasse ce ticket sans ça, et un autre pour réfléchir à cet aspect (j'ai cherché un peu sans trouver de captures d'écrans ou de passages de documentation ux sur le sujet, et je voudrais réfléchir à ça au-delà de cette option (ex: aussi l'appliquer sur les actions de workflows pas configurées ?, aussi monter le marqueur d'erreur au niveau supérieur, pour attirer l'attention sur le formulaire où elle se trouve ?, uniquement des erreurs ou des avertissements également ? etc.) (je viens de créer #57080).
~~
Pour en venir au patch ici, en effet pas à lier à "always_advertise".
Mis à jour par Thomas Noël il y a plus de 2 ans
Frédéric Péters a écrit :
Pour en venir au patch ici, en effet pas à lier à "always_advertise".
Yep, simplifions ce ticket pour l'instant et donc oui, juste cette correction, Lauréline.
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
L'option « always_advertise » va juste faire afficher un lien vers le formulaire y compris si celui-ci n'est pas accessible
Haaa, c'est donc ça ! J'en découvre tous les jours :)
ok je reprends le patch
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Fichier 0001-admin-warning-on-only_allow_one-field-56788.patch 0001-admin-warning-on-only_allow_one-field-56788.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Thomas Noël il y a plus de 2 ans
Et le pénible de service en remet une couche : « Warning: this option concerns logged in users only. » → « Warning: this option concerns logged in users only, however this form is accessible anonymously. Consider adding an sender role. »
Mis à jour par Lauréline Guérin il y a plus de 2 ans
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d9b0f3ba33920ea7f9887e5d523092335b47cfed Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Sep 17 15:23:04 2021 +0200 admin: warning on only_allow_one field (#56788)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
admin: warning on only_allow_one field (#56788)