PHP OO Confirmación registro por mail

[OfSerker]

Hola forosdelwebmaniacos. Necesito vuestra ayuda, quiero hacer que cuando un usuario se registre, automaticamente le envie un correo con una confirmación del registro.

De momento, tengo este código de mail.php:

Código PHP:

  <?php
require_once('./config/funciones.php');
conectar('localhost', 'root', '123456', 'ofserker');
 
$usuario = $_POST['usernamesignup']; 
$correo = $_POST['emailsignup']; 
//... 
 
//<-- Tus rutinas para validar los datos, si están completos etc... 
 
if(empty($usuario)){ 
    echo "Debes poner algo como usuario"; 
    exit; 
} 
 
//--> 
 
$aleatorio = uniqid(); //Genera un id único para identificar la cuenta a traves del correo. 
$contrasena = rand(1999, 9999); //Devuelve un número aleatorio entre los dos rangos. Lo usuaremos como 
                                //Contraseña temporal. 
                                 
$sql = "Insert Into users (user, pass, mail, code, active) Values ('$usuario', '$contrasena', '$correo', '$aleatorio', 0)"; 
 
//Tus rutinas para insertar en la base de datos. 
 
$mensaje = "Registro en tuweb.com\n\n"; 
$mensaje .= "Estos son tus datos de registro:\n"; 
$mensaje .= "Usuario: $usuario.\n"; 
$mensaje .= "Contraseña: $contrasena.\n\n"; 
$mensaje .= "Debes activar tu cuenta pulsando este enlace: http://www.ofserker.netai.net/activacion.php?id=$aleatorio"; 
 
$asunto = "Activación de tu cuenta en ofserker.netai.net"; 
 
if(mail($correo,$asunto,$mensaje)){ 
   
echo " Se ha enviado un mensaje a tu correo electronico con el código de activación"; 
}else{ 
    echo "Ha ocurrido un error y no se puede enviar el correo"; 
} 
 
?>   

Lo malo, es que en el localhost, cuando registro a alguien, no sale un código al azar. Yo creo que es culpa de la db. Bueno no me enrollo más y aqui va la foto:



Espero atentamente vuestras respuestas:
-OfSerker

[Triby]

Qué te hace pensar que "es culpa de la db"?

De entrada, estás armando la consulta, pero no veo que la ejecutes.

[loncho_rojas]

Y se supone que deberías verificarlo XD Pero no estas ejecutando la consulta $sql
No se que usas si MYSQL o PDO o no se, pero te faltaría

Código PHP:

mysql_query($sql)
 

si el usuario va a presionar ese link debería existir el archivo activacion.php

Y como esta enviando una variable llamada id deberias recibirla en dicha página...

ya despues te toca un trabajo de verificacion por ejemplo:

Código PHP:

  <?php
$key = $_GET['id'];

$query=mysql_query("SELECT * FROM users WHERE id='$key'");
if($check=mysql_fetch_array($query)) //con este if verificamos que el codigo existe en la BD 
            { 
            
                if($check['active']=="1") //Si el mail ya esta verificado 
                { 
                    echo "Tu mail ya esta verificado";
                } 
                elseif($check['active']=="0") //Si no esta verificado 
                { 
                //Actualizamos la BD, y cambiamos el valor de dafault "NO" en verificado por "SI" 
                    $sql = "UPDATE users SET active='1' WHERE id='$key'"; 
                    mysql_query($sql) or die("Error query ".mysql_error());  
                     echo "<p>Gracias. Hemos verificado su email. Ahora puede acceder con su usuario y clave</p>";
                     echo "<a href='../index.php'>Loguearse</a>";
                     //habilitamos el formulario de login luego de la bienvenida
                    
                
                } 
            } 
            else //Si el codigo de verificacion no existe 
            { 
            echo "<p>Tu c&oacute;digo es incorrecto</p>"; 
            echo "<a href='../index.php'>Loguearse</a>";
            } 
        ?>   

[OfSerker]

Hola, gracias por responder. Cuando pueda, probaré el codigo y te dire algo.

Yo estube mirando hilos y dicen algunos que se necesita el php.ini para enviar emails. Yo uso el servidor 000webhost. No se si eso afecta en algo.

