ESPACE D'ADMINISTRATION POUR VOTRE SITE INTERNET

Commentaire de malalam le 06/04/2008 21:11:19 administrateur CS

Hello,

difficile de commenter en détail ton code...ce qui m'amène à te faire le seul reproche que je peux te faire : ton code est très difficile à lire. Tu mélanges allègrement php, html, javascript...c'est le bordel.
Je te conseille d'assainir tout ça parce que c'est le genre de codes que, même en en étant l'auteur, on ne peut pas retoucher facilement. Et ça facilitera aussi la vie des éventuels développeurs voulant s'y plonger.
Sans compter que ton code n'est absolument pas prévu pour être réutilisé...tu mets TES url en dur dedans. Et je n'ai absolument pas compris (je n'ai pas trop cherché non plus) comment ce code permettait de se passer d'un client FTP vu que je n'ai pas trouvé de façon d'uploader des fichiers sur un serveur distant.
Je ne note pas, attendant quelques améliorations.

Commentaire de malalam le 06/04/2008 22:21:17 administrateur CS

Ah ok (merci Yoman :-) ). Donc l'upload se fait via copy() ? Cela ne marchera pas chez tous les hébergeurs mutualisés ça (les autres on s'en fout, on peut modifier la conf de php).

Pour le for(), en effet.
Néanmoins, pour faire plaisir à Malik ou JG je ne sais plus, je précise que cette syntaxe est ok (et à Yoman, pour autre chose, clin d'oeil...):
for($i = 0, $j = count($_POST); $i <= $j; ++$i)

Commentaire de yoman64 le 06/04/2008 23:21:46

Malalam: J'ai pas fait gaffe comment il copiait les fichiers. Puisque tu me dis que c'est avec copy alors tu as raison c'est pas beau (voir move_uploaded_file). Moi j'avais juste chercher le formulaire comme tel, pas comment il déplaçait les fichiers.

Pour les for j'aime bien cette syntaxe je l'utilise bien souvent (autant le ++i que la déclaration de j dans le for :) ) je la trouve propre et claire.

J'en profite pour clarifier une chose: Je fais des remarques dans le but d'améliorer ton code, pas dans le but de te critiquer personnellement. Au contraire ! dans ton code il y a même plusieurs points positif même si je n'ai pas pris la peine de les nommées. Par exemple tu utilises parfaitement les quotes/doubles quotes, c'est une excellente chose! Je précisais ça suite aux commentaires de GillesWebmaster sur ton autre source, je ne veux pas de malentendu...

Note à un admin: Il serait bien de suggèrer à Nix(je pourrais toujours le faire, mais toujours plus directe si ça vient d'un admin :P) d'ajouter les .htaccess comment étant des fichiers visionnable au même titre que les .php et .css !

Commentaire de Vyk12 le 13/04/2008 13:09:17

Voilà j'ai fais toutes les modifications que vous m'avez citées, sauf une : la sécurité. Etant donné qu'il n'y a que l'administrateur qui a accès au répertoire, je ne vois vraiment pas pourquoi mettre des limites/contraintes (à moins que l'on veuille se pirater son propre site ...). Après si vous voulez mettre des htmlspecialchars(), libre à vous de faire ce choix.

Sinon je le redis, j'ai fais toutes les modifications que vous m'avez dites (j'en ai peut-être oublié une ou deux par erreur, dites-le moi si c'est le cas svp)

@+ !

Commentaire de Vyk12 le 21/04/2008 14:35:27

C'est une erreur interne du serveur, ça peut être dû à beaucoup de choses ... vérifie ton .htaccess pour voir s'il y a des erreurs, ou sinon essaye-le en ligne carrément, et non pas en local ... sinon désolé de ne pas pouvoir t'aider !

Commentaire de yoman64 le 21/04/2008 20:39:57

Je vois pas trop ce que ton PerlSetVar fait la, sachant que AuthFile est bien une instruction de configuration pour apache ...