Projet

Général

Profil

Development #56788

ajouter à la boite "limiter à un seul formulaire" l'info comme quoi ça concerne les usagers connectés

Ajouté par Frédéric Péters il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Quelques mots, peut-être en niveau warning, quand l'accès anonyme au formulaire est permis.


Fichiers


Demandes liées

Lié à Publik - 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 trucFermé08 septembre 2021

Actions

Révisions associées

Révision d9b0f3ba (diff)
Ajouté par Lauréline Guérin il y a plus de 2 ans

admin: warning on only_allow_one field (#56788)

Historique

#1

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é
#2

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Assigné à mis à Lauréline Guérin
#3

Mis à jour par Lauréline Guérin il y a plus de 2 ans

#4

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 :)

#5

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).

#6

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 ;) )

#7

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".

#8

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.

#9

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

#11

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. »

#13

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
#14

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)
#15

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

Formats disponibles : Atom PDF