Atentamente:
-OfSerker

[TSG]

Hola muy buenas,

No necesitas php.ini.

Solo necesitas este código:

Código PHP:

$para = $email; //hace referencia a la variable email...
$titulo = "Activar cuenta";
$mensaje = "El mensaje es este chaval. ejemplo xd";
$cabeceras  = 'MIME-Version: 1.0' . "\r\n";
$cabeceras .= 'Content-type: text/plain; charset=iso-8859-1' . "\r\n";
$cabeceras .= 'From: Activar cuenta <tucorreo>' . "\r\n";
 
mail($para, $titulo, $mensaje, $cabeceras);
 

Esto lo introduces en un registro, cuándo te registres la web, en dónde te lanza el mensaje de te has registrado correctamente introduces esto...

Claro ya lo que sería para verificación de un código has de sacar los datos de la base de datos y enviarlo nada complicado.

Ya para activar tu cuenta, necesitas manejar las entradas al servidor, osea los $_GET...

Pero si sabes del tema....

Bueno, prueba y tal y coméntanos.

Un cordial saludo!

[OfSerker]

Vale, esto es como un mensaje sin confirmacion. Muy simple. Lo incluyo cuando me registre correctamente y enviara el mensaje. ¿Lo he entendido bien?

Atentamente, OfSerker.

P.D: Empiezan Los Simpsons.

[TSG]

Iniciado por OfSerker

Vale, esto es como un mensaje sin confirmacion. Muy simple. Lo incluyo cuando me registre correctamente y enviara el mensaje. ¿Lo he entendido bien?

Atentamente, OfSerker.

P.D: Empiezan Los Simpsons.

Perdón eso fue culpa mía no terminé le leer bien y tal y puse eso de enviar el correo.
----------------------------------------------------------------------------------------------------------

Hola muy buenas,

No necesitas php.ini.

Código PHP:

$email = utf8_decode(mysqli_real_escape_string($conexion, ((isset($_GET['email']) && !empty($_GET['email']))?$_GET['email']:false)));
$key = utf8_decode(mysqli_real_escape_string($conexion, ((isset($_GET['key']) && !empty($_GET['key']))?$_GET['key']:false)));
 
