Ajouté : 15 juin 2005
NPDS (le blog)
J'essaie d'installer NPDS avec votre super interface d'installation 'easy' install.
C'est rare que je laisse des messages mais ca m'a veritablement saoulé (ya pas d autre mot desole)
d'avoir a matter votre source pour deceler le souci.
L'interface d'installation est tres mal faite:
Lorsque l'on rentre les informations SQL, *AUCUN* controle n'est fait pour s'assurer de la bonne
connexion au serveur SQL, excepté trois pages plus tard, et quand cette troisieme page plus tard
nous pete une erreur a la gueule, il faut revenir a la seconde page et tout recommencer, aucun lien
pour modifier la configuration, rien !
Bref, pas la peine de donner un nom aussi pompeux a une interface apparement aussi incomplete.
Seconde chose, le probleme je viens de le trouver, le fichier config.php n'etait pas ecrit correctement:
Vous controlez le HTTP_USER_AGENT (c'est a dire la machine du client) pour savoir sur quel type de systeme
vous allez ecrire le fichier ???! Je veux dire, c'est totalement illogique excepté si vous prenez en compte
le fait que la personne peut vouloir sauvegarder son fichier de configuration sur son systeme local.
Mais bref c est pas ca le pire, reprenons: donc si vous detectez un systeme windows en client, vous
ecrirez les retour a la ligne avec r seulement et dans le cas contraire avec un n ????!
Ola !
r = retour chariot, on reste sur la meme ligne et on revient au debut de cette meme ligne sur laquelle on vient
d ecrire,je vous epargne donc le copie colle de mon fichier de config qui devient inutile (puisqu aucun
saut de ligne donc la seule ligne == commentaire).
Un patch simple est d'ajouter $crlf = "n"; apres le code de l horreur:
$client = getenv("HTTP_USER_AGENT");
if(ereg("[^(]*((.*))[^)]*", $client, $regs))
{
$os = $regs[1];
if(eregi("Win", $os))
{
$crlf = chr(10);
}
}
else
{
$crlf = "n";
}
Bref sinon je m etonne que des gens aient pu l installer (!!) mais apparement ca a l air d etre super joli
donc bon boulot a ce niveau la, je reviendrais en foutre une couche si l occasion se presente ;)
PS: devoir foutre des
dans ce formulaire pour faire des sauts de ligne.. decidement
vous avez l'air d'etre de sacres experts en la matiere :)