Booking: n'importe quoi dans extra_data (TypeError: unhashable type: 'list') (#74872) #43

Merged
lguerin merged 2 commits from wip/74872-api-extra-data into main 2023-03-02 14:36:32 +01:00
Owner
No description provided.
aberriot approved these changes 2023-03-02 11:24:19 +01:00
aberriot left a comment
Owner

Je crois que plutôt qu'un if isinstance(v, (list, dict)): un if not isinstance(v, (string, int, float, bool)): (ou approchant) aurait été plus explicite et sur, mais peut-être que on attend des types plus variés qui sont sommes toutes convertibles en string à la fin?

Je crois que plutôt qu'un `if isinstance(v, (list, dict)):` un `if not isinstance(v, (string, int, float, bool)):` (ou approchant) aurait été plus explicite et sur, mais peut-être que on attend des types plus variés qui sont sommes toutes convertibles en string à la fin?
Author
Owner

toutes convertibles en string à la fin

D'ailleurs je vais changer ça (le str(v)), je ne suis pas sûre de moi; donc c'est juste un blindage contre list et dict :)

> toutes convertibles en string à la fin D'ailleurs je vais changer ça (le str(v)), je ne suis pas sûre de moi; donc c'est juste un blindage contre list et dict :)
lguerin force-pushed wip/74872-api-extra-data from eb64c900c2 to c35dffb1bb 2023-03-02 11:48:21 +01:00 Compare
lguerin requested review from aberriot 2023-03-02 11:48:36 +01:00
aberriot approved these changes 2023-03-02 11:50:54 +01:00
aberriot left a comment
Owner

Va pour le blindage ;)

Va pour le blindage ;)
lguerin merged commit c35dffb1bb into main 2023-03-02 14:36:32 +01:00
lguerin deleted branch wip/74872-api-extra-data 2023-03-02 14:36:32 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 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/chrono#43
No description provided.