Hi Sebastien, Your pull request is in my 3.4 tree. I suggest you run 'git blame *filename*' to try and see if and who reverted your changes.
I looked at your fournisseur.product.class.php.patch and the changes suggested here were not in your pull request, at least not in your "Change field name recuperableonly to info_bits" commit. Looks like you're the one to blame. You simply overlooked some code. In my tree I have a bunch of references to recuperableonly all over the place. Maybe it's time to learn how to use grep or ack ;) FYI, here's the file list : htdocs/admin/dict.php htdocs/compta/journal/purchasesjournal.php htdocs/compta/journal/sellsjournal.php htdocs/core/class/html.form.class.php htdocs/core/lib/functions.lib.php htdocs/fourn/class/fournisseur.product.class.php htdocs/product/admin/product_tools.php htdocs/product/class/product.class.php htdocs/product/price.php I'm also suprprised to see a database field renaming without the proper code into the sql migration script in the install directory. Good luck fixing that. Once fixed, I suggest that you make a pull request on the 3.4 branch with a comment about the broken feature. Cheers ! Le 3 septembre 2013 14:09, Sébastien <seb-mail...@itoi.fr> 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<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 > Dolibarr-dev@nongnu.org > https://lists.nongnu.org/mailman/listinfo/dolibarr-dev > > -- *Raphaël Doursenaud* 05 35 53 97 13 - 06 68 48 20 10 rdoursen...@gpcsolutions.fr -- <http://gpcsolutions.fr> http://gpcsolutions.fr Technopole Hélioparc 2 avenue du Président Pierre Angot 64053 PAU CEDEX 9 SARL GPC.solutions au capital de 7 500 € - R.C.S. PAU 528 995 921 <https://www.google.com/a/partnersearch/#partner?partner_id=46687933_a0n60000000sqpWAAQ><http://wiki.dolibarr.org/index.php/Dolibarr_suppliers_France#GPC.solutions>
_______________________________________________ Dolibarr-dev mailing list Dolibarr-dev@nongnu.org https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
