Projet

Général

Profil

Bug #67214

TypeError: 'NoneType' object is not subscriptable

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
11 juillet 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/61926/

TypeError: 'NoneType' object is not subscriptable
(4 additional frame(s) were not displayed)
...
  File "combo/public/views.py", line 583, in publish_page
    cell.modify_global_context(ctx, request)
  File "combo/apps/wcs/models.py", line 1020, in modify_global_context
    card_ids = self.get_card_ids(context, request)
  File "combo/apps/wcs/models.py", line 1270, in get_card_ids
    return self.get_card_ids_from_related(original_context, request)
  File "combo/apps/wcs/models.py", line 1238, in get_card_ids_from_related
    card_id = first_cell.get_card_id(context)
  File "combo/apps/wcs/models.py", line 1288, in get_card_id
    return context.get(self.global_context_key)[repeat_index]

Fichiers

Révisions associées

Révision 907a3d74 (diff)
Ajouté par Lauréline Guérin il y a presque 2 ans

wcs: fix card cell global context with misordered cells (#67214)

Historique

#1

Mis à jour par Lauréline Guérin il y a presque 2 ans

  • Projet changé de Suivi des traces à Combo
  • Assigné à mis à Lauréline Guérin
#4

Mis à jour par Lauréline Guérin il y a presque 2 ans

Pour reproduire: avoir 2 cellules "contenu d'une fiche", l'une avec un slug, l'autre avec en identifiant une fiche liée à la première.
Changer l'ordre des cellules pour positionner la cellule avec un slug (la source) après la cellule avec un related.
500.

#5

Mis à jour par Lauréline Guérin il y a presque 2 ans

Patch qui corrige un peu le problème.

2 cas:
- seule la cellule "source" a un slug; si on appelle en premier modify_global_context pour les cellules avec un slug, ça marche
- la cellule "source" et la cellule "related" ont toutes les deux un slug: pas géré, c'est pas si simple. (mais il n'y aura plus de 500)

(pour que ça marche à coup sûr: positionner les cellules "source" avant les cellules "related")

#6

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

  • Statut changé de Solution proposée à Solution validée
#7

Mis à jour par Lauréline Guérin il y a presque 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 907a3d7408db8ed57f0e7341fbfe369c286c6aea
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Jul 11 12:15:16 2022 +0200

    wcs: fix card cell global context with misordered cells (#67214)
#8

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF