FORMMAIL TRÈS SIMPLE ET PERSONALISABLE

klicksoe Messages postés 38 Date d'inscription vendredi 9 mars 2007 Statut Membre Dernière intervention 23 avril 2009 - 6 juin 2008 à 11:37
nicomilville Messages postés 3472 Date d'inscription lundi 16 juillet 2007 Statut Membre Dernière intervention 28 février 2014 - 18 juin 2008 à 10:13
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/46898-formmail-tres-simple-et-personalisable

nicomilville Messages postés 3472 Date d'inscription lundi 16 juillet 2007 Statut Membre Dernière intervention 28 février 2014 36
18 juin 2008 à 10:13
Effectivement, j'arrive un peut tard et comme l'a si bien dit coucou747 :

- $HTTP_POST_VAR c'est très vieux, maintenant on utilise plutot : $_POST['nom']...

Et comme l'a dit aussi Morphinof :

- il n'y a aucune vérification, tu pourrai au moins vérifier si les variables existent et si elle ne sont pas vide : de cette façon :

- pour vérifier l'existance des variables : if(isset($_POST['nom_du_champs']) && isset($_POST...) {...
- pour vérifier si les variables ne sont pas vides : if(!empty($_POST['nom_du_champs']) && !empty($_POST...)

a++

PS : je préfère ne pas mettre de note !
bouztarak25 Messages postés 5 Date d'inscription lundi 25 juin 2007 Statut Membre Dernière intervention 16 juin 2008
16 juin 2008 à 17:46
En effet, je ne suis pas très intéressé par 'FORMMAIL'!
Pourriez vous peut être un peu modifier et améliorer (test mail si il est bien parti etc..)?!

Bon courage !
webdeb Messages postés 488 Date d'inscription samedi 5 avril 2003 Statut Membre Dernière intervention 31 mars 2009 4
6 juin 2008 à 19:07
Quelle horreur ce code....
Morphinof Messages postés 255 Date d'inscription vendredi 20 avril 2007 Statut Membre Dernière intervention 9 août 2013 4
6 juin 2008 à 13:28
En dehors de ce qui disais très justement klicksoe au niveau du html, tu ne fais aucune vérification sur tes champs de formulaire !
Déjà au niveau sécurité c'est à revoir mais le minimum serait de vérifier le format de l'e-mail.
Quand tu utilises la fonction mail il faut tester son retour :
if(mail(...)) pour afficher un message d'erreur si jamais le mail passe pas etc.
coucou747 Messages postés 12303 Date d'inscription mardi 10 février 2004 Statut Membre Dernière intervention 30 juillet 2012 44
6 juin 2008 à 12:15
salut
$HTTP_POST_VARS...
c'est du vieux php3... on en est a php6 actuellement
ton script n'a rien de personalisable, il n'est absolument pas generique, n'utilise aucune fonction, aucun template, etc...
klicksoe Messages postés 38 Date d'inscription vendredi 9 mars 2007 Statut Membre Dernière intervention 23 avril 2009
6 juin 2008 à 11:37
Salut,

quelques petites "erreurs" de HTML et de PHP...

En voilà une : (index.php)

#
-
#

ça devrait être :

#
-
#

Respect des normes W3C... Je te conseille le module HTML validator pour Firifox, il te permettra de faire ce genre de vérification.

Ensuite pour le PHP (formulaire.php)

Etant donné qu'il n'y en a pas beaucoup, je ne pas très bien voir si il y a des choses que u devrais approfondir, mais je te conseillerai de mettre des simple côte au lieu des guillemets.

//Klick.
Rejoignez-nous