|
From: | Laurent Destailleur (Eldy) |
Subject: | Re: [Dolibarr-dev] Bug arrondis prix unitaires |
Date: | Fri, 30 Mar 2007 01:25:55 +0200 |
User-agent: | Thunderbird 1.5.0.10 (Windows/20070221) |
Yannick Warnier a écrit :
Le mardi 27 mars 2007 à 10:22 +0200, Rodolphe Quiedeville a écrit :Yannick Warnier a écrit :Salut, Pour ceux qui n'ont pas suivi le bug 17811 http://savannah.nongnu.org/bugs/?17811 Je propose de faire la modification suivante dans le code, modification qui pourrait (si mal configurée) apporter pas mal d'effets de bord dans Dolibarr
ATTENTION GROS PAVE DANS LA MARE. :-)Pour moi la modif proposée n'est pas bonne. En effet, la fonction price est une fonction qui formate un montant pour l'affichage. Cette ne fonction ne devrait en aucun cas modifier une valeur en base ou en mémoire au moment de l'affichage. Si la valeur réelle est 1.2345 alors la fonction price doit renvoyer 1.2345 sinon cela signifie que ce que voit l'utilisateur n'est pas ce que contient la base ou la donnée, et la, c'est super grave.
La gestion des arrondis ne doit surtout pas se faire au moment de l'affichage (que ce soit écran HTML ou PDF) mais au moment du stockage en base, c'est à-dire au moment ou on insère un ligne détail dans une facture par exemple.
Attention à bien différencier la notion de décimale dans le prix final et le prix unitaire. 5 décimales sont intéressantes quand on vent des produits par 100 000 pièces, la cinquième décimale devient alors une dizaine dans le prix finales.Le problème étant que les décimales ne sont pas toujours arrondies comme on le voudrait, et que le nombre de décimales à utiliser pour les calculs intermédiaires dépend souvent du type de produits que l'on veut vendre. Ainsi, la solution que je propose est la suivante: 1) on ajoute deux variables de configuration globales (modifiables uniquement par l'admin) qui donnent un nombre de décimales par défaut selon le type de montant:$show_3_decimals_if_smaller_than = 1; $show_4_decimals_if_smaller_than = 0.5;2) le code de la fonction price(), qui est l'endroit où l'on effectue les arrondis jusqu'ici, deviendrait du type: ...// On pose par defaut 2 decimales $decimal = 2; $amount = ereg_replace(',','.',$amount); $abs = abs($amount); if($abs < $show_4_decimals_if_smaller_than){ $decimal = 4; }elseif($abs < $show_3_decimals_if_smaller_than){ $decimal = 3; }// ensuite la fonction price() arrondit en fonction de $decimal ... 3) On pourrait également (dans un second temps) proposer des paramètres à la fonction price() qui permettent de modifier le nombre de chiffres après la virgule au cas par cas. Je ne pense pas qu'il y ait une seule monnaie au monde qui propose une écriture à plus de 4 décimales mais je me trompe peut-être...Mes 2 centsMerci pour tes 2 cents. Je n'ai pas eu le temps de faire le développement en soi et je serai incapable de le faire avant dimanche, mais que ça n'empêche pas qui veut de faire une 2.1 RC1 ou même une release stable. Je réglerai le problème dimanche, promis. Yannick _______________________________________________ Dolibarr-dev mailing list address@hidden http://lists.nongnu.org/mailman/listinfo/dolibarr-dev
-- Laurent Destailleur. --------------------------------------------------------------- EMail: address@hidden Web: http://www.destailleur.fr IM: IRC=Eldy, Jabber=Eldy AWStats (Author) : http://awstats.sourceforge.net CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net AWBot (Author) : http://awbot.sourceforge.net Dolibarr (Contributor) : http://www.dolibarr.org
[Prev in Thread] | Current Thread | [Next in Thread] |