Project

General

Profile

Development #8201

teamnet: service de notification de paiement d'une facture

Added by Serghei Mihai almost 6 years ago. Updated about 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
Start date:
07 Sep 2015
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Files

Associated revisions

Revision 362796a1 (diff)
Added by Serghei Mihai over 5 years ago

contrib.teamnet_axel: invoice payment notify endpoint (#8201)

History

#1

Updated by Benjamin Dauvergne almost 6 years ago

On peut donner un nom à une URL généré dans un template:

  <a href="{% url 'teamnet-axel-invoice-payment' slug=object.slug regie_id='42' family_id='999' invoice_id='20150808' %}">
    {{ site_base_uri }}{% url 'teamnet-axel-invoice-payment' slug=object.slug regie_id='42' family_id='999' invoice_id='20150808' %}?NameID=..

->

  <a href="{% url 'teamnet-axel-invoice-payment' slug=object.slug regie_id='42' family_id='999' invoice_id='20150808' as payment_url %}">
    {{ site_base_uri }}{{ payment_url }}?NameID=..

C'est pour faire un exemple les valeurs données pour regie_id, family_id et invoice_id ?

def get_billing_regie(slug):
    billing_ids = {'42': 'EN29-RESTC',
                   '31': 'EN31-C.V.'}
    return billing_ids.get(slug)

Settings ?

#2

Updated by Serghei Mihai almost 6 years ago

Benjamin Dauvergne a écrit :

C'est pour faire un exemple les valeurs données pour regie_id, family_id et invoice_id ?

Oui,

Settings ?

C'est spécifique à Teamnet: l'identifiant de la régie, utilisé pour lister les factures n'est pas le même que celui utilisé pour le paiement. Donc je préfére que ça reste dans le code du connecteur.

#3

Updated by Benjamin Dauvergne almost 6 years ago

C'est spécifique à teamnet ou à teamnet "à" Fontenay ?

#4

Updated by Serghei Mihai almost 6 years ago

Je dirais à teamnet en général, vu les infos fournies par Nathalie(exemple de capture du logiciel Teamnet à Fontenay: https://dev.entrouvert.org/attachments/download/7571/regie_2.png)

#5

Updated by Benjamin Dauvergne almost 6 years ago

Ça ressemble à du paramétrage local quand même.

#6

Updated by Serghei Mihai almost 6 years ago

  • File 0001-contrib.teamnet_axel-invoice-payment-notify-endpoint.patch added

En fait oui, les valeurs des régies et des régies d'encaissement sont spécifiques à Fontenay.
Ça a tout à fait sa place dans settings

#7

Updated by Serghei Mihai almost 6 years ago

  • File deleted (0001-contrib.teamnet_axel-invoice-payment-notify-endpoint.patch)
#8

Updated by Serghei Mihai almost 6 years ago

  • File 0001-contrib.teamnet_axel-invoice-payment-notify-endpoint.patch added
#9

Updated by Benjamin Dauvergne almost 6 years ago

Ce serait pas l'occasion de tenter de faire de tests avec mock (pour simuler les appels à Teamnet) ?

#10

Updated by Serghei Mihai almost 6 years ago

  • Target version set to 0.10
#11

Updated by Serghei Mihai over 5 years ago

  • File deleted (0001-contrib.teamnet_axel-invoice-payment-notify-endpoint.patch)
#12

Updated by Serghei Mihai over 5 years ago

  • File 0001-contrib.teamnet_axel-invoice-payment-notify-endpoint.patch added

Oui il faudrait, je ferai à patch séparé

#13

Updated by Serghei Mihai over 5 years ago

  • File deleted (0001-contrib.teamnet_axel-invoice-payment-notify-endpoint.patch)
#15

Updated by Frédéric Péters over 5 years ago

s/Correspondence/mapping/.

Je ne trouve vraiment pas clair qu'un appel à get_data() fasse une action; même si c'est juste un alias, avoir post_data serait bien mieux.

#17

Updated by Frédéric Péters over 5 years ago

Oula, c'était pas ça du tout que je voulais dire.

Il y a ça dans pay_invoice :

data = self.get_data('PaiementFactures', payment_xml)

Et c'est ça que je trouvais très curieux.

Les changements pour que ce soit du POST de notre côté, je ne les ferais pas, histoire de ne pas partir dans une nouvelle série de commentaires.

#18

Updated by Serghei Mihai over 5 years ago

Frédéric Péters a écrit :

Oula, c'était pas ça du tout que je voulais dire.

Il y a ça dans pay_invoice :

[...]

Et c'est ça que je trouvais très curieux.

C'est une fonction générique, appelée par toutes les méthodes pour formater la requete pour Teamnet et retourner la réponse.
Je ne vois pas pourquoi elle devrait avoir un alias post_data.

#19

Updated by Frédéric Péters over 5 years ago

Parce qu'à la lecture du code, qui est une partie importante du travail, c'est plus facile de connaitre l'intention de cet appel de méthode quand il est écrit post_data() (ou send_data() si on ne veut pas empiéter sur le mot "post"), qu'on n'a ainsi pas à aller voir que get_data(), en fait, ça peut aussi servir à envoyer des données.

#20

Updated by Frédéric Péters over 5 years ago

  • Status changed from Nouveau to Résolu (à déployer)

Ça a été poussé (sans appliquer ma suggestion).

#21

Updated by Serghei Mihai about 4 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF