Bug #69859
API fillslot avec exclude_user & user_external_id, les résa annulées sont prises en compte
0%
Description
le code:
if exclude_user and user_external_id:
if event.booking_set.filter(user_external_id=user_external_id).exists():
raise APIError(
N_('event %s is already booked by user'),
event.slug,
err_class='event is already booked by user',
on n'exclut pas les résa annulées
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a plus d'un an
- Fichier 0001-api-fix-fillslot-with-user-exclusion-cancelled-booki.patch 0001-api-fix-fillslot-with-user-exclusion-cancelled-booki.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus d'un an
Valentin, j'ai un doute, le fix n'est pas incohérent dans le cadre d'une utilisation events/fillslots ?
Je vois dans le serializer qu'on a bien exclude_user en param possible pour ce endpoint, mais dans publik famille par exemple on ne l'utilise pas.
A y réfléchir, j'ai plutôt l'impression que utiliser exclude_user pour events/fillslots serait incohérent. Peut-être qu'on devrait le supprimer du serializer pour ces endpoints ?
Mis à jour par Valentin Deniaud il y a plus d'un an
Lauréline Guérin a écrit :
Valentin, j'ai un doute, le fix n'est pas incohérent dans le cadre d'une utilisation events/fillslots ?
Je n'ai pas l'impression que le fix change grand chose à la situation actuelle, pour toi ça irait modifier/casser un truc à l'api events/fillslots ?
A y réfléchir, j'ai plutôt l'impression que utiliser exclude_user pour events/fillslots serait incohérent. Peut-être qu'on devrait le supprimer du serializer pour ces endpoints ?
Ouep, mais il n'y a pas vraiment de rapport avec le bug de ce ticket, si ? (je peux pas voir le contexte CD13 parce que je suis redirigé sur un truc microsoft, peut-être que je comprendrais mieux si je pouvais)
Mis à jour par Lauréline Guérin il y a plus d'un an
mais il n'y a pas vraiment de rapport avec le bug de ce ticket
non du tout, pas de rapport, mais un questionnement que je pose là pour éventuellement créer un ticket sur le sujet.
Je n'ai pas l'impression que le fix change grand chose à la situation actuelle, pour toi ça irait modifier/casser un truc à l'api events/fillslots ?
pour moi events/fillslots et exclude_user sont incompatibles, du fait que events/fillslots gère des booking uniques pour un user_external_id et un event donnés (on annule une résa existante si non postée, etc)
et à ma connaissance cette configuration n'est pas utilisée, en tout cas pas pour publik famille.
Mis à jour par Valentin Deniaud il y a plus d'un an
Lauréline Guérin a écrit :
pour moi events/fillslots et exclude_user sont incompatibles, du fait que events/fillslots gère des booking uniques pour un user_external_id et un event donnés (on annule une résa existante si non postée, etc)
et à ma connaissance cette configuration n'est pas utilisée, en tout cas pas pour publik famille.
Oui je suis d'accord, sûrement pas utilisé dans les tests non plus, je suis pour virer ça dans un autre ticket.
Mis à jour par Lauréline Guérin il y a plus d'un an
ok je merge en l'état et j'ouvre un autre ticket, merci
Mis à jour par Lauréline Guérin il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit af7ffbdae58fd7717383313cb4a07688d721a53d
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Oct 4 13:38:00 2022 +0200
api: fix fillslot with user exclusion & cancelled bookings (#69859)
Mis à jour par Lauréline Guérin il y a plus d'un an
- Lié à Development #69867: API events/fillslots, ne pas permettre l'utilisation du param exclude_user ajouté
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
api: fix fillslot with user exclusion & cancelled bookings (#69859)