Projet

Général

Profil

Development #24831

improper binding for lasso_profile_get_issuer

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Catégorie:
-
Version cible:
Début:
27 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

The FAQ entry from #4378 mentions this:

     msg = request.META['QUERY_STRING']
     provider_id = lasso.Profile.getIssuer(msg)

but the reality is a less Pythonic lasso.profileGetIssuer(msg).


Fichiers

Révisions associées

Révision eb6e0e23 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

python: add a classmethod for lasso.profileGetIssuer (#24831)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Tracker changé de Support à Development
#2

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

With current binding infrastructure it's not simple, methods are attached to class based on their first argument, such a class method would have to find the class based on the undetermined prefix qui correspond to the class, maybe helped by the docstring as for owner class of pointer (caller-owned/callee-owned).

#3

Mis à jour par Frédéric Péters il y a presque 6 ans

Indeed (that's why I set it as low priority) but we may just do it in pure Python, like in attached patch?

#4

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Solution proposée à Solution validée

Lack of imagination on my side.

#5

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit eb6e0e23b4f984d77c014d45a375b3474ced34e2
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Jun 27 19:29:40 2018 +0200

    python: add a classmethod for lasso.profileGetIssuer (#24831)
#6

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Assigné à mis à Frédéric Péters
#7

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Version cible mis à 2.6.1
#8

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF