Bug #22146
Préremplissage d'un champ "liste à choix multiples"
0%
Description
les listes à choix multiples sont pas pré-remplies non plus
Fichiers
Révisions associées
fields: allow prefilling of items fields (#22146)
Historique
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0002-fields-allow-prefilling-of-items-fields-22146.patch 0002-fields-allow-prefilling-of-items-fields-22146.patch ajouté
- Fichier 0001-backoffice-only-allow-python-as-prefilling-source-fo.patch 0001-backoffice-only-allow-python-as-prefilling-source-fo.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Un patch pour assurer le préremplissage, l'autre pour uniquement autoriser le mode "formule (python)".
Mis à jour par Thomas Noël il y a environ 6 ans
0001 :
On a d'un côté, actuellement, pour une carte :
prefill_kwargs = {'map': True}
(qui ne va autoriser que le pré-remplissage via geoloc)
et de l'autre, pour une liste à choix multiple :
prefill_kwargs = {'string': False, 'user': False, 'geolocation': False}
qui inverse la logique, et le jour où on a ajouter un nouveau type de prefill, on sera un peu dans l'ennui.
Ca m'ennuie d'avoir deux façons de faire, sans compter qu'on envoie ces kwargs dans CompositeWidget.__init__
Je préférerais au niveau de ItemsField un prefill_kwargs = {'formula': True}
voire même prefill_kwargs = {'items': True}
et dans le PrefillSelectionWidget :
... elif kwargs.get('items'): options = [('none', _('None')), ('formula', _('Formula (Python)'),]
0002 :
au niveau du if force_string:
je poserais bien un commentaire que explique que ret peut être un objet Python (genre une liste) pour itemsfield
Pour le self._parsed = True
je te fais totale confiance, pas réussi à remonter la piste de sa nécessité.
Mis à jour par Frédéric Péters il y a environ 6 ans
Je préférerais au niveau de ItemsField un prefill_kwargs = {'formula': True} voire même prefill_kwargs = {'items': True} et dans le PrefillSelectionWidget
À ce compte alors, passer un paramètre field=self et dans PrefillSelectionWidget la logique pour faire varier le contenu selon le type de champ; ça t'irait ?
Mis à jour par Thomas Noël il y a environ 6 ans
Frédéric Péters a écrit :
Je préférerais au niveau de ItemsField un prefill_kwargs = {'formula': True} voire même prefill_kwargs = {'items': True} et dans le PrefillSelectionWidget
À ce compte alors, passer un paramètre field=self et dans PrefillSelectionWidget la logique pour faire varier le contenu selon le type de champ; ça t'irait ?
Ouaip, c'est encore moins magique.
Mis à jour par Frédéric Péters il y a environ 6 ans
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 3b4d51a9d7e6e43eb0d92ff94d921f15832018d3 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Feb 28 10:42:42 2018 +0100 fields: allow prefilling of items fields (#22146) commit 44080b2fae454a04f303ea170e40726b5089e880 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Feb 28 10:42:00 2018 +0100 backoffice: only allow python as prefilling source for items field (#22146)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
backoffice: only allow python as prefilling source for items field (#22146)