Skip to main content
se añadieron 125 caracteres en el cuerpo
Origen Enlace
Alberto Siurob
  • 6.5k
  • 7
  • 33
  • 72

Tienes un error en la lógica, el sistema está haciendo lo que le estás pidiendo.

  • Estas intentando settear una variable de sesión $_SESSION['intentos'] = 0 antes del session_start()
  • Cuando se vuelve a invocar tu script, el primer comando es settear $_SESSION['intentos'] = 0, esto hará que los intentos sean infinitos
  • Siempre antes que un echo, van las instrucciones que debes hacer, lo estás haciendo mal en el paso 3, estás invocando un script que redirige a otra página, eso puede ocasionar que el PHP no termine de hacer su trabajo.
  • En ningún caso estás invocando la ejecución de tus consultas

Te propongo esta solución antes del try

session_start();
if( array_key_exists( 'intentos', $_SESSION ) ){
    $_SESSION['intentos'] = $_SESSION['intentos'] + 1;
} else {
    $_SESSION['intentos'] = 0;
}
$sql = "UPDATE usuario SET intentos = $_SESSION['intentos'] WHERE Usuario = '$_POST['login']'";
$base->exec($sql);

Esto validará si hay ya una variable de sesión con el índice intentos, si existe, le suma 1, si no existe, la crea en 0

if( $_SESSION['intentos'] >= 3 ){
$sql = "UPDATE usuario SET Estado = 'Inactivo' WHERE Usuario = '$login'";
$base->exec($sql);
      echo "<script>swal({
        title: 'Cuenta bloqueada',
        text: 'Ingrese de nuevo sus datos',
        type: 'error',
        confirmButtonText: 'Aceptar',
        allowOutsideClick: false,
        confirmButtonColor: '#E12A2A'})  
        .then((result) => { 
        window.location=window.location='index.php'});</script>";
}

Cambié la condición que sea == 3 a que sea >= 3, si la persona por X razón vuelve a enviar la información, lo dejará pasar, puesto que $_SESSION['intentos'] valdrá 4 y romperá la condición. Por último y no menos importante, cambia que el query vaya primero Y QUE SE EJECUTE!

Eso debería resolver muchos de tus problemas

Tienes un error en la lógica, el sistema está haciendo lo que le estás pidiendo.

  • Estas intentando settear una variable de sesión $_SESSION['intentos'] = 0 antes del session_start()
  • Cuando se vuelve a invocar tu script, el primer comando es settear $_SESSION['intentos'] = 0, esto hará que los intentos sean infinitos
  • Siempre antes que un echo, van las instrucciones que debes hacer, lo estás haciendo mal en el paso 3, estás invocando un script que redirige a otra página, eso puede ocasionar que el PHP no termine de hacer su trabajo.
  • En ningún caso estás invocando la ejecución de tus consultas

Te propongo esta solución antes del try

session_start();
if( array_key_exists( 'intentos', $_SESSION ) ){
    $_SESSION['intentos'] = $_SESSION['intentos'] + 1;
} else {
    $_SESSION['intentos'] = 0;
}

Esto validará si hay ya una variable de sesión con el índice intentos, si existe, le suma 1, si no existe, la crea en 0

if( $_SESSION['intentos'] >= 3 ){
$sql = "UPDATE usuario SET Estado = 'Inactivo' WHERE Usuario = '$login'";
$base->exec($sql);
      echo "<script>swal({
        title: 'Cuenta bloqueada',
        text: 'Ingrese de nuevo sus datos',
        type: 'error',
        confirmButtonText: 'Aceptar',
        allowOutsideClick: false,
        confirmButtonColor: '#E12A2A'})  
        .then((result) => { 
        window.location=window.location='index.php'});</script>";
}

Cambié la condición que sea == 3 a que sea >= 3, si la persona por X razón vuelve a enviar la información, lo dejará pasar, puesto que $_SESSION['intentos'] valdrá 4 y romperá la condición. Por último y no menos importante, cambia que el query vaya primero Y QUE SE EJECUTE!

