Projet

Général

Profil

Development #8354

avoir un get_name_id

Ajouté par Thomas Noël il y a plus de 8 ans. Mis à jour il y a environ 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
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

#1

Mis à jour par Thomas Noël il y a plus de 8 ans

#2

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.

#3

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 ;)

#4

Mis à jour par Thomas Noël il y a environ 8 ans

  • Statut changé de En cours à Fermé

fait autrement, notamment dans #10530

Formats disponibles : Atom PDF