entrouvert
/
chrono
15
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

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
Conversation 1 Commits 2 Files Changed 9
lguerin commented 2023-03-02 11:14:02 +01:00
Owner
There is no content yet.
lguerin added 2 commits 2023-03-02 11:14:02 +01:00
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?
lguerin commented 2023-03-02 11:46:52 +01:00
Poster
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.
Reviewers
No reviewers
aberriot
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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
There is no content yet.
Delete Branch "wip/74872-api-extra-data"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?