Project

General

Profile

Development #39167

option "lecture seule"

Added by Frédéric Péters 2 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
22 Jan 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Pour qu'un champ, prérempli côté serveur, ne puisse pas être édité côté client.

0001-forms-add-option-for-prefilling-fields-as-readonly-3.patch View (20.9 KB) Frédéric Péters, 25 Jan 2020 03:54 PM


Related issues

Related to Publik - Development #27429: Permettre les champs lecture seule / caché / etc. En cours 19 Oct 2018

Associated revisions

Revision b4e3b767 (diff)
Added by Frédéric Péters about 1 month ago

forms: add support for checkbox with inline label (#39167)

Revision 0bd57943 (diff)
Added by Frédéric Péters about 1 month ago

forms: add option for locking prefilled fields (#39167)

History

#1 Updated by Frédéric Péters 2 months ago

#2 Updated by Frédéric Péters 2 months ago

Question vocabulaire, je me demande si ce serait plus clair d'avoir "verrouillé" que "lecture seule" ?

#3 Updated by Benjamin Dauvergne 2 months ago

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

#4 Updated by Benjamin Dauvergne about 2 months ago

  • 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_xml force_str n'est plus nécessaire

#5 Updated by Frédéric Péters about 2 months ago

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.

#6 Updated by Benjamin Dauvergne about 2 months ago

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 ?

#7 Updated by Frédéric Péters about 2 months ago

Parce que les évaluations live se font uniquement pour les gabarits/conditions django.

#8 Updated by Benjamin Dauvergne about 2 months ago

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.

#9 Updated by Thomas Noël about 1 month ago

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

#10 Updated by Frédéric Péters about 1 month ago

Modifications apportées à la branche.

#11 Updated by Thomas Noël about 1 month ago

  • Status changed from Solution proposée to Solution validée

#12 Updated by Frédéric Péters about 1 month ago

  • Status changed from Solution validée to 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)

#13 Updated by Frédéric Péters about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF