Projet

Général

Profil

Development #19521

Vider les variables "non accessibles" à la validation finale d'une demande

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
18 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

Aujourd'hui on peut avancer une page conditionnelle, la remplir, retourner en arrière, modifier un champ, repartir de l'avant et ne plus être sur la page conditionnelle mais sur une autre, etc. Et sur des situations comme ça on voit souvent ensuite des utilisations genre =vars().get('form_var_champ_de_la_page1') or vars().get('form_var_champ_de_la_page2') mais c'est un raccourci, pour être correcte elle devrait reprendre la condition d'accès à la page.

Du coup, à la validation, je me dis qu'on pourrait conserver uniquement les données des champs des pages visibles.

Révisions associées

Révision 96f57e96 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

tests: check fields from undisplayed conditional pages are not saved (#19521)

Historique

#1

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

  • Description mis à jour (diff)
#2

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

  • Description mis à jour (diff)
#3

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

Autre explication peut-être plus nette : une page de formulaire, une case à cocher [ ] prévenir une personne tierce; une deuxième page "email du tiers", on la complète, retourne en arrière, décoche, et dans le workflow pourtant on va faire un envoi de mail à form_var_email_tiers, qui va exister parce que la page a été visitée et remplie à un moment.

#4

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)

Et contrairement à ce que j'imaginais et répétais depuis longtemps, ça fonctionne déjà.

commit 96f57e96e37dc613af4eaa9aa58449a89bbf20ee
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun May 27 11:51:10 2018 +0200

    tests: check fields from undisplayed conditional pages are not saved (#19521)
#5

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

Formats disponibles : Atom PDF