0

Intento hacer al login con el password_verify, pero el error Undefined Index: contrasena no deja de aparecer. Este es el código. Por cierto, sí registro al usuario con la contraseña encriptada (password_hash).

    if(isset($_POST['login_btn']))
    {   //Valida valores del formulario
        //$errores = validarUsuario($_POST, ['login_btn']);
        $correo     = $_POST['correo'];
        $contrasena = $_POST['password'];


            $sql = "SELECT * FROM usuarios1 WHERE nombre=? OR correo=? LIMIT 1";
            $usuario = getRegistroSimple($sql, 'ss', [$correo, $correo]);

            if (!empty($usuario)) //Si se encontró el correo(usuario)
            {   
                if (password_verify($contrasena, $usuario['contrasena'])) //Esta línea es la que mara undefined index
                {
                    loginPorId($usuario['id']);
                }
                else //Si no coincide 
                {
                    $_SESSION['error_msg'] = "Correo o contraseña incorrecto";
                }       
            }
            else 
            {//Si el usuario no se encuentra
                $_SESSION['error_msg'] = "Usuario o contraseña incorrecto";
            }   
        }
 ?>

Aquí la función:

function getRegistroSimple($sql, $types, $params)
{
    global $conexion;
    $stmt = $conexion->prepare($sql);
    $stmt->bind_param($types, ...$params);
    $stmt->execute();
    $result = $stmt->get_result();
    $usuario = $result->fetch_all(MYSQLI_ASSOC);
    $stmt->close();
    return $usuario;

}
11
  • Validaste que $usuario tiene un valor valido para password? Commented el 15 abr. 2019 a las 14:11
  • 3
    Dice claramente que en $usuario no hay un índice llamado contrasena. Depura tu objeto mediante var_dump($usuario); así verás cómo está constituido y cómo tienes que usarlo.
    – A. Cedano
    Commented el 15 abr. 2019 a las 14:13
  • ["contrasena"]=> string(60) "$2y$10$cCSDRH0vDDNWCy0Y1BElTOtiSon6YTG2hybFOTGV0HGp7Kht09OBq" Me muestra todos los valores de la consulta, y el campo que quiero verificar, al parecer debería estar correcto, ¿no?
    – Enrique
    Commented el 15 abr. 2019 a las 14:18
  • Pon la funcion getRegistroSimplepara que podamos ver como se recupera.
    – JDev
    Commented el 15 abr. 2019 a las 14:23
  • 1
    Es lo que te decia, al hacer un fetch_all, obtienes un array de resultados. Usa fetch_array para obtener solo un registro. php.net/manual/es/mysqli-result.fetch-array.php
    – JDev
    Commented el 15 abr. 2019 a las 14:49

1 respuesta 1

2

El problema es que al hacer un fetch_all obtienes un array de registros.

Para obtener un unico resultado usa fetch_array

En tu caso:

$usuario = $result->fetch_array(MYSQLI_ASSOC);

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.