toulouse-maelis: check person_id on subscription endpoints (#76218) #176

Merged
nroche merged 1 commits from wip/76218-parsifal-check-person_id-on-add-basket into main 2023-04-04 16:20:23 +02:00
Owner

Parce que Maélis ne détecte pas,
vérifier que la personne appartient bien à la famille.
Fait pour l'inscription via le panier et l'inscription directe.

Parce que Maélis ne détecte pas, vérifier que la personne appartient bien à la famille. Fait pour l'inscription via le panier et l'inscription directe.
lguerin reviewed 2023-04-04 15:13:33 +02:00
@ -3209,6 +3209,7 @@ class ToulouseMaelis(BaseResource, HTTPResource):
family_id=None,
):
family_id = family_id or self.get_link(NameID).family_id
assert self.get_rl_or_child_raw(family_id, post_data['person_id'])
Owner

un peu moche le assert dans le code, on peut s'en passer, non ?

un peu moche le assert dans le code, on peut s'en passer, non ?
Author
Owner

Si, merci.
Je pensais que c'était requis par Pylint, j'ai confondu avec :
assert family_id or self.get_link(NameID)

Si, merci. Je pensais que c'était requis par Pylint, j'ai confondu avec : `assert family_id or self.get_link(NameID)`
nroche force-pushed wip/76218-parsifal-check-person_id-on-add-basket from 73303c7cd5 to 4b6ceac39e 2023-04-04 15:33:44 +02:00 Compare
lguerin approved these changes 2023-04-04 15:56:31 +02:00
nroche merged commit 4b6ceac39e into main 2023-04-04 16:20:23 +02:00
nroche deleted branch wip/76218-parsifal-check-person_id-on-add-basket 2023-04-04 16:20:23 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#176
No description provided.