Re: [Dolibarr-dev] Vérification TVA

[Laurent Destailleur (eldy)]

Le 18/10/2010 09:45, Laurent Léonard a écrit :
> Bonjour,
>
> Je suis content que la vérification TVA soit à nouveau fonctionnelle dans
> Dolibarr 2.9.0.
>
> 3 petites remarques cependant :
>
> - La fenêtre de popup fait 260 pixels de haut, ce qui oblige à scroller afin
> d'atteindre le bouton "Fermer" lorsque celle-ci affiche une réponse "standard"
> avec Iceweasel (Firefox) sous Debian GNU/Linux. L'agrandir à 285 pixels de
> haut corrigerait ce problème :
>
> --- htdocs/societe/soc.orig.php 2010-10-07 00:06:28.000000000 +0200
> +++ htdocs/societe/soc.php      2010-10-18 09:24:18.000000000 +0200
> @@ -1288,7 +1288,7 @@
>                  print "\n";
>                  print '<script language="JavaScript" type="text/javascript">';
>                  print "function CheckVAT(a) {\n";
> -               print
> "newpopup('".DOL_URL_ROOT."/societe/checkvat/checkVatPopup.php?vatNumber='+a,'".dol_escape_js($langs-
> > trans("VATIntraCheckableOnEUSite"))."',500,260);\n";
> +               print
> "newpopup('".DOL_URL_ROOT."/societe/checkvat/checkVatPopup.php?vatNumber='+a,'".dol_escape_js($langs-
> > trans("VATIntraCheckableOnEUSite"))."',500,285);\n";
>                  print "}\n";
>                  print '</script>';
>                  print "\n";
J'ai modifié
> - On retrouve ce même code Javascript 3 fois dans le fichier
> htdocs/societe/soc.php. J'ignore à quoi servent les 2 autres occurrences mais
> si elles ont une utilité je pense qu'il serait bien de réorganiser cela dans
> un but d'uniformisation.
En effet.
> - Aurait-on des avantages/inconvénients à utiliser une librairie Javascript de
> popup (comme ColorBox par exemple) plutôt qu'une nouvelle fenêtre de
> navigateur ?
La version 3.0 est en cours de refonte pour utiliser jquery. Le box 
seront donc surement gérée par jquery.

> Merci,