Development #7872
Déplacer l'option "always advertise" vers la liste des options
0%
Description
Pour le moment l'option est présente dans la fenêtre de sélection du rôle de l'usager; elle pourrait être déplacée vers la liste des options, d'une part c'est plutôt logique, d'autre part ça permettra d'aller vers une boite de sélection de rôle plus générique (elle pourra notamment être partagée avec la section des rôles habilités à faire la saisie en backoffice).
Fichiers
Révisions associées
admin: fix setting of always_advertise (#7872)
Historique
Mis à jour par Frédéric Péters il y a presque 9 ans
- Fichier 0001-admin-move-always-advertise-to-list-of-options-7872.patch 0001-admin-move-always-advertise-to-list-of-options-7872.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 9 ans
Je trouve un peu bof le :
+ if not self.formdef.roles: + kwargs['disabled'] = 'disabled'
parce que ça ne me dit pas pourquoi je ne peux pas cliquer sur la case... et que si je sais pourquoi, j'aimerai bien cliquer quand même dessus (parce que je mettrais les rôles après). Autre cas : je l'avais activé, et j'ai finalement ouvert le formulaire au public, et la case est grisée et cochée, c'est bizarre.
(en gros, les tests if formdef.roles, je m'en passerai)
Ack, pour le reste.
Mis à jour par Frédéric Péters il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
Voilà, avec la suppression de la mise en disabled.
commit fe0e75ce5994680e2d5f484e76e7400a5c9dd6b2
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Jul 19 14:55:34 2015 +0200
admin: move "always advertise" to list of options (#7872)
admin: move "always advertise" to list of options (#7872)