Development #69264
manager: la recherche sur le type d'évènement dans le journal est ignorée si le queryset résultant est vide
0%
Description
Et donc ça retournera tous les évènements au lieu de n'en retourner aucun.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-manager-apply-event_type-filter-even-if-not-event-ty.patch 0001-manager-apply-event_type-filter-even-if-not-event-ty.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-manager-apply-event_type-filter-even-if-not-event-ty.patch 0001-manager-apply-event_type-filter-even-if-not-event-ty.patch ajouté
- Tracker changé de Bug à Development
Ok.
Mis à jour par Valentin Deniaud il y a plus d'un an
Possible d'utiliser
response.form['event_type'].select(text='User deletions') response = response.form.submit()
pour la lisibilité et la cohérence avec les autres tests ?
Bon j'arrête de temporiser, je ne comprends rien au patch, tu peux expliciter le bug et sa solution ?
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-manager-apply-event_type-filter-even-if-not-event-ty.patch 0001-manager-apply-event_type-filter-even-if-not-event-ty.patch ajouté
Voilà modification au test pour utiliser un peu plus l'API webtest.
Pour l'explication: si event_type = <Queryset []>
(le queryset vide) alors la condition est fausse et on ne filtre pas et donc on affiche tous le journal, alors qu'on devrait afficher un journal vide (c'est le bug relevé par Mike).
J'ai viré le code qui n'avait pas de rapport mais que je trouvais plus clair dans clean_event_type.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
PS: Le queryset d'EventType est vide s'il n'y pas eu encore de suppression par exemple, dans ce cas le type n'est pas créé (il est créé à la demande).
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Compris, top
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 0a334b5d2198bd459ff4f570d6bf2c0cda61fda9 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Mon Sep 19 14:58:37 2022 +0200 manager: apply event_type filter even if not event type was found (#69264) An empty queryset is not the same as the absence of a queryset.
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
manager: apply event_type filter even if not event type was found (#69264)
An empty queryset is not the same as the absence of a queryset.