login redireccionado

rafaxusirati · #1 (**permalink**) 28/11/2014, 21:14

Hola buenas noches/ días.

Tengo el siguiente código que debería chequear al usuario que entra por su usuario y contraseña y posteriormente re-dirigirlo a otra página, donde debería mostrar todos los registros de este usuario (correspondientes a su email) que se encuentran en una BD.

Alguna sugerencia o código mejorado.

mi código de verificación es:

Código PHP:

Ver original<?php require_once('../../Connections/contactos.php'); ?>
<?php
if (!function_exists("GetSQLValueString")) {
function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") 
{
  if (PHP_VERSION < 6) {
    $theValue = get_magic_quotes_gpc() ? stripslashes($theValue) : $theValue;
  }
 
  $theValue = function_exists("mysql_real_escape_string") ? mysql_real_escape_string($theValue) : mysql_escape_string($theValue);
 
  switch ($theType) {
    case "text":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;    
    case "long":
    case "int":
      $theValue = ($theValue != "") ? intval($theValue) : "NULL";
      break;
    case "double":
      $theValue = ($theValue != "") ? doubleval($theValue) : "NULL";
      break;
    case "date":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;
    case "defined":
      $theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue;
      break;
  }
  return $theValue;
}
}
 
mysql_select_db($database_contactos, $contactos);
$query_Recordset1 = "SELECT * FROM contactos";
$Recordset1 = mysql_query($query_Recordset1, $contactos) or die(mysql_error());
$row_Recordset1 = mysql_fetch_assoc($Recordset1);
$totalRows_Recordset1 = mysql_num_rows($Recordset1);
?>
<?php
// *** Validate request to login to this site.
if (!isset($_SESSION)) {
  session_start();
}
 
$loginFormAction = $_SERVER['PHP_SELF'];
if (isset($_GET['accesscheck'])) {
  $_SESSION['PrevUrl'] = $_GET['accesscheck'];
}
 
if (isset($_POST['textfield'])) {
  $loginUsername=$_POST['textfield'];
  $password=$_POST['textfield2'];
  $loginUsername2=$_POST['textfield3'];
  $MM_fldUserAuthorization = "";
  $MM_redirectLoginSuccess = "modificar.php";
  $MM_redirectLoginFailed = "acceso no permitido.php";
  $MM_redirecttoReferrer = false;
  mysql_select_db($database_contactos, $contactos);
  
  $LoginRS__query=sprintf("SELECT nombre_usuario, password, mailcontacto FROM contactos WHERE nombre_usuario=%s AND password=%s AND mailcontacto=%s",
    GetSQLValueString($loginUsername, "text"), GetSQLValueString($password, "text"), GetSQLValueString($loginUsername2, "hidden"));
   
  $LoginRS = mysql_query($LoginRS__query, $contactos) or die(mysql_error());
  $loginFoundUser = mysql_num_rows($LoginRS);
  if ($loginFoundUser) {
     $loginStrGroup = "";
    
    //declare two session variables and assign them
    $_SESSION['MM_Username'] = $loginUsername;
    $_SESSION['MM_UserGroup'] = $loginStrGroup;
    $_SESSION['MM_Username'] = $loginUsername;        
 
    if (isset($_SESSION['PrevUrl']) && false) {
      $MM_redirectLoginSuccess = $_SESSION['PrevUrl'];  
    }
    header("Location: " . $MM_redirectLoginSuccess );
  }
  else {
    header("Location: ". $MM_redirectLoginFailed );
  }
}
?><style type="text/css">
<!--
body {
    background-color: WHITE;
}
a:link {
    text-decoration: none;
}
a:visited {
    text-decoration: none;
}
a:hover {
    text-decoration: underline;
}
a:active {
    text-decoration: none;
}
-->
</style>
<div align="center">
  <table width="78%" border="0">
    <tr>
      <td height="120"><div align="center">
        <table width="100%" border="0" cellspacing="0">
          <tr>
            <td height="0" bgcolor="WHITE"><div align="center">___ </div></td>
            </tr>
          <tr>
            <td height="52" bgcolor="WHITE"><div align="center">
              <table width="60%" border="0">
                <tr>
                  <td bgcolor="WHITE"><div align="center"><strong style="font-size: xx-large; color: #000;">Talla variada</strong></div></td>
                  </tr>
                </table>
              </div></td>
            </tr>
          <tr>
            <td height="17" bgcolor="WHITE">&nbsp;</td>
            </tr>
          <tr>
            <td bgcolor="WHITE"><div align="center">
              <table width="17%" border="0">
                <tr>
                  <td width="47%"><div align="center"> <img src="http://.../datos internos/fotos/regresar.png" alt="Regresar " width="33" height="35" hspace="10" border="0" align="absmiddle" onclick="history.back(-1);" />Regresar</div></td>
                  </tr>
                </table>
              </div></td>
            </tr>
          </table>
      </div></td>
    </tr>
    <tr>
      <td><div align="center">
        <table width="100%" border="0">
          <tr>
            <td height="4" bgcolor="WHITE">&nbsp;</td>
            </tr>
          </table>
      </div></td>
    </tr>
  </table>
