[Dolibarr-dev] Vérification TVA

[Laurent Léonard]

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";

- 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.

- 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 ?

Merci,
-- 
Laurent Léonard