THELIA Forum

Lequel est juste ?

Bon après plusieurs test ce matin je viens de trouver le problème qui est certes bête mais ennuyant.

Pour être sur que le problème ne vienne pas de moi j'ai testé sur le template default. Au fait c'est tout simplement une erreur avec les décimales. Sur le récapitulatif les valeurs sont arrondies mais durant le calcul de la commande c'est les valeurs avec toutes leurs décimales qui sont utilisées ce qui donne au final un résultat différent à quelques centimes près de celui affiché dans le récapitulatif.

Bonjour,

@RONANAUPETIT En quelle version de Thelia es-tu ? Il y a un gros correctif en base de données en (2.2.0, 2.1.7) https://github.com/thelia/thelia/pull/1581

@INFORMATIQUEPROG Peux-tu retirer les rounds 2 de ton php dans la PR. Il faut éviter d'arrondir dans les fichiers core Thelia, car cela peut en effet créer des problèmes d'arrondi si un autre développeur utilise les méthodes des modèles pour faire des calcules. Il faut arrondir uniquement à l'affichage avec les fonctions smarty.
Il y a encore quelques round qui traînent dans le core, il y en a même quelques un dans les requêtes SQL Cela prend du temps à corriger, car il faut vraiment bien tester si le correctif corrige bien le bug et ne pose pas de problème pour les mises à jour.

@INFORMATIQUEPROG Il faut aussi faire la doc

@INFORMATIQUEPROG de préférence oui, il faudra surtout vérifier qu'il ne manque pas de format_money dans les templates. Enfin, on le verra avec le temps. Pour le moment, c'est pour l'alpha 1 de Thelia 2.3

Bonjour,

Je suis en 2.2.0, dois-je effectuer toutes les modifs de ta PR du coup ?

D'accord, merci à vous trois !