Como bien indica el Manual de PHP el correo sería enviando a la dirección que pongas en el primer parámetro de la función mail()
, que en tu caso sería $email
.
Lo que pareces tener es una confusión de variables. Es simple: puedes llamar tus variables como quieras, a condición de que luego uses lo que has definido, no otra cosa. También, por claridad de código, se recomienda que tus variables hagan referencia a lo significa, de modo que al verlas tú o cualquier otro programador (pensemos en un proyecto grande donde participen varias personas) se sepa a qué uso están destinadas esas variables.
Otra práctica que yo recomiendo es asignar los datos de las superglobales mediante el operador de fusión de null (??) si tienes PHP 7+, o mediante operadores ternarios si tienes PHP anterior a la versión 7 y luego usar las variables asignadas en el resto del código, olvidando en lo adelante las superglobales. Así el código queda más claro, más limpio y menos propenso a errores.
Aplicando lo dicho, tu código quedaría así:
if (isset($_POST['enviar'])) {
/*
Asumiendo que tienes PHP 7+
usaremos el operador de fusión de null (??)
Si no es el caso debes hacer las asignaciones con este estilo:
$name = !empty($_POST['name']) ? $_POST['name'] : NULL;
*/
$name = $_POST['name'] ?? NULL;
$asunto= $_POST['asunto'] ?? NULL;
$msg = $_POST['msg'] ?? NULL;
$email = $_POST['email'] ?? NULL;
if ($name && $asunto && $msg && $email) {
$header="From: [email protected]". "\r\n";
$header.= "Reply-To: [email protected]". "\r\n";
$header. "X-Mailer: PHP/". phpversion();
/*
$status sería un nombre más descriptivo
y que da lugar a menos confusión
*/
$status= mail($email, $asunto, $msg, $header);
if ($status) {
echo "<h4>¡Mail enviado exitosamente!</h4>";
}
}
POST-DATA:
Aquí no queda claro para qué recoges $name
. Si no lo vas a usar, no lo recojas ni lo verifiques en el if
.
mail
está mal escrita, no es ail, además querecibe el subject como segundo parámetro. Acá lo puedes ver php.net/manual/en/function.mail.php$email
, que recogiste del POST aquí:$email = $_POST['email'];
$to
al resultado de la llamada a la funciónmail()
, entonces, en la verificación delif
debes usar$to
, en cambio, si la llamas$mail
, en elif
debes usar$mail
. Las variables se pueden llamar como quieras, pero usa los nombres que les hayas dado, procurando, por claridad del código, que al leerlas ya sepas de qué se trata.