Bug #47746
Le filtre form_objects|filter_by:"plop"|filter_value:"bip"|exclude_self|count ne fonctionne pas dans les conditions de sortie de page
0%
Description
Ma condition de sortie de page :
form_objects|filter_by:"no_sequentiel_recherche"|filter_value:form_var_no_sequentiel_recherche|exclude_self|count == 0
est toujours fausse (https://formulaires-demarches-recette.cr-reunion.fr/backoffice/forms/97/fields/43/) alors que l'inspect me valide bien cette même condition :
https://formulaires-demarches-recette.cr-reunion.fr/backoffice/management/fonds-de-solidarite-volet-discotheques/3/inspect
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-misc-make-exclude_self-work-in-post-conditions-47746.patch 0001-misc-make-exclude_self-work-in-post-conditions-47746.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
1/ on pourrait penser que le |exclude_self n'est pas utile vu que la demande pas encore validée sera en brouillon et automatiquement exclue mais c'est utile de quand même pouvoir mettre le |exclude_self si on veut qu'une action d'édition ensuite fonctionne.
2/ le |exclude_self lors de l'action d'édition ne va pas fonctionner parce qu'à ce moment la demande en cours d'édition est une copie temporaire de celle en base, alors qu'on voudrait considérer l'id de la demande originale.
Dans le patch,
+ # keep track of original formdata id so it can be used by + # |exclude_self filter. + formdata._edited_id = self.edited_data.id
de manière plus aventureuse il aurait été possible de faire formdata.id = self.edited_data.id mais petite crainte que ça permette en cours d'édition à la demande originale d'être écrasée, avant même la validation des changements. (malgré un test ajouté pour vérifier que ça n'est pas le cas dans une situation basique).
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ea1657efd6595dc231407bfbd4990044b486f4d5
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Dec 1 08:38:47 2020 +0100
misc: make |exclude_self work in post conditions (#47746)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: make |exclude_self work in post conditions (#47746)