Produits Entr'ouvert » Authentic 2

I would like the local logout to happen before the last redirection, so this patch is not acceptable currently.

C'est là qu'un test autour du logout aurait-été bien... Est-ce que tu ne pourrais pas modifier un peu moins la structure de contrôle ? La fonction est déjà complexe, et le changement ne la simplifiant pas j'aurai préféré que le changement soit moins intrusif.

La structure de contrôle sort de la fonction si l'utilisateur n'est pas loggué localement. Or la vue gère désormais les logouts par redirections après le logout local, ce qui m'a conduit à revoir la structure de contrôle. Si tu vois quelque chose de plus léger, dis moi.

Avec le code actuel le next_url d'origine est perdu quand targets n'est pas vide, j'ajouterai le patch suivant;

diff --git a/src/authentic2/views.py b/src/authentic2/views.py
index ed348a2..13699bc 100644
--- a/src/authentic2/views.py
+++ b/src/authentic2/views.py
@@ -469,7 +469,7 @@ def logout(request, next_url=None, default_next_url='auth_homepage',
         local_logout_done = True
         # Put redirection targets in session (after logout)
         if targets:
-            request.session['logout_redirections'] = targets
+            request.session['logout_redirections'] = [next_url] + targets
     # Full logout by redirections if any
     targets = request.session.pop('logout_redirections', None)
     if targets:

Sinon on peut pousser je pense.

Ack.