Produits Entr'ouvert » Authentic 2

Je ne sais pas pourquoi il y a ici de la gestion du POST à la main au lieu d'avoir une FormView/DeleteView mais soit. Même remarque que l'autre ticket je pense que le .filter() est superflu par rapport à un .get().

En fait ce qu'on se retrouve à faire ici c'est un get_object() en moins bien et de la validation en moins bien, ça simplifierait pas mal d'hériter d'une vue d'édition classique je pense.

Serghei Mihai a écrit :

On peut donner le nom oauth-consents à la vue et aux classes CSS pour que ça soit plus explicite, car on risque d'oublier dans un mois la signification de juste consents.

Pour moi ça n'a aucun rapport avec oauth donc non, je ne sais pas d'où viennent ces oauth partout. Consent c'est très claire pour moi, c'est les consentements aux partages des données personelles ça s'appelle comme ça chez tous nos concurrents keycloak&co.

Pour moi ça n'a aucun rapport avec oauth donc non (...)

Et pour appuyer ça, #55645.

Frédéric Péters a écrit :

Et pour appuyer ça, #55645.

Pas de placement de produit.

Valentin Deniaud a écrit :

Bien tenté

T'es obligé de relire maintenant :)

(ceci n'est pas une relecture)

On pourrait alors prefixer la classe CSS au moins avec un a2- (comme les a2-block).

(ceci n'est pas une relecture)

J'ai testé et je ne vois pas de régression (par rapport à #45200).

Serghei Mihai a écrit :

(ceci n'est pas une relecture)

On pourrait alors prefixer la classe CSS au moins avec un a2- (comme les a2-block).

Ouais non j'arrête avec ça, personne ne le fait ailleurs.

Comme je suis pas bien à l'aise avec les FormView/DeleteView (c'est moi qui ne les ai pas introduites ici),
je préférerai que ce soit quelqu'un qui maîtrise qui valide : Valentin ?

Ça m'a l'air bon, je me demande juste à quoi sert

+    def get(self, request, *args, **kwargs):
+        return HttpResponseRedirect(self.get_success_url())

Valentin Deniaud a écrit :

Ça m'a l'air bon, je me demande juste à quoi sert
[...]
dans ConsentDelete ?

Normalement une DeleteView affiche un template de confirmation de la suppression sur un GET et ne supprime que sur un POST, j'ai voulu m'éviter une erreur sur le GET qui tenterait d'aller chercher un template qui n'existe pas, j'aurai pu retourner une 400 aussi, mais je me suis dit qu'une redirection ça cachait bien les choses aussi.