Development #39167
option "lecture seule"
0%
Description
Pour qu'un champ, prérempli côté serveur, ne puisse pas être édité côté client.
Fichiers
Demandes liées
Révisions associées
forms: add option for locking prefilled fields (#39167)
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Lié à Development #27429: Permettre les champs lecture seule / caché / etc. ajouté
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-forms-add-option-for-prefilling-fields-as-readonly-3.patch 0001-forms-add-option-for-prefilling-fields-as-readonly-3.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Question vocabulaire, je me demande si ce serait plus clair d'avoir "verrouillé" que "lecture seule" ?
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Frédéric Péters a écrit :
Question vocabulaire, je me demande si ce serait plus clair d'avoir "verrouillé" que "lecture seule" ?
Je préfère aussi verrouillé.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- la modif à checkbox_render_content() devrait aller dans son propre patch
- la modif à qommon.form.js et dans formdef.py je n'ai pas l'impression que ce soit lié à cette case à cocher, c'est juste l'ajout du support de la mise à jour live pour les champs autre que item et comment non ?
- pour
prefill_init_with_xmlforce_str n'est plus nécessaire
Mis à jour par Frédéric Péters il y a environ 4 ans
la modif à qommon.form.js et dans formdef.py je n'ai pas l'impression que ce soit lié à cette case à cocher, c'est juste l'ajout du support de la mise à jour live pour les champs autre que item et comment non ?
C'est lié, c'est pour, sur ces champs verrouillés, avoir une valeur actualisée dynamiquement.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Frédéric Péters a écrit :
C'est lié, c'est pour, sur ces champs verrouillés, avoir une valeur actualisée dynamiquement.
Ok, par contre pourquoi est-ce limité au cas prefill=string et pas formula ?
Mis à jour par Frédéric Péters il y a environ 4 ans
Parce que les évaluations live se font uniquement pour les gabarits/conditions django.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Frédéric Péters a écrit :
Parce que les évaluations live se font uniquement pour les gabarits/conditions django.
Ok mais techniquement rien n'empêcherait de le permettre aussi pour les expressions Python.
Mis à jour par Thomas Noël il y a environ 4 ans
(relecture de la branche https://git.entrouvert.org/wcs.git/log/?h=wip/39167-locked
Au niveau de wcs/fields.py dans Field::get_prefill_value quand il y a un soucis de template ou pas de valeur, on renvoie (None, False). Est-ce qu'on ne voudrait pas quand même locker le champ et renvoyer (None, explicit_locked) ? (sauf pour geolocation où ça sera toujours (None, False)) Histoire que le champ soit bien "vidé" dans ce cas lors du reset_locked_data. En disant ça, ça me semble être surtout utile lors du except TemplateError
Mini détail, sur reset_locked_data et live_process_fields il reste des « value, verified = ..get_prefill_value » qui pourraient être renommées « value, locked = ... »
Le reste me semble ok.
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0bd57943ebf1beb4574bfc8d48e3e0d48c5c88e5
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 22 10:28:45 2020 +0100
forms: add option for locking prefilled fields (#39167)
commit b4e3b767c7ff377237deb95596cbc7334d8e3e1b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 22 10:28:45 2020 +0100
forms: add support for checkbox with inline label (#39167)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: add support for checkbox with inline label (#39167)