Vérification de l'email bugguée

LunastyxLunastyx Member
dans Bogues
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.

Réponses

  • StéphaneStéphane Member, Former PluXml Project Manager
    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.

    Consultant PluXml

    Ancien responsable et développeur de PluXml (2010 à 2018)

  • LunastyxLunastyx Member
    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éphaneStéphane Member, Former PluXml Project Manager
    Tu peux donner le n° de la ligne exacte à laquelle tu fais référence stp.

    Consultant PluXml

    Ancien responsable et développeur de PluXml (2010 à 2018)

  • LunastyxLunastyx Member
    740:
    # On verifie le mail
    $comment = (plxUtils::checkMail(trim($content)))?trim($content):'';
  • StéphaneStéphane Member, Former PluXml Project Manager
    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

    Consultant PluXml

    Ancien responsable et développeur de PluXml (2010 à 2018)

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

    Merci :)
  • StéphaneStéphane Member, Former PluXml Project Manager
    trim permet de supprimer les éventuels espaces saisis avant ou après une chaîne de caractère

    Consultant PluXml

    Ancien responsable et développeur de PluXml (2010 à 2018)

Connectez-vous ou Inscrivez-vous pour répondre.