Produits Entr'ouvert » Authentic 2

Est-ce que tu peux me décrire en une phrase le parcours stp ? Je vais essayer de reproduire le bogue.

/manage/users/add/?next_url=plop → /manage/users/1/add/

La vue c'est ça, il faut juste y ajouter la query string :

def user_add_default_ou(request):
    ou = get_default_ou()
    return redirect(request, 'a2-manager-user-add', kwargs={'ou_pk': ou.id})

J'écris des tests pour reproduire le bogue.
Dans l'intitulé du ticket, il est question de /manage/, et dans les remarques du ticket, c'est /api/.
Est-ce que c'est à la fois le manager et l'API DRF qui sont dysfonctionnels ?

Il y a des fonctions utilitaires pour construire des URLs dans authentic qu'il faut utiliser, ici ce serait:

from authentic2.utils import make_url
...
    return make_url('a2-manager-user-detail', kwargs={'pk': self.object.pk}, request=self.request, keep_params=True, include=[REDIRECT_FIELD_NAME])

Ça pourrait être utile d'en faire un raccourci vu que c'est une action qui revient assez souvent, genre:

Ça fait tout le job, par contre la variable next ne peut pas s'appeler next_url c'est next qu'il faut, c'est pareil partout (ou django.contrib.auth.REDIRECT_FIELD_NAME qui évite les typos), ne pas commencer par introduire une dissonance.

Une première version, qui ne conserve la querystring que lorsqu'aucun next_url n'est mentionné.

La modification doit uniquement porter sur la vue que je citais.

Ça fait tout le job, par contre la variable next ne peut pas s'appeler next_url c'est next qu'il faut, c'est pareil partout (ou django.contrib.auth.REDIRECT_FIELD_NAME qui évite les typos), ne pas commencer par introduire une dissonance.

Je parle ici de conserver la querystring, si dedans c'est pas ?next_url mais ?next, ça ne change rien pour moi.

Mais ticket à créer parce que #26652 a utilisé next_url.

Le include=['next'] s'est perdu.

Enfin include=[django.contrib.auth.REDIRECT_FIELD_NAME].

Benjamin Dauvergne a écrit :

Enfin include=[django.contrib.auth.REDIRECT_FIELD_NAME].

À voir avec Frédéric, mais je crois que l'objet du ticket est de conserver toute querystring lors de la redirection sur la page d'ajout d'usager dans l'OU par défaut.
Peut-être que je me suis trompé à la lecture du code et en testant rapidement l'affaire, mais il me semble qu'un paramètre include provoque la perte de tous les paramètres de querystring non explicitement mentionnés, non ?