</div>
<div align="center"></div>
<form name="form1" method="POST" action="<?php echo $loginFormAction; ?>">
  <label><br>
  </label>
  
  <div align="center">
    <table width="43%" border="0">
      <tr>
        <td height="30" colspan="2" bgcolor="white"><div align="center" style="color: black; font-weight: bold; font-size: large;">Autentificaci&oacute;n usuario:</div></td>
      </tr>
      <tr>
        <td width="40%" bgcolor="WHITE"><div align="right" style="color: black">Usuario:</div></td>
        <td width="60%" bgcolor="WHITE"><div align="left">
          <input type="loginUsername" name="textfield" id="textfield">
        </div></td>
      </tr>
      <tr>
        <td bgcolor="WHITE"><div align="right" style="color: black">Contrase&ntilde;a:</div></td>
        <td bgcolor="WHITE"><div align="left">
          <input type="password" name="textfield2" id="textfield2">
        </div></td>
      </tr>
      <tr>
        <td bgcolor="WHITE">&nbsp;</td>
        <td bgcolor="WHITE"><div align="left">
          <input name="loginUsername2" type="hidden" id="textfield3" value="modificar.php?idempleado=<?php echo $row_Recordset1['mailcontacto']; ?>" />  
        </div></td>
      </tr>
    </table>
<table width="88%" border="0">
      <tbody>
        <tr>
          <td height="55" bgcolor="WHITE"><div align="center"><span style="font-size:small; color:black;">Recuerde la referencia de su pedido.</span></div></td>
        </tr>
      </tbody>
    </table>
<p>
      <label>
        <input type="submit" name="button" id="button" value="Enviar">
      </label>
    </p>
  </div>
</form>
<div align="center"></div>
<p align="center"><a href="http://..."></a></p>
<div align="center">_ </div>
<p>
  <?php
mysql_free_result($Recordset1);
?>
</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>
<p>&nbsp;</p>

el código para validar me da error al enviarlo.

Código HTML:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Pretendo que al validar el usuario, se redirija a otra página, donde el usuario pueda visualizar sus artículos contenidos en una BD, para poder modificarlos o eliminarlos.

Muchas gracias

manuparquegiralda · #2 (**permalink**) 29/11/2014, 13:27

Obviamente el fallo lo tienes en las sintaxis del query a mysql, sinceramente, sugerencias para mejorar el código, desde mi punto de vista, muchas, no entiendo muy bien porque haces tantas cosas para un script tan sencillo, si lo único que pretendes es validar un usuario.

- No es para nada necesario que habras y cierres etiquetas <?php ?> si no hay código en otro lenguaje entre una y otra.

- Deberías haber puesto el código del archivo que incluyes al principio del script, ya que hay variables como la de $contactos que no se de donde y porque salen.

- No sé porque preguntas si la función GetSQLValueString() no existe para declararla en lugar de declararla directamente.

- Si ya conoces tu versión PHP, porque andar haciendo comprobaciones de la versión.

- ¿Por qué compruebas si hay variable de sesión para declarar el session_start()?, decláralo y punto.

- Intenta no crear arcivos php con espacios en el nombre, en alguna ocasión me he visto afectado por tonterías así, por ejemplo en lugar de "acceso no permitido.php" llámalo "acceso_no_permitido.php".

- Evita las tablas en HTML, a Google no le hacen mucha gracia según tengo entendido.

No sé, me voy a molestar un poco para darte una guía de como trabajo yo, también fui novato y en este foro hubo gente muy buena que me ayudó bastante.

Código PHP:

Ver original<?php session_start();
 
