EsupPortail / Esup-Pod

plateforme de gestion de fichier vidéo
https://pod.esup-portail.org/
GNU Lesser General Public License v3.0
35 stars 75 forks source link

[DONE] fix live_event save for many_to_many fields #1080

Closed mattbild closed 6 months ago

mattbild commented 6 months ago

When not admin saving live_event discard many_to_many fields. This is because form.save(commit=False) is used. Fields have to be manually set after Event is saved or methode save_m2m() has to be called. see django doc here

Pour tester :

Avant l'évènement était bien enregistré mais sans le(s) groupe(s), ni propriétaire(s). La PR doit résoudre ce cas

mattbild commented 6 months ago

je ne connaissais pas la méthode save_m2m() ~~Elle doit être appelée sur le form pas sur l'entité à sauvegardée. Là, comme le champ 'end_date' n'existe pas dans le formulaire, la valeur est directement set sur l'entité (à partir de deux champs du formulaire "start_date" + "end_time"). Pas sûr que ça fonctionne~~ Effectivement ça fonctionne et ça permet d'éviter de faire un set sur chaque many_to_many Merci