MOO! PHOTO UPLOADER

Commentaire de vega11 le 05/02/2008 19:46:12

Bonjour,

Pour isValidImage il ne faut pas en tenir compte c'est un oubli facheux je dois le supprimer, en effet il ya deja une methode dans la ImageTransform pour verifier si c'est une image.

La classe ImageTransform n'est pas de moi c'est indique dans la classe...j'aurais du le preciser mais tout est commenté dans la classe et elle est en effet instancié apres la declaration de la classe puisque l'on utilise un seul objet de cette classe.

Pour move il me semble que l'upload est verifie puisque je retourne la reponse de if
move_uploaded_file sinon mon test ne marcherait pas lorsque je fais if($img->move) cela renvoi bien un boolean ^^ donc je ne vois pas vraiment le probleme.

Pour la base c'est une classe basique le but n'etait pas de faire tout un acces à la base avec classe abstraite + builder de base etc mais oui pourquoi pas.

Pour le petit plus c'est surtout que pendant l'upload on peut toujours faire des actions sur la page actuelle ce qui reste tres interessant, l'utilisateur n'est pas géné par le rechargement de la page.
De plus  il n'y a pas que l'upload mais la redimension et la vignette qui reste tres tres utile... de bonnes classes de resize et de crop ne sont pas légions et je sais de quoi je parle lol.
Enfin cela initie à un framework javascript interessant qu'est mootools que j'aime beaucoup ^^

Pour la taille des fichiers encore une fois les developpeurs peuvent la specifier eux meme et s'arranger comme ils le souhaitent, si on veut pousser plus loin il faut sécuriser encore plus mais c'est vrai j'essairai d'arranger ca egalement :)
A noter que si le fichier n'est pas une image valide il n'y a pas d'upload.

Donc biensur c'est ameliorable et je tiens compte des remarques merci beaucoup mais si vous pouvez le tester ca peut etre assez pratique egalement pour m'indiquer ce qui va ou ne pas :)

Cordialement,

Laurent.

Commentaire de vega11 le 08/02/2008 19:27:31

Ok ca doit etre une histoire de transparent oui dans la css je regarderais merci !

Pour plusieurs photos je comptais peu etre gerer des fichiers zippés ca peut etre sympa ya juste à dezipper et boucler au final et uploader que les fichiers qui sont des images valides..

je regarderais merci pour les critiques ;)

Commentaire de micropoint le 17/02/2008 00:20:21

Bonjour,
je trouve ton script très bien mais car il y a toujours un mais, corriger certaine chose  :
1 - il y a une petite erreur dans upload.php ligne 42 :
    $req="INSERT INTO medias(titre_media,url,type) VALUES('titre_img','$url','IMAGE')"; : Il manque un espace entre media et (titre...)
2 - Tu devrais centralisé les variables, tant sur le chemin d'accès à la GreyBox que pour la connexion à la base de donné (un fichier ini.php par exemple)
3 - Dans ce même fichier, nous devrions pouvoir choisir la taille max de l'image importée et du thumb généré.

Sinon, comme je te le disait plus haut, c'est un trés bon script.