Eso debería resolver muchos de tus problemas

Tienes un error en la lógica, el sistema está haciendo lo que le estás pidiendo.

  • Estas intentando settear una variable de sesión $_SESSION['intentos'] = 0 antes del session_start()
  • Cuando se vuelve a invocar tu script, el primer comando es settear $_SESSION['intentos'] = 0, esto hará que los intentos sean infinitos
  • Siempre antes que un echo, van las instrucciones que debes hacer, lo estás haciendo mal en el paso 3, estás invocando un script que redirige a otra página, eso puede ocasionar que el PHP no termine de hacer su trabajo.
  • En ningún caso estás invocando la ejecución de tus consultas

Te propongo esta solución antes del try

session_start();
if( array_key_exists( 'intentos', $_SESSION ) ){
    $_SESSION['intentos'] = $_SESSION['intentos'] + 1;
} else {
    $_SESSION['intentos'] = 0;
}
$sql = "UPDATE usuario SET intentos = $_SESSION['intentos'] WHERE Usuario = '$_POST['login']'";
$base->exec($sql);

Esto validará si hay ya una variable de sesión con el índice intentos, si existe, le suma 1, si no existe, la crea en 0

if( $_SESSION['intentos'] >= 3 ){
$sql = "UPDATE usuario SET Estado = 'Inactivo' WHERE Usuario = '$login'";
$base->exec($sql);
      echo "<script>swal({
        title: 'Cuenta bloqueada',
        text: 'Ingrese de nuevo sus datos',
        type: 'error',
        confirmButtonText: 'Aceptar',
        allowOutsideClick: false,
        confirmButtonColor: '#E12A2A'})  
        .then((result) => { 
        window.location=window.location='index.php'});</script>";
}

Cambié la condición que sea == 3 a que sea >= 3, si la persona por X razón vuelve a enviar la información, lo dejará pasar, puesto que $_SESSION['intentos'] valdrá 4 y romperá la condición. Por último y no menos importante, cambia que el query vaya primero Y QUE SE EJECUTE!

Eso debería resolver muchos de tus problemas

Origen Enlace
Alberto Siurob
  • 6.5k
  • 7
  • 33
  • 72

Tienes un error en la lógica, el sistema está haciendo lo que le estás pidiendo.

  • Estas intentando settear una variable de sesión $_SESSION['intentos'] = 0 antes del session_start()
  • Cuando se vuelve a invocar tu script, el primer comando es settear $_SESSION['intentos'] = 0, esto hará que los intentos sean infinitos
  • Siempre antes que un echo, van las instrucciones que debes hacer, lo estás haciendo mal en el paso 3, estás invocando un script que redirige a otra página, eso puede ocasionar que el PHP no termine de hacer su trabajo.
  • En ningún caso estás invocando la ejecución de tus consultas

Te propongo esta solución antes del try

session_start();
if( array_key_exists( 'intentos', $_SESSION ) ){
    $_SESSION['intentos'] = $_SESSION['intentos'] + 1;
} else {
    $_SESSION['intentos'] = 0;
}

Esto validará si hay ya una variable de sesión con el índice intentos, si existe, le suma 1, si no existe, la crea en 0

if( $_SESSION['intentos'] >= 3 ){
$sql = "UPDATE usuario SET Estado = 'Inactivo' WHERE Usuario = '$login'";
$base->exec($sql);
      echo "<script>swal({
        title: 'Cuenta bloqueada',
        text: 'Ingrese de nuevo sus datos',
        type: 'error',
        confirmButtonText: 'Aceptar',
        allowOutsideClick: false,
        confirmButtonColor: '#E12A2A'})  
        .then((result) => { 
        window.location=window.location='index.php'});</script>";
}

Cambié la condición que sea == 3 a que sea >= 3, si la persona por X razón vuelve a enviar la información, lo dejará pasar, puesto que $_SESSION['intentos'] valdrá 4 y romperá la condición. Por último y no menos importante, cambia que el query vaya primero Y QUE SE EJECUTE!

Eso debería resolver muchos de tus problemas