Re: [Dolibarr-dev] NPR VAT

[Destailleur Laurent]

Hi Sébastien.

I am sorry too.

I checked your patch files and it seems most of them are included into the code. But if something does not work as you expect, there is surely something wrong somewhere with this feature.

We make a very big clean of code and uniformize a lot of field name into 3.4, trying to not break even not common features. Our main problem is that the npr feature has so few users using it than we don't have test unit for it to check feature is working correctly beteen each versions, and don't get bug report to check it.

Is the problem on all npr vat management (supplier and customer invoices) or only supplier invoice ?

I think your patch is still here but a regression appears at one place. It is enough may be to break all feature, but i don't think all your patch is lost. Your help to localize regression is surely appreciate (we don't have people knowing this feature like you in our core team).

PS: Don't worry for french. As you said, this feature is used only by french speaking people. Also i prefer your help in french than no help at all ;-)

Le 3 septembre 2013 14:09, Sébastien <address@hidden> a écrit :

hello,

They are lot of error with TVA npr and Dolibarr version 3.4.0
When i see the source code, my patches sent in march has removed ... i don't understand !
The old push request : https://github.com/Dolibarr/dolibarr/pull/805
For exemple, in htdocs/fourn/class/fournisseur.product.class.php the function "fetch_product_fournisseur_price" is completely broken, npr is removed !

Sorry but the next in french ...

Je ne comprends pas trop tout ces changements de code dans la version 3.4.0 au niveau de la tva NPR, c'est un bon retour en arrière ....
En pièce jointe les modifications pour la gestion de la npr avec un patch par fichier modifié et l'explication ci-dessous :

Fichier htdocs/fourn/class/fournisseur.product.class.php
Réintégration de la propriété fourn_tva_npr à partir du champ de base de donnée "info_bits"

Fichier htdocs/product/fournisseurs.php
Lors de l’édition d'un prix fournisseur existant, le champ tva est complété
Correction de l'appel de la fonction "update_buyprice" (le champ npr n’était pas au bon endroit)
Ajout de l'argument npr lors de l'appel de la fonction "vatrate" (Pour l'affichage des prix fournisseurs dans le tableau)

Fichier htdocs/product/price.php
Ajout de l'argument npr lors de l'appel de la fonction "vatrate" pour l'affichage et ajout pour le formulaire d’édition également


Pour l'instant pas d'autre soucis à signaler. Je continue ma petite recherche et j'envoie les patchs si il y en a.
Je me permets de parler pas mal en français, mais ce truc tordu qu'est la tva npr, à mon avis il n'y a qu'en France que ça existe .... et mon anglais, il faut l'avouer, est déplorable ! :)


_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev