entrouvert
/
gadjo
15
0
Fork 0
Code Issues Pull Requests 2 Releases Activity

widget: allow setting empty value in MultiSelectWidget (#77469) #21

Merged
vdeniaud merged 1 commits from wip/77469-multiselectwidget-permettre-de-v into main 2023-05-17 09:37:26 +02:00
Conversation 2 Commits 1 Files Changed 2 +26
vdeniaud commented 2023-05-11 11:50:46 +02:00
Owner
Copy Link

Il faut zapper la surcharge de MultiWidget à value_omitted_from_data, sans quoi Django pense que la valeur du champ est absente de données soumises.

Il faut zapper la surcharge de MultiWidget à value_omitted_from_data, sans quoi Django pense que la valeur du champ est absente de données soumises.
vdeniaud added 1 commit 2023-05-11 11:50:47 +02:00
fpeters commented 2023-05-11 12:02:19 +02:00
Owner
Copy Link

Dans #19 j'ajoute des tests, je le passerai vendredi dans le dépôt, ensuite ça sera bien de reprendre ici pour ajouter un test (si pas trop galère).

Dans https://git.entrouvert.org/entrouvert/gadjo/pulls/19 j'ajoute des tests, je le passerai vendredi dans le dépôt, ensuite ça sera bien de reprendre ici pour ajouter un test (si pas trop galère).
vdeniaud force-pushed wip/77469-multiselectwidget-permettre-de-v from 5d44a6c1fb to e3ad91a61c 2023-05-16 16:01:50 +02:00 Compare
vdeniaud commented 2023-05-16 16:04:33 +02:00
Author
Owner
Copy Link

Ajouté un test, assez bidon car la mécanique qu'on cherche à tester est spécifique ModelForm et ça me paraît très galère de mocker ce qu'il faudrait (la méthode value_omitted_from_data des widgets de Django n'est appelée qu'à un endroit, dans la fonction construct_instance() de django.forms.models).

Ajouté un test, assez bidon car la mécanique qu'on cherche à tester est spécifique ModelForm et ça me paraît très galère de mocker ce qu'il faudrait (la méthode value_omitted_from_data des widgets de Django n'est appelée qu'à un endroit, dans la fonction construct_instance() de django.forms.models).
vdeniaud requested review from fpeters 2023-05-16 18:08:01 +02:00
fpeters approved these changes 2023-05-16 22:37:01 +02:00
vdeniaud merged commit e3ad91a61c into main 2023-05-17 09:37:26 +02:00
vdeniaud deleted branch wip/77469-multiselectwidget-permettre-de-v 2023-05-17 09:37:26 +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/gadjo#21
No description provided.
Delete Branch "wip/77469-multiselectwidget-permettre-de-v"

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?