Produits Entr'ouvert » Passerelle

Il faudrait être clair sur l'histoire des dates. Clairement, dans Publik, on ne peut pas en envoyer. Donc il faut être sur que ça fonctionne sans.

Pour la partie qui sera l'affichage des résa en cours d'un enfant (get_periscol_child_enrollment_planning, utilisé via combo), il faut un webservice qui prennent le numéro de l'enfant, mais qui puisse aussi prendre un name_id en paramètre optionnel dans la query_string. Et si le name_id est présent, alors il faut vérifier que le child_id appartient à la famille liée.

Thomas Noël a écrit :

Il faudrait être clair sur l'histoire des dates. Clairement, dans Publik, on ne peut pas en envoyer. Donc il faut être sur que ça fonctionne sans.

Les dates sont optionnels dans tous les appels, sauf celui de création d'une reservation. Et même dans ce cas la date vient d'une liste proposée par Agora.

Pour la partie qui sera l'affichage des résa en cours d'un enfant (get_periscol_child_enrollment_planning, utilisé via combo), il faut un webservice qui prennent le numéro de l'enfant, mais qui puisse aussi prendre un name_id en paramètre optionnel dans la query_string. Et si le name_id est présent, alors il faut vérifier que le child_id appartient à la famille liée.

Ok, j'avais pas vu ce cas. Je refais le patch.

Et le check de l'identifiant de l'enfant si le name_id est précisé.

(manque le patch je crois)

"which" je l'appelerais "reserved_day"

Et renommer « child_in_name_id_family » en « is_child_in_name_id_family »

Mais surtout, dans cette fonction : if child_id in child['id']: ça doit plutôt être == je pense.

Et peut-être pour se donner bonne conscience, utiliser des urlencode (genre urlencode(('start_date', start_date),('end_date', end_date)) c'est quand même plus rassurant)

Astuce découverte grâce à Agora : quand tu envoies une liste de tuple (au lieu d'un dico), urlencode respecte l'ordre.

Il manque mon petit s/which/reserved_day/ ? A ce niveau, avoir -1/0/1 dans l'URL ça me semble pas une bonne idée, je serais pour le recevoir en query_string, avec -1 par défaut comme tu as prévu (ce qui veut dire que c'est facultatif ⇒ query_string).

Idem pour get_periscol_enrollments(self, child, service, date=None): service est clairement facultatif, il faut le passer en query_string. Et en profiter pour l'appeler d'un bout à l'autre "service_id", ça rend sinon la relecture assez relou.

Lors du normalize_periscol_enrollment, renvoyer l'URL /periscol/enrollment/(?P<enrollment_id>[\w-]+)/planning/, on aura ainsi pas besoin de la construire dans le formulaire (sur le principe de ce qui est fait dans chrono qui renvoie les URL des événements)

Et le enrollment_id qui revient à la fin reservation_id, aussi (renommer ce dernier "enrollment_id" comme partout ailleurs, là encore ça aide à comprendre)

(et en dehors de ça, j'ai encore du mal à voir comment on va utiliser tout ça dans wcs et combo)

Ack