Development #33559
solis rsa: liste de links
0%
Description
Vu ce format de _links dans une réponse :
{u'_links': {u'corrOrient': {u'href': u'http://solis.infodb.fr/solisapi-rsa/referentiels/trans/intervenantSocial/488/'}, u'refOrient': {u'href': u'http://solis.infodb.fr/solisapi-rsa/referentiels/trans/intervenantSocial/488/'}, u'refOrientStructAcc': [{u'href': u'http://solis.infodb.fr/solisapi-rsa/referentiels/trans/services/75/'}, {u'href': u'http://solis.infodb.fr/solisapi-rsa/referentiels/trans/services/75/'}]},
Le code actuel gère bien corrOrient et refOrient mais pas refOrientStructAcc qui est une liste de links.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 5 ans
- Fichier 0001-solis-handle-list-of-links-33559.patch 0001-solis-handle-list-of-links-33559.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 5 ans
Ça me semble ok mais le rsa_get_link_content
qui retournait quelque chose, il se trouve désormais remplir un objet, il faudrait modifier la docstring, voire peut-être même aussi renommer la méthode, genre "rsa_fill_with_link_content" (content/info/details/whatever) ?
Mis à jour par Thomas Noël il y a presque 5 ans
- Fichier 0001-solis-handle-list-of-links-33559.patch 0001-solis-handle-list-of-links-33559.patch ajouté
Bonne idée, j'ai renommé tel que proposé, « rsa_fill_with_link_content ».
Mis à jour par Frédéric Péters il y a presque 5 ans
Il reste la docstring à légèrement adapter, sans trop d'idée de la réalité de l'affaire, je proposerais :
'''use grsa referential to fill one or several link objects'''
Mis à jour par Thomas Noël il y a presque 5 ans
- Fichier 0001-solis-handle-list-of-links-33559.patch 0001-solis-handle-list-of-links-33559.patch ajouté
Frédéric Péters a écrit :
Il reste la docstring à légèrement adapter, sans trop d'idée de la réalité de l'affaire, je proposerais :
Yep yep. En fait la docstring d'origine était un peu n'importe quoi, j'ai tenté une explication qui devrait contenter le prochain qui mettra les doigts dans ce code au regard de la doc fournie par Solis (swagger).
- '''use grsa referentials to fill one or several link objects''' + '''fill one or several link (_links entry in grsa referential object)'''
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
allez, hop,
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6792df1742bb8825f092a9f7543fe85bd03456f2 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Wed May 29 17:15:57 2019 +0200 solis: handle list of links (#33559)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
solis: handle list of links (#33559)