Development #8354
avoir un get_name_id
Début:
24 septembre 2015
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
dans combo on a déjà deux fois des mochetés du genre
if context.get('request') and hasattr(context['request'], 'session') and \ context['request'].session.get('mellon_session'): mellon = context.get('request').session['mellon_session'] query = '?NameID=' + mellon['name_id_content']
ça serait plus joli d'avoir un combo.utils.get_name_id qui sache répondre un name_id ou None, en sachant aller le chercher au fin fond de request
.
Fichiers
Historique
Mis à jour par Thomas Noël il y a plus de 8 ans
- Fichier 0001-utils-add-a-get_name_id-request-function-8354.patch 0001-utils-add-a-get_name_id-request-function-8354.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Je verrais plutôt ça dans mellon.utils, non ? Quitte à avoir du:
try: from mellon.utils import get_nameid except ImportError: def get_nameid(request): return None
Dans combo.utils, tant qu'à cacher la tambouille interne de django-mellon autant la cacher vraiment.
Mis à jour par Thomas Noël il y a plus de 8 ans
Benjamin Dauvergne a écrit :
Je verrais plutôt ça dans mellon.utils, non ?
Si ;)
Mis à jour par Thomas Noël il y a environ 8 ans
- Statut changé de En cours à Fermé
fait autrement, notamment dans #10530