Skip to main content
se añadieron 14 caracteres en el cuerpo
Origen Enlace
aeportugal
  • 10.6k
  • 2
  • 12
  • 24
<?php
// ...
// Lo anterior a esto se queda igual
// Recibimos metodo POST cuando esta enviado
if($_SERVER["REQUEST_METHOD"] == "POST") {
    $username = trim($_POST["username"]);
    $filtered_username = filter_var($username, FILTER_VALIDATE_REGEXP, 
                        array("options"=>array("regexp"=>"/^[a-zA-Z\s]+$/")));
    $email = trim($_POST["email"]);
    $filtered_email = filter_var($email, FILTER_VALIDATE_EMAIL, FILTER_NULL_ON_FAILURE);
    $password = $_POST["password"];
    $confirm_password = $_POST["confirm_password"];

    if(empty($username)) {
        $username_err = "Please enter a name.";
    } elseif (!$filtered_username) {
        $username_err = "Please enter a valid name.";
    } elseif(!$filtered_email) {
        $email_err = "Por favor introduzca un correo valido.";
    } elseif(empty($password)) {
        $password_err = "Please enter a password.";
    } elseif(strlen($password) < 6) {
        $password_err = "Contraseña tiene que tener al menos 6 caracteres.";
    } elseif(empty($confirm_password)) {
        $confirm_password_err = "Por favor, confirme la contraseña.";
    } elseif($password !== $confirm_password) {
        $confirm_password_err = "Contraseñas no conciden.";
    } else {
        //Codigo de confirmacion de email
        $email_verification = rand(100000, 999999);
        $sql = "SELECT id_user FROM users WHERE email_user = :email";
        $stmt = $pdo->prepare($sql);
        $stmt->bindParam(":email", $filtered_email, PDO::PARAM_STR);
        $stmt->execute();
        if($stmt->rowCount() > 0) {
            $email_err = "Este correo ya esta registrado.";
        } else {
            echo "Estamos aqui";
    $stmt->closeCursor(); // Limpiar el stmt de la consulta $stmt->closeCursor();anterior
            $sql = "INSERT INTO users (username, email_user, password, email_verification) 
            VALUES (:username, :email, :password, :email_verification)";
            $stmt = $pdo->prepare($sql);
            $stmt->bindParam(":username", $param_username, PDO::PARAM_STR);
            $stmt->bindParam(":email", $param_email, PDO::PARAM_STR);
            $stmt->bindParam(":password", $param_password, PDO::PARAM_STR);
            $stmt->bindParam(":email_verification", $param_email_verification, PDO::PARAM_STR);
            // Set parameters
            $param_username = $username;
            $param_email = $email;
            $param_email_verification = $email_verification;
            $param_password = password_hash($password, PASSWORD_DEFAULT); // Creates a password hash
            if ($stmt->execute()) {
                // Redirecion despues de registrar
                header("location: login.php");
            } else {
                echo "Algo no va bien, intentelo de nuevo o contacte con nosotros.";
            }
        }
        unset($stmt);
        unset($pdo);
    }
}
?>

