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 : dolibarr-dev-bounces+address@hidden
[mailto:dolibarr-dev-bounces+address@hidden 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
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,
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
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
--
Eldy (Laurent Destailleur).
EMail: address@hidden
Web: http://www.destailleur.fr
Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: address@hidden
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: address@hidden
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev
--
Eldy (Laurent Destailleur).
EMail: address@hidden
Web: http://www.destailleur.fr
Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: address@hidden
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: address@hidden
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net