Bug #15422
indexation vs cellules qui exigent un user
0%
Description
if not self.is_relevant(context):
File "/usr/lib/python2.7/dist-packages/combo/apps/lingo/models.py", line 309, in is_relevant
if not (getattr(context['request'], 'user', None) and context['request'].user.is_authenticated()):
File "/usr/lib/python2.7/dist-packages/django/template/context.py", line 71, in __getitem__
raise KeyError(key)
KeyError: 'request'
Fichiers
Révisions associées
lingo: mark basket cells as depending on user (#15422)
Historique
Mis à jour par Frédéric Péters il y a environ 7 ans
- Fichier 0001-lingo-use-is_visible-to-mask-cells-to-anonymous-user.patch 0001-lingo-use-is_visible-to-mask-cells-to-anonymous-user.patch ajouté
Alors, peut-être que l'indexation devrait poser une vraie classe request bidon et que je me résoudrai à ça à un moment mais pour le moment je garde quand même comme cap de gérer les cellules qui ne doivent pas être visibles aux usagers anonymes via la méthode is_visible.
Ici, je répète encore beaucoup, je pense à un moment que ça devrait être simplifié pour juste avoir un attribut genre "visibility_requires_user = True" à poser sur la classe et géré par la classe de base, et plus besoin de définir is_visble partout.
[...]
En fait, je vais faire ça.
Mis à jour par Frédéric Péters il y a environ 7 ans
- Fichier 0001-general-add-flag-to-mark-a-cell-as-requiring-a-user-.patch 0001-general-add-flag-to-mark-a-cell-as-requiring-a-user-.patch ajouté
- Sujet changé de lingo vs indexation de cellules à indexation vs cellules qui exigent un user
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Voilà, ça supprime du code donc c'est nécessairement bon.
Mis à jour par Thomas Noël il y a environ 7 ans
Frédéric Péters a écrit :
Voilà, ça supprime du code donc c'est nécessairement bon.
Annule et remplace le précédent, isn't it ?
Mis à jour par Frédéric Péters il y a environ 7 ans
Oui, le précédent il s'est trouvé attaché par erreur.
Mis à jour par Thomas Noël il y a environ 7 ans
Pourquoi ne pas utiliser user_dependant ? (pour moi ça avait la même signification)
Mis à jour par Frédéric Péters il y a environ 7 ans
À l'origine, le "user_dependant", c'est pas rapport au rendu en mode "squelette", pour dire que le rendu doit absolument se faire en mode ajax (pour une cellule qui autrement ferait son rendu direct, genre la cellule "Lien vers le panier"). Même en réfléchissant je ne suis pas tout à fait sûr que ça corresponde pile (ni tout à fait sûr du contraire).
Mis à jour par Frédéric Péters il y a environ 7 ans
- Fichier 0001-misc-limit-indexing-to-anonymous-cells-15422.patch 0001-misc-limit-indexing-to-anonymous-cells-15422.patch ajouté
À réfléchir davantage dessus, il y a d'abord que la visibilité de la page n'était pas prise en compte, d'autre part qu'on peut zapper l'indexation pour les cellules dont le contenu se déclare dépendant de l'utilisateur connecé.
Cela étant, le code précédent factorisait quand même une situation un peu différente (pas de rendu du tout quand pas d'usager) et ça serait chouette de la voir à nouveau creusée plus tard.
Mis à jour par Frédéric Péters il y a environ 7 ans
- Fichier 0001-lingo-mark-basket-cells-as-depending-on-user-15422.patch 0001-lingo-mark-basket-cells-as-depending-on-user-15422.patch ajouté
Ça + marquer les cellules lingo avec user_dependant = True.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans
Ack, dit-on dans la salle.
Mis à jour par Frédéric Péters il y a environ 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 3e4eef551ad1f26edeb4d9fbd4bc80645beb765d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Mar 16 08:29:52 2017 +0100
lingo: mark basket cells as depending on user (#15422)
commit e0d5cc20da4d08ee530a1728b21862626ac2578c
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Mar 15 23:44:03 2017 +0100
misc: limit indexing to anonymous cells (#15422)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: limit indexing to anonymous cells (#15422)