Re: [Dolibarr-dev] Bug arrondis prix unitaires

[Laurent Destailleur (Eldy)]

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.

> > > 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...
> > >       
> > 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.
> >
> > Mes 2 cents
> >     
>
> Merci 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