Question

Tengo un problema con las aperturas y cierre de etiquetas en un php, tengo un lío de aupa

<?php session_start();

    $dbhost = "localhost";
    $dbuser = "corbachu";
    $dbpass = "xxx";
    $conx = mysql_connect($dbhost, $dbuser, $dbpass);

     mysql_select_db("corbachu_percy", $conx)or die ('Error al seleccionar la Base de Datos: '.mysql_error());;

    if(trim($_POST['email']) != "" && trim($_POST['password']) != "")
    {

    $sql = 'SELECT password, email, username FROM usuarios WHERE email=\''.$email.'\' and password=\''.$password.'\'';
    //$result = mysql_query('SELECT password, email, username FROM usuarios WHERE email=\''.$email.'\' and password=\''.$password.'\'');
    $result = mysql_query($sql);

        if($row = mysql_fetch_array($result)){  

        if($row['password'] == $password){
                $_SESSION['fp_username'] = $row['username'];
                $_SESSION['email'] = $row['email'];

                $_SESSION['fp_username'];
    ?>

                <SCRIPT LANGUAGE="javascript">

                location.href = "zona.php";

                    </SCRIPT>'

            }else{

                location.href = "errorPassword.html";
        <?      
            }
        }else{
    ?>      location.href = "errorPassword.html";
    <?  }
        mysql_free_result($result);
    }else{
        echo 'Debe especificar un usuario y password';
    }

    mysql_close();
    ?>

Cuando la password es incorrecta se me muestra literalmente en pantalla "errorPassword.html"

Muchas gracias

Comments

¿Tendría entonces que poner el cierre del </SCRIPT> digamos que justo antes de cerrar la bbdd . antes de mysql_close() , ?. ¿ Cómo entonces redirigiría al usuario?

Muchas gracias

Answer 1

Hola gonzalss,

el error no está en la apertura y cierres de PHP, sino que estás cerrando el tag </SCRIPT> después del location.href = "zona.php"; en vez de cerrarlo cuando termina el código JavaScript.

Todo código JavaScript puesto fuera de <SCRIPT></SCRIPT> no será ejecutado por el navegador...

Cambiando de tema, estás usando una forma muy endeble de validar un usuario, ya que el código JavaScript del que epende la validación está del lado del cliente (y sometido a sus cambios).

Hacé todo esto en PHP y redirigí las paǵinas con header("Location:pagina.php"); que al estar en el servidor, no es modificable por el que navega la página.

Saludos!