<!-- Resto HTML -->
<?php
// ...
// Lo anterior a esto se queda igual
// Recibimos metodo POST cuando esta enviado
if($_SERVER["REQUEST_METHOD"] == "POST") {
    $username = trim($_POST["username"]);
    $filtered_username = filter_var($username, FILTER_VALIDATE_REGEXP, 
                        array("options"=>array("regexp"=>"/^[a-zA-Z\s]+$/")));
    $email = trim($_POST["email"]);
    $filtered_email = filter_var($email, FILTER_VALIDATE_EMAIL, FILTER_NULL_ON_FAILURE);
    $password = $_POST["password"];
    $confirm_password = $_POST["confirm_password"];

    if(empty($username)) {
        $username_err = "Please enter a name.";
    } elseif (!$filtered_username) {
        $username_err = "Please enter a valid name.";
    } elseif(!$filtered_email) {
        $email_err = "Por favor introduzca un correo valido.";
    } elseif(empty($password)) {
        $password_err = "Please enter a password.";
    } elseif(strlen($password) < 6) {
        $password_err = "Contraseña tiene que tener al menos 6 caracteres.";
    } elseif(empty($confirm_password)) {
        $confirm_password_err = "Por favor, confirme la contraseña.";
    } elseif($password !== $confirm_password) {
        $confirm_password_err = "Contraseñas no conciden.";
    } else {
        //Codigo de confirmacion de email
        $email_verification = rand(100000, 999999);
        $sql = "SELECT id_user FROM users WHERE email_user = :email";
        $stmt = $pdo->prepare($sql);
        $stmt->bindParam(":email", $filtered_email, PDO::PARAM_STR);
        $stmt->execute();
        if($stmt->rowCount() > 0) {
            $email_err = "Este correo ya esta registrado.";
        } else {
            echo "Estamos aqui";
            $stmt->closeCursor();
            $sql = "INSERT INTO users (username, email_user, password, email_verification) 
            VALUES (:username, :email, :password, :email_verification)";
            $stmt = $pdo->prepare($sql);
            $stmt->bindParam(":username", $param_username, PDO::PARAM_STR);
            $stmt->bindParam(":email", $param_email, PDO::PARAM_STR);
            $stmt->bindParam(":password", $param_password, PDO::PARAM_STR);
            $stmt->bindParam(":email_verification", $param_email_verification, PDO::PARAM_STR);
            // Set parameters
            $param_username = $username;
            $param_email = $email;
            $param_email_verification = $email_verification;
            $param_password = password_hash($password, PASSWORD_DEFAULT); // Creates a password hash
            if ($stmt->execute()) {
                // Redirecion despues de registrar
                header("location: login.php");
            } else {
                echo "Algo no va bien, intentelo de nuevo o contacte con nosotros.";
            }
        }
        unset($stmt);
        unset($pdo);
    }
}
?>

<!-- Resto HTML -->
<?php
// ...
// Lo anterior a esto se queda igual
// Recibimos metodo POST cuando esta enviado
if($_SERVER["REQUEST_METHOD"] == "POST") {
    $username = trim($_POST["username"]);
    $filtered_username = filter_var($username, FILTER_VALIDATE_REGEXP, 
                        array("options"=>array("regexp"=>"/^[a-zA-Z\s]+$/")));
    $email = trim($_POST["email"]);
    $filtered_email = filter_var($email, FILTER_VALIDATE_EMAIL, FILTER_NULL_ON_FAILURE);
    $password = $_POST["password"];
    $confirm_password = $_POST["confirm_password"];

    if(empty($username)) {
        $username_err = "Please enter a name.";
    } elseif (!$filtered_username) {
        $username_err = "Please enter a valid name.";
    } elseif(!$filtered_email) {
        $email_err = "Por favor introduzca un correo valido.";
    } elseif(empty($password)) {
        $password_err = "Please enter a password.";
    } elseif(strlen($password) < 6) {
        $password_err = "Contraseña tiene que tener al menos 6 caracteres.";
    } elseif(empty($confirm_password)) {
        $confirm_password_err = "Por favor, confirme la contraseña.";
    } elseif($password !== $confirm_password) {
        $confirm_password_err = "Contraseñas no conciden.";
    } else {
        //Codigo de confirmacion de email
        $email_verification = rand(100000, 999999);
        $sql = "SELECT id_user FROM users WHERE email_user = :email";
        $stmt = $pdo->prepare($sql);
        $stmt->bindParam(":email", $filtered_email, PDO::PARAM_STR);
        $stmt->execute();
        if($stmt->rowCount() > 0) {
            $email_err = "Este correo ya esta registrado.";
        } else {
            $stmt->closeCursor(); // Limpiar el stmt de la consulta anterior
            $sql = "INSERT INTO users (username, email_user, password, email_verification) 
            VALUES (:username, :email, :password, :email_verification)";
            $stmt = $pdo->prepare($sql);
            $stmt->bindParam(":username", $param_username, PDO::PARAM_STR);
            $stmt->bindParam(":email", $param_email, PDO::PARAM_STR);
            $stmt->bindParam(":password", $param_password, PDO::PARAM_STR);
            $stmt->bindParam(":email_verification", $param_email_verification, PDO::PARAM_STR);
            // Set parameters
            $param_username = $username;
            $param_email = $email;
            $param_email_verification = $email_verification;
            $param_password = password_hash($password, PASSWORD_DEFAULT); // Creates a password hash
            if ($stmt->execute()) {
                // Redirecion despues de registrar
                header("location: login.php");
            } else {
                echo "Algo no va bien, intentelo de nuevo o contacte con nosotros.";
            }
        }
        unset($stmt);
        unset($pdo);
    }
}
?>

