Re: [Dolibarr-dev] Ca code

[Eldy]

Eric Seigne wrote:

> On Sat, Jan 24, 2004 at 11:26:17AM +0100, Rodolphe Quiedeville wrote:
>  
>
> > On Fri, 2004-01-23 at 19:31, Eric Seigne wrote:
> >    
> >
> > > Moi je suis dans  un truc qui me cause des vrai  gros soucis, sur la 1.1
> > > que j'utilise en prod, quand j'entre un paiement d'une facture, ce n'est
> > > pas mis dans la  table bank (de mémoire) ce qui fait  que je ne peux pas
> > > faire mes rapprochements de comptes ... 
> > >
> > >      
> > Tu peux inclure cela dans la banque par la suite, cela n'empeche pas les
> > rappros je suis d'accord que cela sera mieux quand l'ajout sera
> > automatique mais ce n'est pas bloquant.
> >    
>
> C'est fait ce soir, j'ai modifié le fichier htdocs/fourn/facture/paiement.php
> et la classe qui est avec pour afficher sur la page de paiement d'une facture
> fournisseur "compte à débiter" (à utiliser quand on paye par virement ou autre)
> et si c'est le cas hop on retrouvera cette transaction dans le compte bancaire
> lors des rappro à faire :)
>
> testé et validé ici, par contre un truc pas glop, je passe en production  sur
> la version CVS de dolibarr:(
>
> a+
> Éric
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Dolibarr-dev mailing list
> address@hidden
> http://mail.nongnu.org/mailman/listinfo/dolibarr-dev
>  
J'avais fait la meme modif. Meme pas eu le temps de lire mes mails et de 
commiter. Bien joué, Eric 1 - Eldy 0 !

J'ai cependant compléter avec quelques bouts de ma version pour les 
raisons suivantes:

Au moment de l'insertion, tu fais
   $sql = "INSERT INTO llx_bank (datec, dateo, amount, author, label, 
fk_account)";
   $sql .= " VALUES (now(), '$this->datepaye', '$this->amount', 
'$this->author', '$label', '$this->accountid')";

Ma version préconise plutot

   $sql = "INSERT INTO llx_bank (datec, dateo, amount, author, label, 
fk_type, fk_account, num_chq)";
   $sql .= " VALUES (now(), $this->datepaye, 
-$this->amount,'$this->author', '$this->label', '$this->paiementid', 
$this->accountid, $this->num_paiement)";
    
Tu noteras les différence suivante :
- Le type de paiement est aussi inséré (on a l'info, donc pourqui ne pas 
la mettre. Je fais juste une conversion en plus dans mon code pour 
transformé le numérique dans la chaine attendu comme pour le paiement 
client).
- Le numéro de chèque ou virement est également inséré (on a l'info 
brute, donc a la stocke. Elle facilitera le rapprochement).
- Enfin et c'est le plus important la valeur du montant doit etre entrée 
en négatif. En effet la table bank gère les débits crédits grace au 
signe. Elle est en positif dans paiementfourn mais doit etre négativé 
pour insertion dans llx_bank.

Pour le reste, c'est ok.

Autre sujet, j'ai ajouté la création de la table llx_co_fa dans le 
script de migration (j'étais en 1.1.0
et ne l'avais pas, d'ou des problemes, malgrè le passage du script, en 
passant en 1.2.0).


--
Laurent Destailleur for AWStats, AWBot and CVSChangeLogBuilder.
---------------------------------------------------------------
EMail: address@hidden
AWStats : http://awstats.sourceforge.net
AWBot : http://awbot.sourceforge.net
CVSChangeLogBuilder : http://cvschangelogb.sourceforge.net