if((isset($_POST['user']))&&(isset($_POST['pass']))){
    // CONECTAS CON LA BASE DE DATOS //
    $mysqli = new mysqli("tu dominio", "tu usuario", "tu password", "tu base de datos");
    $mysqli->set_charset("utf8");
    // PREGUNTAS SI EXISTE EL USUARIO CON EL PASSWORD //
    $p = $mysqli->query("SELECT * FROM contactos WHERE nombre_usuario = '".$_POST['user']."' AND password = '".$_POST['pass']."'");
    $aUser = $p->fetch_assoc();
    if(!$aUser['nombre_usuario']){
        // SI NO SE EXTRAEN DATOS DE LA CONSULTA MOSTRAMOS EL ERROR //
        $error = "El usuario o el password no son correctos.";
    }else{
        ////////////////////////////////////////////////////////////////////////////////////////////////////////////////
        // CREAMOS VARIABLES DE SESSION CON EL USUARIO ESTE DEBERÍA SER ÚNICO PARA QUE EN LA PÁGINA DE DESTINO /////
        // PUEDAS VOLVER A HACER LA CONSULTA A LA BASE DE DATOS PARA EXTRAER LOS DATOS DEL USUARIO O TAMBIÉN ////////
        // PUEDES CREAR UNA VARIABLE DE SESSION PARA CADA DATO DE LA BASE DE DATOS, PERO SI TIENES MUCHOS CAMPOS //
        // PUEDE ACABAR SIENDO UN POCO LIOSO LA VERDAD, ESO LO ELIGES TÚ. YO TRABAJO CON CLASES POR LO QUE LO /////////
        // GUARDARÍA TODOS LOS DATOS NECESARIOS EN UN OBJETO QUE SERIALIZARÍA EN UNA VARIABLE DE SESSION QUE  //////
                // PUEDA LEER EN CADA PÁGINA SIN NECESIDAD DE DE VOLVER A CONSULTAR A LA BASE DE DATOS NI DE TENER UN ///////
        //  NÚMERO INMENSO DE VARIABLES DE SESSION /////////////////////////////////////////////////////////
        //////////////////////////////////////////////////////////////////////////////////////////////////////////////////
        $_SESSION['user'] = $aUser['nombre_usuario'];
        header("location:modificar.php");
        exit();
    }
}else{
    $error = "Tienes que rellenar todos los campos.";
}
 
?>
 
 
<!doctype html>
<html>
<head>
<meta charset="utf-8">
<title>Loguin</title>
<style type="text/css">
body {
    margin:0px;
    padding:0px;
    padding-top:20px;
}
.table{
    position:relative;
    display:table;
    width:500px;
    margin:0 auto;
}
.table > div{
    position:relative;
    display:table-row;
}
.cell_izq{
    position:relative;
    display:table-cell;
    width:50%;
    text-align:right;
    vertical-align:middle;
    padding:5px;
    font-weight:bold;
}
.cell_der{
    position:relative;
    display:table-cell;
    width:50%;
    text-align:left;
    vertical-align:middle;
    padding:5px;
}
input[type=text],input[type=password]{
    width:200px;
}
input[type=submit]{
    font-weight:bold;
    color:#FFF;
    background:#42349F;
    border:solid 1px #CCC;
}
.cont_button{
    position:relative;
    display:table;
    width:500px;
    margin:0 auto;
    text-align:center;
}
input[type=submit]:hover{
    color:#42349F;
    background:#EDEDED;
}
p{
    text-align:center;
    margin-top:10px;
    margin-bottom:10px;
    font-weight:bold;
    color:#F00;
}
</style>
</head>
 
<body>
<form method="POST" action="<?php echo $_SERVER['PHP_SELF']; ?>">
<div class="table">
    <div>
        <div class="cell_izq">
            Usuario:
        </div>
        <div class="cell_der">
            <input type="text" name="user" />
        </div>
    </div>
    <div>
        <div class="cell_izq">
            Contraseña:
        </div>
        <div class="cell_der">
            <input type="password" name="pass" />
        </div>
    </div>
</div>
<div class="cont_button">
    <input type="submit" value="Enviar">
</div>
<p><?php echo $error; ?></p>
</body>
</form>
</html>

Prueba con esta página que te he hecho, debería valerte, tan solo tienes que cambiar la conexión a la base de datos y modificar los campos en las consultas si yo los he puesto mal.

rafaxusirati · #3 (**permalink**) 29/11/2014, 20:14

muchas gracias