RE: [Dolibarr-dev] commande : dernière lignedoublée

[Régis Houssin]

> 
> Je commence à regarder ça, et j'ai déjà quelques commentaires :
> 
> - le code source de la page est absolument illisible. Aucune
> indendation, aucun retour à la ligne dans toute la partie concernée.
> C'est affreux à analyser.

c'est André qui va se prendre des coups de bâtons ;-))

 
> - éviter les <td nowrap> en remplaçant pas <td nowrap="nowrap"> comme
> cela a déjà été fait par Laurent dans bon nombre de scripts.
> 
> - une erreur sur un href qui ne comporte pas les ""
> 
> - mais surtout, pourquoi charger 8 fois (autant que de lignes) la lib
> prototype.js ? Cela ralenti considérablement l'affichage, et je ne vois
> pas ce que cela apporte.
> 
> - ceci n'a pas beaucoup de sens dans la fonction select_produits :
>                       print '</script>';
>                       print '<script type="text/javascript">';


déjà merci pour avoir regardé.

j'ai essayé ta solution, ca fonctionne par contre une recherche met 3 fois
plus de temps pour créer la liste déroulante.

est-ce que tu vois ce que je veux dire ?

Régis


> - le bug vient de l'affectation de idprod8 avec la même valeur que le
> dernier produit sélectionné. A mon sens, le fait de déclarer autant de
> fois des fonctions comme publish_selvalue() qu'il n'y a de lignes est la
> cause de l'erreur.
> En extrayant :
> //                    print '<script type="text/javascript"
> src="'.DOL_URL_ROOT.'/lib/prototype.js"></script>';
> //                    print 'function
> publish_selvalue(obj){$("'.$htmlname.'").value =
> obj.options[obj.selectedIndex].value; alert(obj.name);}';
> du code de select_produit()
> et en rajoutant :
> if($conf->use_ajax) {
>       print '<script type="text/javascript"
> src="'.DOL_URL_ROOT.'/lib/prototype.js"></script>';
>       print '<script type="text/javascript">';
>       print 'function publish_selvalue(obj){$(obj.name).value =
> obj.options[obj.selectedIndex].value; alert(obj.name);}';
>       print '</script>';
> }
> avant la boucle :
>                               for ($i = 1 ; $i <= $NBLINES ; $i++)
> 
> Ça a l'air de fonctionner correctement.
> Bon, j'ai regardé ça en vitesse, je ne suis sûr de rien, mais c'est vrai
> que l'écriture du code ne fait pas du tout envie de s'y plonger.
> 
> --
> Christophe
> 
> 
> 
> _______________________________________________
> Dolibarr-dev mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/dolibarr-dev