Question - Delete cascade

[gildeluermoz]

Je me pose une question sur les permissions et les conséquences du delete cascade.

Dans la définition de la table gn_monitoring.t_base_visits, on a un delete cascade
CONSTRAINT fk_t_base_visits_id_base_site FOREIGN KEY (id_base_site) REFERENCES gn_monitoring.t_base_sites(id_base_site) ON DELETE CASCADE

Dans une instance, un site peut être utilisé dans plusieurs sous modules. C'est plutôt courant.
En interface, si je créé une visite sur un site, je ne peux plus supprimer ce site (voir image ci-dessous). Très bien.
Mais dans ce même module, tous les autres sites où je n'ai aucune visite peuvent être supprimés.

Donc si je supprime un site depuis ce module mais que ce site a des visites issues d'autres modules, je supprime toutes ses visites ?
Un user qui n'a des permissions que sur un module peut très bien penser qu'il peut supprimer un site car il ne sait pas qu'il est utilisé dans un autre module.

Je n'ai pas testé car je n'ai pas de base pour tester ça mais si oui, c'est une grosse faille.

[IdrissaD]

Je viens de tester, et même ayant les droits sur tous les modules je peux supprimer un site depuis un module si aucune visite n'y est associée dans ce contexte. Toutes les visites associées via un autre module sont supprimées.

La propriété nb_visits ne prend en compte que le contexte du sous-module, et semble être le seul paramètre pris en compte en front pour activer ou désactiver la suppression d'un site.

A minima il faudrait supprimer ce DELETE CASCADE, le front donnera toujours l'impression de pouvoir supprimer, mais la BDD bloquera, et au mieux avoir une autre propriété nb_visits_total qui ne soit pas affichée mais qui soit celle prise en compte pour la désactivation du bouton de suppression.

[gildeluermoz]

Ok, top ! Merci pour les tests.
Effectivement, il faudrait supprimer le DELETE CASCADE sur la base et récupérer le message postgres en cas d'erreur pour que l'utilisateur comprenne. Idéalement il faudrait réécrire le message postgres pour qu'il soit compréhensible et que l'utilisateur comprenne que ce site est utilisé pour des visites dans un autre module.
Je supprime le DELETE CASCADE sur mes instances en prod.

[gildeluermoz]

ALTER TABLE gn_monitoring.t_base_visits DROP CONSTRAINT fk_t_base_visits_id_base_site;
ALTER TABLE gn_monitoring.t_base_visits ADD CONSTRAINT fk_t_base_visits_id_base_site FOREIGN KEY (id_base_site) REFERENCES gn_monitoring.t_base_sites(id_base_site) ON DELETE NO ACTION;

[IdrissaD]

En fait problème déjà repéré en #417 et PR #429 associée

[gildeluermoz]

Merci pour le lien et la PR.
Je pense qu'au delà des améliorations du front, il faut aussi garder l'idée de supprimer le DELETE CASCADE.
Quoi qu'il en soit, ce CASCADE n'est pas souhaitable. Supprimer toutes les visites d'un site sans prévenir, c'est un peu radical et pas forcement compris par celui qui fait cette suppression.