On ne gère pas les demandes de mise en avant associées à des contenus supprimés

[Situphen]

Les demandes de mises en avant sont représentées par l'objet FeaturedRequested qui peut faire référence à différents objets (contenu, sujet du forum...) grâce à la table l'infrastructure des types de contenus. Au lieu d'utiliser seulement un champ models.ForeignKey classique, on utilise un champ virtuel GenericForeignKey qui combine un champ models.ForeignKey pour stocker le type de contenu et un champ models.PositiveIntegerField pour stocker l'objet :

zds-site/zds/featured/models.py

Lines 59 to 61 in 3505295

	content_type = models.ForeignKey(ContentType, verbose_name=_("Type de l'objet"), on_delete=models.CASCADE)
	object_id = models.PositiveIntegerField(db_index=True, verbose_name=_("Id de l'objet"))
	content_object = GenericForeignKey("content_type", "object_id")

Lorsqu'on supprime un objet en base de données, les conséquences sur les autres objets sont généralement gérés par Django (via l'attribut on_delete sur les champs). Ce n'est pas le cas avec l'infrastructure des types de contenus. Dans notre cas, si un des objets (contenu, sujet du forum...) est supprimé, l'objet FeaturedRequested n'est pas supprimé en base de données et son champ content_object renvoie simplement None. Il faut donc prendre en compte que ce champ peut être nul dans notre code.

Ce n'est pas le cas dans le code de la vue FeaturedRequestedList qui correspond à l'URL /mise-en-avant/unes/requetes/ car cette ligne renvoie une AttributeError lorsque content_object est nul pour l'un des objets :

zds-site/zds/featured/views.py

Line 257 in 3505295

return [q for q in queryset.all() if isinstance(q.content_object, Topic) or not q.content_object.is_obsolete]

À noter qu'il est possible (probable ?) que ce soucis soient présent ailleurs dans le code aussi !

---

Pour reproduire :

1. (n'importe quel utilisateur) aller sur un contenu publié et cliquer sur "proposer la mise en une" dans la barre latérale
2. (un staff) dépublier le contenu
3. (l'auteur) enlever les éventuels autres auteurs et supprimer le contenu

[philippemilink]

Corrigé avec #6626