Development #53505
TypeError: 'NoneType' object does not support item assignment
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/44715/
TypeError: 'NoneType' object does not support item assignment (8 additional frame(s) were not displayed) ... File "django/views/generic/edit.py", line 183, in post return self.form_valid(form) File "django/views/generic/edit.py", line 162, in form_valid self.object = form.save() File "chrono/manager/forms.py", line 224, in save self.instance.create_all_recurrences(excluded_datetimes) File "chrono/agendas/models.py", line 1471, in create_all_recurrences recurrences = self.get_recurrences(localtime(now()), make_aware(max_datetime), excluded_datetimes) File "chrono/agendas/models.py", line 1409, in get_recurrences self.recurrence_end_date, datetime.time(0, 0)
Fichiers
Révisions associées
agendas: do not set recurrence end date on event recurrences (#53505)
Historique
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Fichier 0001-agendas-do-not-set-recurrence-end-date-on-event-recu.patch 0001-agendas-do-not-set-recurrence-end-date-on-event-recu.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Oups, bug présent depuis le début mais invisible parce qu'on ne pouvait pas accéder à cette page avant #52112. Je pense que ça mérite un hotfix :/
Mis à jour par Thomas Noël il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Valentin Deniaud a écrit :
Oups, bug présent depuis le début mais invisible parce qu'on ne pouvait pas accéder à cette page avant #52112. Je pense que ça mérite un hotfix :/
Go, à pousser dès que c'est vert.
Le hotfix c'est parce qu'on peut se retrouver avec des données faussées ? Ou bien que la page provoque "juste" un crash ? Je peux m'en occuper en tout cas (mais pas illico, plutôt cette nuit, parce que migration, donc downtime).
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e4cdc61474775f0ca13f171cff8f31e0764273f6 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Apr 28 15:37:53 2021 +0200 agendas: do not set recurrence end date on event recurrences (#53505)
Mis à jour par Valentin Deniaud il y a presque 3 ans
Thomas Noël a écrit :
Le hotfix c'est parce qu'on peut se retrouver avec des données faussées ? Ou bien que la page provoque "juste" un crash ?
Juste un crash, mais comme la prochaine mise à jour est dans longtemps je doute qu'on coupe au hotfix (quelqu'un finira par tomber dessus en prod).
Mis à jour par Thomas Noël il y a presque 3 ans
Valentin Deniaud a écrit :
Thomas Noël a écrit :
Le hotfix c'est parce qu'on peut se retrouver avec des données faussées ? Ou bien que la page provoque "juste" un crash ?
Juste un crash, mais comme la prochaine mise à jour est dans longtemps je doute qu'on coupe au hotfix (quelqu'un finira par tomber dessus en prod).
Super donc ça pourra attendre cette nuit, merci ! Je m'occupe du hotfix.
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agendas: do not set recurrence end date on event recurrences (#53505)