Produits Entr'ouvert » EOPayment

Ça faciliterait les choses plus loin (documentation, interface) si les différents backends utilisaient les mêmes noms pour des paramètres amenant la même chose (et comme aujourd'hui sips2 gère ça, je suggérerais de le prendre comme référence, plutôt qu'avoir à gérer ensuite dépréciation/migration).

À propos de dépréciation d'ailleurs, 'deprecated': True est déjà posé ici.

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

Ça faciliterait les choses plus loin (documentation, interface) si les différents backends utilisaient les mêmes noms pour des paramètres amenant la même chose (et comme aujourd'hui sips2 gère ça, je suggérerais de le prendre comme référence, plutôt qu'avoir à gérer ensuite dépréciation/migration).

Oui, mais le nom du paramètre ne me paraît pas clair: "capture_day" signifierait plutôt le jour de la capture et non le délai en jours. Je verrais plutôt le paramètre s'appeler "capture_days".

À propos de dépréciation d'ailleurs, 'deprecated': True est déjà posé ici.

Erreur de ma part, c'est plutôt un 'required': False qu'il faut.

Ok pour nommer pareil, mais l'explication ne doit pas être "Caption Day" qui ne fait référence à rien dans la doc Paybox.

Plutôt un 'caption': _('Nombre de jours pour un paiement différé'), comme leur doc et le reste du code (qui est déjà en french).

Aussi, la doc que j'ai lue http://www1.paybox.com/espace-integrateur-documentation/dictionnaire-des-donnees/paybox-system/ ne fait pas mention des 7 jours, et le fait qu'ils ont prévue deux chiffre pourrait laisser penser que non, y'a pas de limite absolue. Je serais presque pour la retirer de nos vérifications et compter sur un retour erreur paybox au cas où.

Je ne sais plus dans quel coin de la doc j'avais trouvé le texte:

Pour les paiements par carte, Verifone préconise au commerçant de ne pas dépasser 7 jours
entre la date de la demande d’autorisation et la date effective de remise en banque. Au-delà,
le commerçant peut avoir à gérer des impayés pour encaissement tardif.

mais tu as raison, laissons Paybox gérer cela.

Pour le self.capture_day > 0 je pense que capture_day ça va pas être un int, mais une str... non ? (ie ça marche, mais par chance). A priori les parameters, c'est que des str.

D'ailleurs, poser aussi un truc genre :

                'validation': lambda x: isinstance(x, basestring) and x.isdigit() and (0 < len(x) < 3),

Le début du test me semble pas bon, genre ça teste rien par rapport à « params['capture_day'] = 8 ». Selon moi il faudrait vérifier que ça donne bien PBX_DIFF == "08" -- et on verra à ce moment qu'il ne fallait pas retirer le zfill(2). Aussi, dans le test, vérifier aussi que l'absence de capture_day provoque bien l'absence de PBX_DIFF.

self.assertNotIn('PBX_DIFF', values), vu que values est une constante définie juste avant, ça me semble pas le bon test.