Projet

Général

Profil

Development #53505

TypeError: 'NoneType' object does not support item assignment

Ajouté par Sentry Io il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
28 avril 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision e4cdc614 (diff)
Ajouté par Valentin Deniaud il y a presque 3 ans

agendas: do not set recurrence end date on event recurrences (#53505)

Révision 01c0cbb8 (diff)
Ajouté par Valentin Deniaud il y a presque 3 ans

agendas: do not set recurrence end date on event recurrences (#53505)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Projet changé de Suivi des traces à Chrono
#2

Mis à jour par Valentin Deniaud il y a presque 3 ans

  • Assigné à mis à Valentin Deniaud
#3

Mis à jour par Valentin Deniaud il y a presque 3 ans

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 :/

#4

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).

#5

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)
#6

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).

#7

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.

#8

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

Formats disponibles : Atom PDF