calculate prices with generic local taxes management

[altairis-tof]

No description provided.

[altairis-tof]

wait please before merge, i have to update the commit

[altairis-tof]

should be ok now

[eldy]

J'ai intégré le push.
J'ai fais quelques modifs pour rendre la situation compatible avec l'espagne. Tout semble refonctionner à nouveau et le code en l'état doit etre opérationnel pour tout le monde sans régression, en l'état, meme pour l'espagne.
Merci d'attendre que Juanjo ait fait quelques tests sur cet état avant de passer à la suite (l'idéal serait d'ajouter des cas de tests dans le PHPUnit PricesTest.php afin de valider différentes combinaisons de cas).

[altairis-tof]

je viens de regarder, il me semble que dans ton test de localtax2 pour l'espagne, tu devrais passer $localtax2_rate en négatif, puisqu'avant on soustrayais la taxe2

[simnandez]

Oui, le localtax2 a l'Espagne est negatif, on soustrais

[eldy]

Le 11/09/2012 08:26, Christophe Battarel a écrit :

je viens de regarder, il me semble que dans ton test de localtax2 pour
l'espagne, tu devrais passer $localtax2_rate en négatif, puisqu'avant
on soustrayais la taxe2

En effet, mais j'ai demandé à Juanjo de voir si un taux négatif pouvait
etre ok.
Si c'est ok, on modifiera plutot la table llx_c_tva pour mettre le
localtax2_rate négatif.

Il testera dans les jours qui viennent...

—
Reply to this email directly or view it on GitHub
#382 (comment).

Eldy (Laurent Destailleur).

EMail: eldy@destailleur.fr
Web: http://www.destailleur.fr

Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: contact@destailleur.fr
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: contact@destailleur.fr
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net

[eldy]

Le 11/09/2012 08:53, Juanjo Menent a écrit :

Oui, le localtax2 a l'Espagne est negatif, on soustrais

—
Reply to this email directly or view it on GitHub
#382 (comment).

Donc si on met le taux négatif dans la table llx_c_tva, cela doit
fonctionner.
C'est possible cela (cela evitera d'avoir à gerer une soustraction dans
calcul_price_total) ?

Eldy (Laurent Destailleur).

EMail: eldy@destailleur.fr
Web: http://www.destailleur.fr

Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: contact@destailleur.fr
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: contact@destailleur.fr
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net

[simnandez]

J'ai fait le test en pousant a le llx_c_tva le localtax2 en negatif et il fonctionne tres bien. (il faut faire quelques modifications parce que j'ai vu certains bugs a l'insertion et update de lignes dans propals, par exemple, je les vais a resoudre...). Je vais parler à un conseiller fiscal pour être sûr de savoir comment appliquer les deux taux en Espagne

[eldy]

Le 12/09/2012 15:21, Juanjo Menent a écrit :

J'ai fait le test en pousant a le llx_c_tva le localtax2 en negatif et
il fonctionne tres bien. (il faut faire quelques modifications parce
que j'ai vu certains bugs a l'insertion et update de lignes dans
propals, par exemple, je les vais a resoudre...). Je vais parler à un
conseiller fiscal pour être sûr de savoir comment appliquer les deux
taux en Espagne

—
Reply to this email directly or view it on GitHub
#382 (comment).

Super.
Tiens nous au courant. Je donnerais le go à Christophe pour lancer la
suite de la gestion des taux multiples dès que tu auras pu corriger les
bugs que tu as identifiés.

Eldy (Laurent Destailleur).

EMail: eldy@destailleur.fr
Web: http://www.destailleur.fr

Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: contact@destailleur.fr
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: contact@destailleur.fr
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net

[eldy]

Hi christophe,

can you push the next step: This next step is a patch on the dictionnary
setup of local taxes table to be able to set localtax type (for the
moment only localtax rate is possible).

Le 12/09/2012 15:26, Laurent Destailleur (eldy) a écrit :

Le 12/09/2012 15:21, Juanjo Menent a écrit :

J'ai fait le test en pousant a le llx_c_tva le localtax2 en negatif
et il fonctionne tres bien. (il faut faire quelques modifications
parce que j'ai vu certains bugs a l'insertion et update de lignes
dans propals, par exemple, je les vais a resoudre...). Je vais parler
à un conseiller fiscal pour être sûr de savoir comment appliquer les
deux taux en Espagne

—
Reply to this email directly or view it on GitHub
#382 (comment).

Super.
Tiens nous au courant. Je donnerais le go à Christophe pour lancer la
suite de la gestion des taux multiples dès que tu auras pu corriger
les bugs que tu as identifiés.

Eldy (Laurent Destailleur).

EMail:eldy@destailleur.fr
Web:http://www.destailleur.fr

Dolibarr (Project leader):http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal:contact@destailleur.fr
AWStats (Author) :http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal:contact@destailleur.fr
AWBot (Author) :http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) :http://cvschangelogb.sourceforge.net

Eldy (Laurent Destailleur).

EMail: eldy@destailleur.fr
Web: http://www.destailleur.fr

Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: contact@destailleur.fr
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: contact@destailleur.fr
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net

[altairis-tof]

ok