entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

admin: allow numeric field inside block during test edit (#89734) #1422

Merged
vdeniaud merged 1 commits from wip/89734-testdef-vs-champ-de-type-nombre into main 2024-04-22 13:27:16 +02:00
Conversation 0 Commits 1 Files Changed 2 +31
vdeniaud commented 2024-04-18 18:27:34 +02:00
Owner
Copy Link

Joli bug, quand on édite les données d'un test, on crée un objet formdata, et pour chaque champ on appelle sa méthode from_json_value pour convertir les données stockées dans le test. Pour un bloc de champ, cette méthode appelle posted_json_data_to_formdata_data, qui a le mauvais goût de modifier le dictionnaire qui lui est passé. Et suite à ça nos données dans le test se retrouvent à contenir un objet Decimal, donc crash.

Joli bug, quand on édite les données d'un test, on crée un objet formdata, et pour chaque champ on appelle sa méthode from_json_value pour convertir les données stockées dans le test. Pour un bloc de champ, cette méthode appelle `posted_json_data_to_formdata_data`, qui a le mauvais goût de modifier le dictionnaire qui lui est passé. Et suite à ça nos données dans le test se retrouvent à contenir un objet Decimal, donc crash.
vdeniaud added 1 commit 2024-04-18 18:27:34 +02:00
gitea/wcs/pipeline/head There was a failure building this commit Details
72148e3189 admin: allow numeric field inside block during test edit (#89734)
vdeniaud force-pushed wip/89734-testdef-vs-champ-de-type-nombre from 72148e3189 to 8cdb333a94 2024-04-18 18:44:09 +02:00 Compare
vdeniaud force-pushed wip/89734-testdef-vs-champ-de-type-nombre from 8cdb333a94 to 01799c9124 2024-04-22 11:38:55 +02:00 Compare
vdeniaud force-pushed wip/89734-testdef-vs-champ-de-type-nombre from 01799c9124 to d6771b2869 2024-04-22 11:41:28 +02:00 Compare
vdeniaud changed title from WIP: admin: allow numeric field inside block during test edit (#89734) to admin: allow numeric field inside block during test edit (#89734) 2024-04-22 11:49:26 +02:00
fpeters approved these changes 2024-04-22 13:26:32 +02:00
vdeniaud merged commit d6771b2869 into main 2024-04-22 13:27:16 +02:00
vdeniaud deleted branch wip/89734-testdef-vs-champ-de-type-nombre 2024-04-22 13:27:16 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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/wcs#1422
No description provided.
Delete Branch "wip/89734-testdef-vs-champ-de-type-nombre"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?