Bug #4385
category.get_substitution_variables n'est pas json-able
0%
Description
Dans Category::get_substitution_variables on a un
'category': self,
ce qui rend le résultat incompatible avec un json.dumps() :
Exemple d'erreur obtenue :
TypeError: <Category 'Liaison avec mon dossier famille' id:1> is not JSON serializable
Selon moi, le reste des variables devrait suffire :
def get_substitution_variables(self):
d = {
'category': self,
'category_name': self.name,
'category_description': self.description,
'category_id': self.url_name,
}
return d
Je propose donc de retirer la variable "category". Sauf si elle est déjà utilisée qlq part...?
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 10 ans
Mis à jour par Thomas Noël il y a environ 2 ans
Je vote pour.
Je propose d'ajouter aussi « 'category': self » dans la liste des objets envoyés, afin de pouvoir avoir une syntaxe simple [if-any category]
Mais je ne sais pas si ça a ensuite été utilisé; en janvier 2012, ça aurait pu servir à quel projet ?
Mis à jour par Thomas Noël il y a environ 10 ans
#4386 permet, si on utilise toujours "minimal=True" pour le json, de s'en sortir autrement, et tranquillement.
Mis à jour par Thomas Noël il y a environ 10 ans
Après discussion avec Victor et des grep sur la production au-quotidien.com, [category] ne semble avoir jamais été utilisé. Je propose de supprimer la variable.
Patch attaché.
(note : la variable "category" n'est pas référencée dans la doc de l'api)
Mis à jour par Thomas Noël il y a environ 10 ans
- Statut changé de Nouveau à Résolu (à déployer)
- Assigné à mis à Thomas Noël
Et donc c'est poussé.
Mis à jour par Thomas Noël il y a environ 10 ans
- Statut changé de Résolu (à déployer) à Solution déployée
remove "category" variable (#4385)
[category] is a Python object, it's not useful and it can't be
serialized (json). We remove it.
Note: this variable does not appear to have been used.