<!-- Resto HTML -->
Origen Enlace
aeportugal
  • 10.6k
  • 2
  • 12
  • 24

El problema es que $param_email no existe en esta linea

if ($stmt = $pdo->prepare($sql)) {
    // Vinculamos parametros
    $stmt->bindParam(":email", $param_email, PDO::PARAM_STR);

Se debe recoger $param_email antes con $param_email = $_POST['email']; para que entonces pueda ser utilizada.

Lo siguiente puede ser una opinión, pero veo el código un poco enredado. Podría mejorarse mucho usando funciones o una class. Pero si no quieres usar ninguna, una forma más organizada del mismo, en la parte del if principal, sería:

<?php
// ...
// Lo anterior a esto se queda igual
// Recibimos metodo POST cuando esta enviado
if($_SERVER["REQUEST_METHOD"] == "POST") {
    $username = trim($_POST["username"]);
    $filtered_username = filter_var($username, FILTER_VALIDATE_REGEXP, 
                        array("options"=>array("regexp"=>"/^[a-zA-Z\s]+$/")));
    $email = trim($_POST["email"]);
    $filtered_email = filter_var($email, FILTER_VALIDATE_EMAIL, FILTER_NULL_ON_FAILURE);
    $password = $_POST["password"];
    $confirm_password = $_POST["confirm_password"];

    if(empty($username)) {
        $username_err = "Please enter a name.";
    } elseif (!$filtered_username) {
        $username_err = "Please enter a valid name.";
    } elseif(!$filtered_email) {
        $email_err = "Por favor introduzca un correo valido.";
    } elseif(empty($password)) {
        $password_err = "Please enter a password.";
    } elseif(strlen($password) < 6) {
        $password_err = "Contraseña tiene que tener al menos 6 caracteres.";
    } elseif(empty($confirm_password)) {
        $confirm_password_err = "Por favor, confirme la contraseña.";
    } elseif($password !== $confirm_password) {
        $confirm_password_err = "Contraseñas no conciden.";
    } else {
        //Codigo de confirmacion de email
        $email_verification = rand(100000, 999999);
        $sql = "SELECT id_user FROM users WHERE email_user = :email";
        $stmt = $pdo->prepare($sql);
        $stmt->bindParam(":email", $filtered_email, PDO::PARAM_STR);
        $stmt->execute();
        if($stmt->rowCount() > 0) {
            $email_err = "Este correo ya esta registrado.";
        } else {
            echo "Estamos aqui";
            $stmt->closeCursor();
            $sql = "INSERT INTO users (username, email_user, password, email_verification) 
            VALUES (:username, :email, :password, :email_verification)";
            $stmt = $pdo->prepare($sql);
            $stmt->bindParam(":username", $param_username, PDO::PARAM_STR);
            $stmt->bindParam(":email", $param_email, PDO::PARAM_STR);
            $stmt->bindParam(":password", $param_password, PDO::PARAM_STR);
            $stmt->bindParam(":email_verification", $param_email_verification, PDO::PARAM_STR);
            // Set parameters
            $param_username = $username;
            $param_email = $email;
            $param_email_verification = $email_verification;
            $param_password = password_hash($password, PASSWORD_DEFAULT); // Creates a password hash
            if ($stmt->execute()) {
                // Redirecion despues de registrar
                header("location: login.php");
            } else {
                echo "Algo no va bien, intentelo de nuevo o contacte con nosotros.";
            }
        }
        unset($stmt);
        unset($pdo);
    }
}
?>

<!-- Resto HTML -->

Otras cosas a considerar es que, a pesar de que parecería lo correcto, NO le debes hacer trim() ni al username ni al password. Si el usuario explicitamente colocó esos caracteres y tu sistema los logra registrar (porque los limpió con trim()), entonces le estás diciendo al usuario que su información tiene espacios en blanco, pero dentro de la BD ya no los tiene, por lo que no podrá loguearse con la información que el creía que habia escrito. Si no quieres esos caracteres, lo mejor sería indicarle al usuario que esos caracteres no son válidos y que los quite del input.