Re: [Dolibarr-dev] Fonction price() et split("\.",$amount)

[Yannick Warnier]

Le mardi 31 juillet 2007 à 00:33 +0200, Laurent Destailleur (Eldy) a
écrit :
> En toute logique le bon code c'est
> 
> '\.' et non '.'
> En effet, le . en regex nveut dire n'importequel caractere. Hors on veut 
> coupé la partie entiere et decimal sur le point.
> Et pour eviter tout souci avec le , ces dernieres sont converties en . 
> juste avant. Tout devrait donc fonctionner.
> 
> Peut-tu donner un exemple de cas qui ne fonctionne pas. Peux tu afficher 
> la trace     //print "amount=".$amount." html=".$html." trunc=".$trunc." 
> nbdecimal=".$nbdecimal." dec=".$dec." thousand=".$thousand;
> en ligne 1972 sur la version 1.328 dans ce cas.
> Car tout devrait fonctionner en l'état.

OK, en effet, tu as raison. Il y avait un effet de bord bizarre quand
j'ai retiré le \ qui a supprimé la partie après le point, ce qui a rendu
les montants un peu plus lisibles.

En attendant je pensais qu'on avait réglé le problème du nombre de
chiffres après la virgule dans la version 2.1 mais il semblerait que,
par exemple, la constante MAIN_MAX_DECIMALS_SHOWN n'est pas créée par le
script de migration.

Cette constante intervient par ailleurs uniquement sur l'affichage des
montants, donc on a toujours des montants calculés avec TVA de 18,3475,
mais bon, tant pis.

Merci pour la correction.

Yannick


> 
> Yannick Warnier a écrit :
> > Salut,
> >
> > Dans la fonction price() (htdocs/lib/functions.inc.php), au moins dans
> > la version 2.1 beta sur CVS, il y a la ligne suivante:
> >
> > $datas = split("\.",$amount);
> >
> > Or la définition de la fonction split() ne fait pas mention d'un type de
> > formatage particulier pour la chaîne de caractère qui sert de
> > délimiteur...
> >
> > En fait, ici je me retrouve avec plein de factures fournisseur où il
> > confond . et , (ça donne de très grosses dettes envers les fournisseurs
> > dans l'écran de factures fournisseur par exemple).
> >
> > En retirant le "\", tout refonctionne à nouveau. Des commentaires avant
> > que je l'enlève de la fonction?
> >
> > Yannick
> >
> >
> >
> > _______________________________________________
> > Dolibarr-dev mailing list
> > address@hidden
> > http://lists.nongnu.org/mailman/listinfo/dolibarr-dev
> >
> >   
> 
>