Produits Entr'ouvert » EOPayment

Et juste virer cette conversion ? On va retomber sur le bug avec les entiers, puis avec les flottants, etc. La validation est désormais faite en amont, #6710, dont d'ailleurs est probablement issu ce bug, j'écrivais sur le salon :

11:46:15 - vdeniaud: avant https://dev.entrouvert.org/issues/6710 (formulaire à la place de jsonfield), le paramètre manual_validation n'était pas renseigné, et du code dans eopayment lui donne sa valeur par défaut, le booléen False. Avec le patch, le formulaire récupère le défaut, et désormais la valeur est spécifiée en amont : elle passe donc par la moulinette qui la convertit en string, ce qui fait met en évidence le bug qui a toujours été là mais qui n'était pas déclenché jusqu'à présent.
11:47:34 - vdeniaud: mieux, le bug n'est pas déclenché au moment où le patch passe en prod, mais à la prochaine validation du formulaire, qui met à jour les paramètres du backend en base

Justement on convertit les flottants, entiers en chaînes.
Virer maintenant la conversion en string me paraît un peu risqué. Il faudrait repasser sur tous les backends et s'assurer que tout fonctionne.

Serghei Mihai a écrit :

Il faudrait repasser sur tous les backends et s'assurer que tout fonctionne.

À faire de toute façon IMO, il n'y a pas de raison que ce bug « un booléen est convertit en string et le backend attend un booléen » n'ait pas son pendant avec entier/flottant/liste/whatever.

Oui, mais je suis d'avis de ne pas le faire ici, car avec ce patch je souhaite corriger le bug en prod. Bug bloquant pour Strasbourg.

On peut vire la conversion, ça n'avait de sens qu'en python2 où on était un peu à la rue entre w.c.s. et combo.

J'ai peur de pousser à J moins quelques heures. Ça sera demain.

Serghei Mihai a écrit :

J'ai peur de pousser à J moins quelques heures. Ça sera demain.

Si tu veux pousser le premier patch pour ce soir, histoire de corriger un éventuel souci en prod, je ne vois rien contre.

Moi je suis d'accord que c'est risqué comme patch. Pour débloquer la situation, un petit patch safe ça serait d'ajouter PaymentBackend à apps/lingo/admin.py (il y a déjà Regie), tu pourrais configurer le truc comme avant avec le jsonfield, et ça resservira peut-être un jour.

Ah non oui « le premier patch », pourquoi pas.

Dans le premier patch (note 5) il y a :

-          transaction_id=transaction, time=time)
+          transaction_id=transaction, time=time, manual_validation=False)

Il sert à réproduire l'appel de lingo lors de la création de l'objet payment. Lancer le test sans:

if not isinstance(kwargs[param], bool):
    ...

Si dans eopayment/__init__.py::request on ne passe pas le paramètre manual_validation c'est la valeur par défaut de eopayment/paybox.py::request qui est prise en compte et est bien None et le code est bon.

Le souci est que dans @eopayment/__init__.py::request on convertit tout en string.

Serghei Mihai a écrit :

Il sert à réproduire l'appel de lingo lors de la création de l'objet payment.

Pigé. Go, donc.