#57 La suppression d'un compte ayant des contenus associés plante

Open
opened 2 months ago by Darks · 11 comments
Darks commented 2 months ago

Étant donné que ça casse des relations dans la BDD, on ne peut pas supprimer un compte qui a des contenus associés.

Il faut créer deux options :

  • tout supprimer
  • passer les contenus en Guest
Étant donné que ça casse des relations dans la BDD, on ne peut pas supprimer un compte qui a des contenus associés. Il faut créer deux options : - tout supprimer - passer les contenus en `Guest`
Darks added the
bug
label 2 months ago
Filoji commented 2 months ago

Je propose en Guest pour plusieurs raisons :

  • Les programmes proposés peuvent être intéressant, même si le compte devient cancer.
  • Si on choisit de créer des lien vers l’un des sujet de l’utilisateur depuis un autre topic, on tombera sur un lien mort (mais il aurait pu nous aider), ce qui peut devenir désagréable.

Ainsi, si on le veut, on peut très bien personnaliser la page user de celui-ci en affichant : "Est mort au combat dans la lutte anti-modération." ?

Je propose en Guest pour plusieurs raisons : * Les programmes proposés peuvent être intéressant, même si le compte devient cancer. * Si on choisit de créer des lien vers l'un des sujet de l'utilisateur depuis un autre topic, on tombera sur un lien mort (mais il aurait pu nous aider), ce qui peut devenir désagréable. Ainsi, si on le veut, on peut très bien personnaliser la page user de celui-ci en affichant : `"Est mort au combat dans la lutte anti-modération."` ?
Lephenixnoir commented 2 months ago
Owner

Je pense que le RGPD nous oblige à donner l’option de tout supprimer. Je propose de mettre l’option sur la page de suppression de compte, en précisant bien que conserver les contenus ne conservera aucune information personnelle et en laissant sur Guest par défaut.

Je pense que le RGPD nous oblige à donner l'option de tout supprimer. Je propose de mettre l'option sur la page de suppression de compte, en précisant bien que conserver les contenus ne conservera aucune information personnelle et en laissant sur Guest par défaut.
Lephenixnoir commented 2 months ago
Owner

Conséquence immédiate : un programme peut être posté par un invité.

Conséquence immédiate : un programme *peut* être posté par un invité.
Filoji commented 2 months ago

Non, un programme peut avoir comme auteur un invité, mais ne peut pas être posté par un invité.

Non, un programme peut avoir comme auteur un invité, mais ne peut pas être posté par un invité.
Lephenixnoir commented 2 months ago
Owner

Ma remarque n'était pas vraiment sur cette distinction (qui est correcte) mais sur les modification proposées du modèle hier dans #20 : on ne peut pas modifier la contrainte de clé étrangère sur author_id pour interdire les invités parce que ça ne permettrait pas d’implémenter cette fonctionnalité.

Ma remarque n'était pas vraiment sur cette distinction (qui est correcte) mais sur les modification proposées du modèle hier dans #20 : on ne peut pas modifier la contrainte de clé étrangère sur `author_id` pour interdire les invités parce que ça ne permettrait pas d'implémenter cette fonctionnalité.
Filoji commented 2 months ago

Et bien, c’est simple, on bloque l’accès au formulaire de post de programme aux Guests ?

Et bien, c'est simple, on bloque l'accès au formulaire de post de programme aux Guests ?
Lephenixnoir commented 2 months ago
Owner

À postériori du moins. On peut vouloir restreindre l’upload de programmes aux seuls membres.

Par contre en effet ça veut dire qu’ne faut pas surcharger la relation d’auteur dans Program.

Voilà c’est ça. Mais du coup il faut faire gaffe dans toutes les vues et opérations de manipulation de programmes qu’on peut être en train de jouer avec des invités.

> À postériori du moins. On peut vouloir restreindre l'upload de programmes aux seuls membres. > > Par contre en effet ça veut dire qu'ne faut pas surcharger la relation d'auteur dans Program. Voilà c'est ça. Mais du coup il faut faire gaffe dans toutes les vues et opérations de manipulation de programmes qu'on peut être en train de jouer avec des invités.
Filoji commented 2 months ago

Bah pourquoi on bloque le fait qu’un invité puisse créer un programme ? On dit que tous les Users peuvent créer un Programme, et ensuite dans la route, on vérifie que la personne actuelle est un membre. On évite les conflits et les administrateurs pourront toujours modifier les anciens programmes ?

Ou autre solution, on crée un compte bidon ‘décharge’ où on y met tout les topic et programmes d’une personne, et pour ses commentaires, on bascule en Guest ?

Bah pourquoi on bloque le fait qu'un invité puisse créer un programme ? On dit que tous les Users peuvent créer un Programme, et ensuite dans la route, on vérifie que la personne actuelle est un membre. On évite les conflits et les administrateurs pourront toujours modifier les anciens programmes ? Ou autre solution, on crée un compte bidon 'décharge' où on y met tout les topic et programmes d'une personne, et pour ses commentaires, on bascule en Guest ?
Darks commented 2 months ago
Owner

J’ai rien compris, mais en gros :

  • on reste sur l’héritage de Post par Program, sans surcharger la propriété author
  • on restreint l’accès à la publication de programmes par les membres
  • si un compte est supprimé avec l’option de garder les contenus, on créé un Guest du même pseudo, et on lui attribue les contenus
J'ai rien compris, mais en gros : - on reste sur l'héritage de Post par Program, sans surcharger la propriété `author` - on restreint l'accès à la publication de programmes par les membres - si un compte est supprimé avec l'option de garder les contenus, on créé un `Guest` du même pseudo, et on lui attribue les contenus
Lephenixnoir commented 2 months ago
Owner

C’est ça. Pour le point 2, on passe par @login_required ou un truc du genre. Il faudra aussi valider lors de la soumission du formulaire parce qu’on aura besoin d’upcaster vers Member.

C'est ça. Pour le point 2, on passe par `@login_required` ou un truc du genre. Il faudra aussi valider lors de la soumission du formulaire parce qu'on aura besoin d'upcaster vers `Member`.
Darks commented 2 months ago
Owner

Le @login_required s’applique lors du GET et lors du POST. Donc pas besoin de checker à la main une fois de plus 😉

Le `@login_required` s'applique lors du `GET` et lors du `POST`. Donc pas besoin de checker à la main une fois de plus :wink:
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.