Development #68197
dans l'évaluation des champs "source live" on considère aussi les champs d'autres pages
Début:
16 août 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
La méthode set_live_condition_sources() pose un attribut sur les champs qui peuvent être source de modifications dynamiques sur d'autres champs; historiquement c'était :
def set_live_condition_sources(self, form, fields): live_condition_fields = {} for field in fields:
Ça a été changé dans #51688 pour la gestion des blocs de champs, pour faire
- for field in fields: + def get_all_fields(): + for field in fields: + yield field + if field.key == 'block': + for subfield in field.block.fields: + yield subfield + + for field in get_all_fields():
L'itération sur les champs a ensuite été factorisée, #60706.
Et à ce moment on est passé de considérer les champs de la page à considérer tous les champs de la démarche.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus d'un an
- Fichier 0001-forms-do-not-consider-fields-from-other-pages-for-li.patch 0001-forms-do-not-consider-fields-from-other-pages-for-li.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Lauréline Guérin il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 3385a003d0de8b48b466cf466c117ed1918056c0 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Aug 16 18:25:18 2022 +0200 forms: do not consider fields from other pages for live sources (#68197)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
forms: do not consider fields from other pages for live sources (#68197)