Bonjour, J'ai fait les modifications, voir ci-dessous pour le détail. (commit : https://github.com/zedzedtop/dolibarr/commit/5921033f9437e7ab7bb135c7fde734431ab9978d) Si tout est ok, je pousse l'ensemble des commits sur la branch develop de dolibarr. Par contre je n'ai pas trouvé où se gère les modifications de base pour les mise à jour, actuellement seul l'installation d'une nouvelle instance dolibarr gèrera les nouveaux champs en base.
Il ne restera que les modifications sur le formulaire de saisi des prix fournisseur (uniquement de l'affichage du coup), je devrais pouvoir le faire d'ici la fin de semaine (j'ai plein d'autre patch à mettre au propre ...) Cordialement, Sébastien De : [email protected] [mailto:[email protected]] De la part de Laurent Destailleur (eldy) Envoyé : jeudi 28 mars 2013 00:54 À : Posts about Dolibarr ERP & CRM development and coding Objet : Re: [Dolibarr-dev] TVA NPR Le 26/03/2013 05:54, Sébastien mailing a écrit : Bonjour, Quelques corrections/patch concernant la tva npr : · La table llx_product_fournisseur_price Elle ne comporte pas d'information sur la tva npr, le champ tva est un réel. Solution : ajouter un champ recuperableonly de type entier avec comme valeur par défaut 0. On le passera à 1 quand la tva sera npr. Commit : https://github.com/zedzedtop/dolibarr/commit/be8a95a22b6ad7bb9bd4e64f19ff14235aaa75a9 Afin d'etre uniforme avec les facture clients, le champ a été nommé comme les factures clients, soit info_bits. As tu moyen de pousser ce commit sur la branch develop de dolibarr ? Changement du nom du champ pour la table product_fournisseur_price · La table llx_facture_fourn Comme précédemment, il faut rajouter le champs pour la gestion de la tva npr : ajout d'un champ info_bits comme sur les factures clients. Commit : https://github.com/zedzedtop/dolibarr/commit/be8a95a22b6ad7bb9bd4e64f19ff14235aaa75a9 La gestion tva se fait au niveau ligne et non au niveau facture, donc cela ne devrait pas etre nécessaire. Une facture peut tres bien avoir 3 lignes, dont une de tva 19,6, une de tva y et une autre seulement avec la tva npr. je me suis trompé dans mon explication, il s'agit de la table « facture_fourn_det » qui nécessite bien un champ supplémentaire « info_bits » (le commit porte bien sur cette table) · La classe facture fournisseur (htdocs/fourn/class/fournisseur.facture.class.php) Prise en compte de la tva npr dans la récupération et la mise à jour des lignes de facture. Commit : https://github.com/zedzedtop/dolibarr/commit/6cfedfff9fb205dd5ec78f4e5a54d1532862a5b0 Cela me semble ok. As tu moyen de pousser ce commit sur le git repo de dolibarr ? · La classe ProductFournisseur (htdocs/fourn/class/fournisseur.product.class.php) Prise en compte de la tva npr dans les méthodes le necessitant. Commit : https://github.com/zedzedtop/dolibarr/commit/b2b0d1616defca62e833381cbcbe95ec15991f6e Afin d'etre uniforme avec la gestion faite pour les factures client, on place une etoile apres le taux de tva dans le param $tva_tx pour dire que c'est une tva npr As tu moyen de faire ainsi aussi ? Je ne vois pas trop ou je dois gérer le taux de tva avec *. Si je regarde les factures clients, le paramètre info_bits est envoyé en paramètre, la décomposition de la tva (taux et *) est fait avant l'appel de la fonction. C'est donc le fichier htdocs/product/fournisseurs.php qui doit être modifié. J'ai ajouté le commit : https://github.com/zedzedtop/dolibarr/commit/eda61879821af0f2d4bd9f00ee47e88aa099e1c3 · La classe Product (htdocs/product/class/product.class.php) Calcul du prix en prenant en compte la tva npr Commit : https://github.com/zedzedtop/dolibarr/commit/aaaa5242d6a2b05102b8d6ca7475c487c428df7c Il faudrait faire $newnpr = preg_match('/\*/',$tva_tx)?1:0 pour définir si taux de tva npr ou pas. Comme précédemment, je ne vois pas pourquoi je dois gérer le * dans le taux de tva, la variable $newnpr étant envoyé en paramètre de la fonction. Le * est traité dans le code précèdent les appels à la fonction (sauf un : htdocs/product/price.php:683) mais je ne vois pas trop ce que fait ce code ici ... Les autres appels me semblent correct : htdocs/product/admin/product_tools.php:101: $ret=$objectstatic->updatePrice($objectstatic->id, $newprice, $price_base_type, $user, $newvat, $newminprice, $newlevel, $newnpr); htdocs/product/price.php:91: if ($object->updatePrice($object->id, $newprice, $newpricebase, $user, $newvat, $newprice_min, $level, $newnpr, $newpsq) > 0) htdocs/product/price.php:116: $object->updatePrice($object->id, 0, $object->price_base_type, $user, $object->tva_tx, 0, $level, $object->tva_npr, 1); · La fiche facture fournisseur (htdocs/fourn/facture/fiche.php) Ajout du paramètre npr aux appel de fonction et prise en compte du code de tva avec * à la fin indiquant que la tva est npr Commit : https://github.com/zedzedtop/dolibarr/commit/bbbc4ad8c44af76b01a622e0a8a032b7cb557e27 · Le journal des ventes et des achats (htdocs/compta/journal/purchasesjournal.php et htdocs/compta/journal/sellsjournal.php) Les lignes sont dupliqués (left join pas bon) et on enlève la tva npr de l'affichage. Sur ce point je ne suis pas sûr de devoir enlever les lignes NPR, j'attends la confirmation de mon comptable. Commit : https://github.com/zedzedtop/dolibarr/commit/148ad65150ca44f2a4a9c8d930a1ad0ed42665da J'ai eu la confirmation, la tva npr ne dois pas apparaitre dans le journal, mon code doit être bon · La function get_default_npr (htdocs/core/lib/functions.lib.php) La function etait vide et renvoyait toujours 0. J'ai mis les test qui vont bien, mais je ne suis pas sur à 100% que tout les cas de figures sont traités. Pour l'instant pas de bug de mon côté avec la fonction comme ça. Commit : https://github.com/zedzedtop/dolibarr/commit/28999af70b4986854a4fb524a556db981078f60e · La sélection de la tva sur la page « prix fournisseur » Comme décrit ci-dessous, je proposerais un patch qui : désactive le champ tva au chargement du formulaire vierge, recharge automatiquement la page lorsque le fournisseur est choisi ou modifié, active et peuple le champ tva en fonction du choix du fournisseur. Avec les patchs ci-dessus on peut déjà utilisé la tva npr en indiquant * dans le champ de saisie du prix fournisseur Comme pour les autres cas, as tu moyen de pousser ces commit sur le repo develop de dolibarr afin de permettre l'intégration en quelques clics ? Cordialement, Sébastien De : [email protected]<mailto:[email protected]> [mailto:[email protected]] De la part de Sébastien mailing Envoyé : vendredi 22 mars 2013 15:20 À : Posts about Dolibarr ERP & CRM development and coding Objet : Re: [Dolibarr-dev] TVA NPR Bonjour, Je regarde pour fournir tous les patchs nécessaires. Il y a en fait beaucoup de modifications à faire, il manque des petites bouts un peu partout. Je travaillais sur la version 3.2.1, je suis passé à la 3.3.1 pour être à jour niveau code et j'ai pas mal de régression, du coup la tva npr est mise de côté en attendant que je corrige tout ça. Je vous tiens au courant des que j'avance la dessus. Cordialement, De : [email protected]<mailto:[email protected]> [mailto:[email protected]] De la part de Laurent Destailleur (eldy) Envoyé : mercredi 20 mars 2013 19:11 À : [email protected]<mailto:[email protected]> Objet : Re: [Dolibarr-dev] TVA NPR Le 11/03/2013 08:39, Sébastien mailing a écrit : Bonjour tout le monde, Nouvel utilisateur Dolibarr depuis quelques mois et résidant dans les dom, je rencontre de multiples problèmes avec la gestion de la tva NPR (tva non perçue récupérable). Je m'aperçois, malgré l'annonce de l'implémentation de celle-ci sur le site de Dolibarr, qu'elle n'est pas du tout fonctionnelle. Premier problème : Lorsque je saisie un prix fournisseur pour un produit, je dois saisir la tva à la main, donc saisir un réel, ce qui ne permet pas de spécifier que la tva est en NPR. Si je vais voir dans le code (fichier htdocs/product/fournisseurs.php ligne 310), j'ai la ligne avec le « select » de tva commenté : //print $form->load_tva('tva_tx',$product->tva_tx,$supplier,$mysoc); // Do not use list here as it may be any vat rates for any country En gros, le select n'est pas utilisé parce que le fournisseur n'est pas encore choisi, donc on ne connait pas le pays pour lequel cela s'applique, donc on ne peut pas afficher une liste de tva fiable. J'y vois deux solutions : - La tva n'est modifiable qu'une fois qu'on a choisi le fournisseur : select désactivé lors de l'affichage du formulaire vierge, rechargement de la page lors de la sélection du fournisseur et affichage des tva disponibles. Je suis preneur pour un patch qui réaliserait cette option (celle du dessus), avec rechargement quand on a choisi le fournisseur. - La tva est disponible dès le chargement du formulaire : on charge l'ensemble des taux de tva pour tous les fournisseurs, lors de la confirmation du formulaire on vérifie que le taux de tva est possible pour ce fournisseur Je peux faire les modifications et proposer les patch nécessaires au bon fonctionnement de la tva NPR, j'aimerais juste savoir quelle est la meilleur façons de faire pour que mes patch soient acceptés. Pour cela, l'url fourni par Florian déjà doit etre suffisante: http://wiki.dolibarr.org/index.php/Developer_FAQ#Submit_a_patch.2C_work_and_help_on_Dolibarr_development Cordialement, Sébastien de Chateauvieux _______________________________________________ Dolibarr-dev mailing list [email protected]<mailto:[email protected]> https://lists.nongnu.org/mailman/listinfo/dolibarr-dev _______________________________________________ Dolibarr-dev mailing list [email protected]<mailto:[email protected]> https://lists.nongnu.org/mailman/listinfo/dolibarr-dev -- Eldy (Laurent Destailleur). EMail: [email protected]<mailto:[email protected]> Web: http://www.destailleur.fr Dolibarr (Project leader): http://www.dolibarr.org To make a donation for Dolibarr project via Paypal: [email protected]<mailto:[email protected]> AWStats (Author) : http://awstats.sourceforge.net To make a donation for AWStats project via Paypal: [email protected]<mailto:[email protected]> AWBot (Author) : http://awbot.sourceforge.net CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net _______________________________________________ Dolibarr-dev mailing list [email protected]<mailto:[email protected]> https://lists.nongnu.org/mailman/listinfo/dolibarr-dev -- Eldy (Laurent Destailleur). EMail: [email protected]<mailto:[email protected]> Web: http://www.destailleur.fr Dolibarr (Project leader): http://www.dolibarr.org To make a donation for Dolibarr project via Paypal: [email protected]<mailto:[email protected]> AWStats (Author) : http://awstats.sourceforge.net To make a donation for AWStats project via Paypal: [email protected]<mailto:[email protected]> AWBot (Author) : http://awbot.sourceforge.net CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
_______________________________________________ Dolibarr-dev mailing list [email protected] https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
