Produits Entr'ouvert » Mandaye

Peut-être simplifier le if request.GET.get('next', None): en if request.GET.get('next'): qui revient au même.
Pareil pour

        url = result.get('url', '/')
    if request.GET.get('next'):
        url = request.GET['next']

url = request.GET.get('next') or request.get('url', '/')

Sinon sur le fond c'est bon pour moi.

De mon point de vue de vagabond qui passe ici par hasard :

• J'ai toujours du mal avec les trucs qui s'appellent next dans une partie du code, et next_url dans une autre. Choisir un nom, s'y tenir, dans le code comme dans la query-string.
• next_url = request.GET.get('next_url') tout simplement, comme le propose Paul
• dans le template, {% if next_url %}?next_url={{ next_url|iriencode }}{% endif %} (je pense le iriencode obligatoire ?)
• sur la partie post_login_do je m'interroge sur l'utilisation systématique du next_url indiqué : on va envoyer la personne vers ce next_url même si le login a foiré ? ça ne me semble pas forcément une bonne idée.

Pourquoi result.get('result') dans le bloc juste avant alors que result['result'] ici ?

Paul Marillonnet a écrit :

Pourquoi result.get('result') dans le bloc juste avant alors que result['result'] ici ?

Question d'historique. ça remonte de l'époque le retour phantomjs n'avait pas forcement une key result. Avec les évolutions elle en renvoit toujours depuis.

Ack, pour la prochaine mise à jour.