Development #65849
TypeError: '<=' not supported between instances of 'list' and 'NoneType'
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/61354/
TypeError: '<=' not supported between instances of 'list' and 'NoneType'
(11 additional frame(s) were not displayed)
...
File "chrono/agendas/models.py", line 624, in get_effective_time_periods_virtual
for weektime_interval in IntervalSet.simple(*time_period_interval) - closed_hours_by_days:
File "chrono/interval.py", line 196, in __sub__
return self.__class__.from_ordered(gen())
File "chrono/interval.py", line 71, in from_ordered
return cls(iterable, already_sorted=True)
File "chrono/interval.py", line 53, in __init__
for begin, end in iterable:
File "chrono/interval.py", line 175, in gen
if c1[1] <= c2[0]:
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Dupliqué par Bug #64971: '<=' not supported between instances of 'list' and 'NoneType' ajouté
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-wip.patch 0001-wip.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le bug se produit quand un agenda virtuel a des périodes d'exclusion et qu'un des agendas réels a une période qui spécifie des numéros de semaine.
À ce moment là on compare des WeekTime de la forme (jour, numéro de semaine, heure), pour la période d'exclusion ça donne (jour, None, heure) et pour la période d'ouverture (jour, une liste, heure) et crash.
Comme la période d'exclusion s'applique à toutes les semaines sans distinction, il suffit de ne pas comparer les numéros de semaine.
Patch pour info, il reste au moins à étoffer le test et peut-être à trouver une façon plus claire de gérer les choses (on est pas vraiment aidés par la méthode sub de IntervalSet).
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-api-ignore-weekday_indexes-when-comparing-WeekTime-o.patch 0001-api-ignore-weekday_indexes-when-comparing-WeekTime-o.patch ajouté
La nuit porte conseil, patch plus simple.
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a4daeeeb672315fc1d951ec665bde9c7b8a694a0
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jun 1 17:34:02 2022 +0200
api: ignore weekday_indexes when comparing WeekTime objects (#65849)Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a environ un an
- Lié à Development #76571: Revoir la logique dans get_effective_timeperiods avec l'introduction de weekday_indexes ajouté
api: ignore weekday_indexes when comparing WeekTime objects (#65849)