Re: [Dolibarr-dev] tout mes changes

[Franky Van Liedekerke]

deuxieme chose a ameliorer (de nouveau: pas de changements dans la
fonctionalite):

dans dolibarr/htdocs/commande/fiche.php, remplace:

$commande->add_product($_POST['idprod1'],$_POST['qty1'],$_POST['remise_percent1']);
$commande->add_product($_POST['idprod2'],$_POST['qty2'],$_POST['remise_percent2']);
 
$commande->add_product($_POST['idprod3'],$_POST['qty3'],$_POST['remise_percent3']);
$commande->add_product($_POST['idprod4'],$_POST['qty4'],$_POST['remise_percent4']);
$commande->add_product($_POST['idprod5'],$_POST['qty5'],$_POST['remise_percent5']);
$commande->add_product($_POST['idprod6'],$_POST['qty6'],$_POST['remise_percent6']);
$commande->add_product($_POST['idprod7'],$_POST['qty7'],$_POST['remise_percent7']);
$commande->add_product($_POST['idprod8'],$_POST['qty8'],$_POST['remise_percent8']);

par (code copie d'une autre partie):
$NBLINES=8;
for ($i = 1 ; $i <= $NBLINES ; $i++)
{
   if ($_POST['idprod'.$i])
   {
      $xid = 'idprod'.$i;
      $xqty = 'qty'.$i;
      $xremise = 'remise_percent'.$i;
      $commande->add_product($_POST[$xid],$_POST[$xqty],$_POST[$xremise]);
   }
}


Et encore:
dans dolibarr/htdocs/contrat/fiche.php:
   $date_start=mktime ...
   $date_end=mktime...
sont peut-etre mieux:
   $date_start=dolibarr_mktime...
   $date_end=dolibarr_mktime


Franky

On Sat, 27 Jan 2007 14:40:44 +0100
Franky Van Liedekerke <address@hidden> wrote:

> Je ne peut pas faire cela, parce-que tous mes changements sont
> integres deja.
> Mais on peut deja simplifier les choses:
> 
> commencons par remplacer:
> if ($objp->fk_product_type)
> par
> if ($objp->fk_product_type==1)
> dans les fichiers:
> dolibarr/htdocs/comm/propal.php
> dolibarr/htdocs/commande/fiche.php
> dolibarr/htdocs/telephonie/client/facture.php
> dolibarr/htdocs/compta/commande/fiche.php
> dolibarr/htdocs/compta/facture.php
> dolibarr/htdocs/compta/propal.php
> dolibarr/htdocs/fourn/product/index.php
> dolibarr/htdocs/fourn/product/liste.php
> dolibarr/htdocs/includes/boxes/box_produits.php
> dolibarr/htdocs/includes/boxes/box_services_vendus.php
> dolibarr/htdocs/product/index.php
> dolibarr/htdocs/product/liste.php
> dolibarr/htdocs/product/popuprop.php
> dolibarr/htdocs/product/reassort.php
> 
> (cela ne change rien, fk_product_type ne peut etre que 0 ou 1 pour le
> moment, mais c'est un debut pour plus tard, et plus correcte aussi)
> 
> et aussi:
> "need to be" ==> "needs to be" dans les fichiers langs/en_US/*
> 
> cela va resoudre deja les petits differences (alors mon diff devient
> plus petit)
> 
> Franky
> 
> On Sat, 27 Jan 2007 09:25:01 +0000
> Yannick Warnier <address@hidden> wrote:
> 
> > Le samedi 27 janvier 2007 à 10:17 +0100, Franky Van Liedekerke a
> > écrit :
> > > On Sat, 27 Jan 2007 08:21:54 +0000
> > > Yannick Warnier <address@hidden> wrote:
> > > 
> > > > Salut,
> > > > 
> > > > Vu qu'on est tous motivés à sortir la 2.1.0 bientôt, je te
> > > > recommande d'attendre encore un peu. Les derniers changements
> > > > causent quelques problèmes par-ci par-là. 
> > > 
> > > sauf qu'il-y-a encore des bugs dans CVS (voir la description de
> > > mes changements) pour different choses.
> > 
> > Oui Franky, j'ai bien compris, seulement je suis aussi *dangereux*
> > que toi au niveau connaissance du code, alors je préfère attendre
> > que quelqu'un intègre tes propositions de changement plutôt que de
> > le faire moi-même.
> > 
> > Par ailleurs, pourrais-tu envoyer tes changements (pas "changes") en
> > udiff comme tu le proposes, afin que celui qui veut les revoir ait
> > directement tout ce qu'il faut à disposition?
> > 
> > Tu peux les envoyer sur la liste, séparés par bug (pas par fichier)
> > par exemple, et quelqu'un finira bien par s'en occuper. Ou tout
> > ensemble c'est bien aussi mais ce sera plus compliqué d'analyser le
> > code problème par problème.
> > 
> > Merci,
> > 
> > Yannick
> > 
> > 
> > 
> > _______________________________________________
> > Dolibarr-dev mailing list
> > address@hidden
> > http://lists.nongnu.org/mailman/listinfo/dolibarr-dev
> > 
> 
> 
> _______________________________________________
> Dolibarr-dev mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/dolibarr-dev
>