Vérification de l'email bugguée

Lunastyx

Plop,

J'ai essayé d'ajouter la vérification de l'email (càd sa structure *@*.*) sauf que je me suis rendu compte qu'elle existait déjà et qu'elle ne fonctionnait donc pas puisque je peux poster des commentaires avec un champ mail vide.

Aux alentours de la ligne 755 du fichier pluxml/core/lib/class.plx.motor.php

Où cela peut il se corriger ?

Lunastyx.

Stéphane

Bonjour

Heu c'est quoi le bug ?

Dans la mesure ou l'adresse email est facultative dans les commentaires, c'est normal que tu puisses commenter avec une adresse vide.

Lunastyx

Oh, je croyais qu'elle était nécessaire, au temps pour moi !
En revanche je comprends donc pas pourquoi il y a cette ligne de vérification x,x

Stéphane

Tu peux donner le n° de la ligne exacte à laquelle tu fais référence stp.

Lunastyx

740:
# On verifie le mail
$comment = (plxUtils::checkMail(trim($content)))?trim($content):'';

Stéphane

Que dire de plus:

Si l'adresse email saisie nettoyée des espaces avant et après l'adresse est valide, on enregistre l'adresse nettoyée, sinon on force à blanc.

avec 1 seul test on traite le cas d'une adresse saisie ou d'une adresse non saisie, sinon on aurait un code du genre

si adresse saisie alors
___ si adresse valide alors
_________ on enregistre l'adresse
____sinon
_________ on force à blanc
____fsi
sinon
____ on force à blanc
fin si

Lunastyx

Je ne connaissais pas la commande trim :<
Pebkac... /me autoslap.

Merci

Stéphane

trim permet de supprimer les éventuels espaces saisis avant ou après une chaîne de caractère