camillemonchicourt
commented
5 months ago J'ai une autre proposition plus radicale.
Supprimer NoDeleteMixin, comme discuté et validé en 2020 lors du premier groupement de commandes : https://github.com/GeotrekCE/Geotrek-admin/issues/2136
On a reporté cela à cause de questions à trancher et de complexité, même cette suppression logique des objets plutôt que physique est clairement à remettre en cause :
- Elle a été mise en place initialement dans l'idée de pouvoir revenir à des objets supprimés par erreur ou de pouvoir y revenir
- Mais au final je pense que personne ne connait vraiment ce fonctionnement ni ne l'utilise
- Les sauvegardes de BDD sont là pour remonter un objet perdu ou supprimé par erreur et on n'en a besoin jamais ou quasiment
- Si on voulait vraiment un mécanisme de "Corbeille" il faudrait aller au bout de la démarche et pouvoir accéder à cette corbeille depuis l'interface et restaurer des objets. Je ne pense pas que cela soit un réel besoin
- Cela complexifie les développements car il faut toujours penser à ne pas remonter ou prendre en considération les objets ayant le champs
deletedàtrue - Cela complexifie l'exploitation de la BDD car il faut toujours penser à exclure les objets supprimés (
where deleted=true) - Cela pollue fortement la BDD de tous les Geotrek sans que les utilisateurs ne le sachent avec tous leurs tests, leurs doublons qui restent stockés dans la BDD
- Cela n'est pas homogène car on n'a pas ce fonctionnement pour les sites et parcours Outdoor donc cela apporte de la confusion
Chatewgne
Bonjour,
nous avons récemment révisé le système de suppression en cascade (exemple de cascade : la suppression d'une pratique sportive qui entraînait la suppression de tous les itinéraires associés à cette pratique), ce afin d'éviter la perte involontaire de données, avoir un système plus logique et homogène, et également mieux tracer la raison des suppressions dans l'historique : https://github.com/GeotrekCE/Geotrek-admin/pull/3549. Désormais la plupart des liens entre les objets sont protégés, c'est à dire qu'on ne peut pas supprimer une pratique (catégorie) si des itinéraires (objets) y sont encore associés.
Mais suite à ce travail on peut désormais rencontrer un autre problème : ne pas réussir à supprimer une catégorie malgré la suppression préalable des objets qui lui sont associés. Cela est du au fait que les objets ne sont pas réellement supprimés dans la base, il sont conservés mais leur champ
deletedest passé àTrue(NoDeleteMixindans les modèles).Ce ticket vise à rassembler les réflexions autour des mécanismes de suppression pour arriver à quelque chose de plus satisfaisant.
Pistes :
NoDeleteMixinaux catégoriesNoDeleteMixinde sorte à déplacer les objets "supprimés" dans une autre table dédiée, au lieu de les garder dans la même table