$ffkk = mysqli_query($conexion, "SELECT tucolumna FROM tabla WHERE email = '".$email."'")or die(mysqli_query($conexion));
$del = mysqli_fetch_assoc($ffkk);
if($del['activacion'] == '1'){
    echo "No puedes volver a activar tu cuenta, ya se encuentra activada.";
}
else{
 
$verificacion = mysqli_query($conexion, "SELECT * FROM tutabla WHERE email = '".$email."' AND code = '".$key."'")or die(mysqli_error($conexion));
if($veri = mysqli_fetch_array($verificacion)){
    $codes = md5 (rand (0,90900));
    $ya = mysqli_query($conexion, "UPDATE tutabla SET activacion = '1' WHERE activacion = '0'")or die(mysqli_error($conexion));
    $ya2 = mysqli_query($conexion, "UPDATE tutabla SET code = '".$codes."' WHERE email = '".$email."'")or die(mysqli_error($conexion));
    echo  "Tu cuenta ha sido activada.";
 
}
 

Ya solo con esto adaptándolo a tu código debería de funcionarte perfectamente.

Un cordial saludo!

[loncho_rojas]

como vas a hacer eso:

"UPDATE tutabla SET activacion = '1' WHERE activacion = '0'"

o sea, vas a tomar todas las cuentas no activadas y las vas a activar?

[TSG]

Bueno ya el se encargaria de solucionar eso nada del otro mundo hace:

Where email = '"$email"'

Igual con el key... Solo deberia adaptarlo bien

saludos!

[OfSerker]

Bueno, no os habéis explicado muy bien... Soy nuevo en esto, solo se seleccionar cosas en la bd y el metodo $_POST.

Aqui os paso los archivos que me dijisteis que escribiera:

activacion.php:

Código PHP:

  <?php
$email = utf8_decode(mysqli_real_escape_string($conexion, ((isset($_GET['email']) && !empty($_GET['email']))?$_GET['email']:false)));
$key = utf8_decode(mysqli_real_escape_string($conexion, ((isset($_GET['key']) && !empty($_GET['key']))?$_GET['key']:false)));
 
$ffkk = mysqli_query($conexion, "SELECT tucolumna FROM tabla WHERE email = '".$email."'")or die(mysqli_query($conexion));
$del = mysqli_fetch_assoc($ffkk);
if($del['activacion'] == '1'){
    echo "No puedes volver a activar tu cuenta, ya se encuentra activada.";
}
else{
 
$verificacion = mysqli_query($conexion, "SELECT * FROM tutabla WHERE email = '".$email."' AND code = '".$key."'")or die(mysqli_error($conexion));
if($veri = mysqli_fetch_array($verificacion)){
    $codes = md5 (rand (0,90900));
    $ya = mysqli_query($conexion, "UPDATE tutabla SET activacion = '1' WHERE activacion = '0'")or die(mysqli_error($conexion));
    $ya2 = mysqli_query($conexion, "UPDATE tutabla SET code = '".$codes."' WHERE email = '".$email."'")or die(mysqli_error($conexion));
    echo  "Tu cuenta ha sido activada.";
 
}
?>   

activacion.php:

Código PHP:

  <?php
$key = $_GET['id'];
 
$query=mysql_query("SELECT * FROM users WHERE id='$key'");
if($check=mysql_fetch_array($query)) //con este if verificamos que el codigo existe en la BD 
            { 
            
                if($check['active']=="1") //Si el mail ya esta verificado 
                { 
                    echo "Tu mail ya esta verificado";
                } 
                elseif($check['active']=="0") //Si no esta verificado 
                { 
                //Actualizamos la BD, y cambiamos el valor de dafault "NO" en verificado por "SI" 
                    $sql = "UPDATE users SET active='1' WHERE id='$key'"; 
                    mysql_query($sql) or die("Error query ".mysql_error());  
                     echo "<p>Gracias. Hemos verificado su email. Ahora puede acceder con su usuario y clave</p>";
                     echo "<a href='../index.php'>Loguearse</a>";
                     //habilitamos el formulario de login luego de la bienvenida
                    
                
                } 
            } 
            else //Si el codigo de verificacion no existe 
            { 
            echo "<p>Tu c&oacute;digo es incorrecto</p>"; 
            echo "<a href='../index.php'>Loguearse</a>";
            } 
        ?>   

mail.php:

Código PHP:

  <?php
require_once('./config/funciones.php');
conectar('localhost', 'root', '123456', 'ofserker');
 
$para = $email; //hace referencia a la variable email...
$titulo = "Activar cuenta";
$mensaje = "El mensaje es este chaval. ejemplo xd";
$cabeceras  = 'MIME-Version: 1.0' . "\r\n";
$cabeceras .= 'Content-type: text/plain; charset=iso-8859-1' . "\r\n";
$cabeceras .= 'From: Activar cuenta <tucorreo>' . "\r\n";
 
mail($para, $titulo, $mensaje, $cabeceras);
 
?>   

Como hago para que los archivos trabajen juntos (los junte), que pongo en la db y si hay algún error porfavor, decidmelo.

[supersiana]

En cuanto a lo que se comentó:

Iniciado por loncho_rojas

como vas a hacer eso:

"UPDATE tutabla SET activacion = '1' WHERE activacion = '0'"

o sea, vas a tomar todas las cuentas no activadas y las vas a activar?

Cierto, esa sentencia es un error de sintaxis. Funcionará a la perfección pero sólo hará que todos los campos "activacion" de la tabla queden activos cuando sólo debiera serlo el del email en cuestión.

Es un error que se termina aprendiendo con la práctica en sintaxis SQL.

MI PREGUNTA SOBRE EL TEMA ES:

Es tuve viendo que la función uniqid() que php.net considera obsoleta y en su lugar, sugiere utilizar la función: openssl_random_pseudo_bytes()

¿Alguno la ha llegado a poner en práctica en sus códigos?
¿Algún ejemplo en español?

Muchas gracias!