Problemas al querer mantener el valor de un select

Swab · #1 (**permalink**) 27/04/2012, 09:41

Amig@s, tengo un problema y ya he visto las FAQ's y he googleado, no sé qué estoy haciendo mal, pero al querer modificar un registro de mi base de datos, me muestra el último ROL de usuario.
Este es mi código:

Código PHP:

Ver original<?php
                $tablaroles = mysql_query("SELECT * FROM roles"); 
                ?>
                <select name="id_roles"  class="select" id="roles">
                     <OPTION>-- Seleccione --</OPTION>
                      <?php
                           while ($roles = mysql_fetch_array($tablaroles))
                            {
                              if ($id_rol == $roles['id_rol'])
                              ?>
                     <OPTION selected='selected' VALUE='<?=$roles['id_rol']; ?>'><?php echo $roles['nombre']; ?></OPTION>                   
                          <?php
                             }
                         ?>
 
            <?php
            mysql_free_result($tablaroles); 
        ?>
  </select>

Tengo una de usuarios, que paso por ID los datos a un FORM, pero si el usuario que selecciono tiene ROL 1, al hacer el clic en el form me muestra el último ID.
Una ayuda porfavor amig@s

Swab

santris · #2 (**permalink**) 27/04/2012, 09:44

como recoges los valores y haces la siguiente consulta??

Swab · #3 (**permalink**) 27/04/2012, 09:56

Hola santris, los valores los tomo así:

Código PHP:

Ver original$id_rol    = $_POST["id_rol"];

los demás datos de igual manera, pasan, me muestra todos los datos y no teno problema al realizar el UPDATE.
tengo una de ususarios que tiene como clave foranea id_rol.
Después como ves arriba comparo el valor que me pasó es el 1 lo comparo en el if.
Saludos

santris · #4 (**permalink**) 27/04/2012, 10:09

Cita:

Iniciado por Swab

Hola santris, los valores los tomo así:

Código PHP:

Ver original$id_rol    = $_POST["id_rol"];

Cita:

<select name="id_roles"

el select se llama id_roles no id_rol

Swab · #5 (**permalink**) 27/04/2012, 10:29

Amigo santris, he probado tomando el name del select, pero igual me muestra el último ID rol, tengo 6 valores de ROL y me muestra siempre el último, sin importar si los usuarios son 1, 2, 3, etc...
He buscado en las FAQ's, en google, no tienes un ejemplo que funcione que me puedas mostrar?, te lo agradecería.

Swab

mogurbon · #6 (**permalink**) 27/04/2012, 10:38

cuando lo veas ya en tu navegador , dale ver codigo fuente en el mismo y ve que es lo que te esta imprimiendo , probablemente estes poniendole selected a todos los options

Swab · #7 (**permalink**) 27/04/2012, 10:43

Efectivamente amigo, vi y me salen todos como dices seleccionados... me estoy mareando, como podría ordenar eso?, cómo quedaría? ...
Mi código está al incio, ayuda porfavor.

Swab

mogurbon · #8 (**permalink**) 27/04/2012, 10:47

tienes un if pero no un else , que el else te dibuje el option sin estar seleccionado

Swab · #9 (**permalink**) 27/04/2012, 11:05

Hago el else y me sigue pasando lo mismo, osea ahora entra en el else... no me muestra seleccionado nada.
pf
:(

mogurbon · #10 (**permalink**) 27/04/2012, 11:08

a ver pon el code por que quien sabe que fue lo que paso ahi XD

Swab · #11 (**permalink**) 27/04/2012, 11:13

Código PHP:

Ver original<?php
error_reporting(E_ALL & ~E_NOTICE); 
require("../config/conexion.php");
require("../config/funciones.php");
 
$iduser  = getParam($_GET["id"], "-1");
$accion    = getParam($_GET["accion"], "");
 
if ($accion == "editar") {
    $iduser    = sqlValue($_POST["id"], "int");
    $nombre    = sqlValue($_POST["nombre"], "text");
    $apellido  = sqlValue($_POST["apellido"], "text");
    $usuario   = sqlValue($_POST["usuario"], "text");
    $password  = sqlValue($_POST["password"], "text");
    $id_role   = sqlValue($_POST["id_roles"], "int");
 
    
    $sql = "UPDATE usuarios SET nombre=".$nombre.", apellido=".$apellido.", user=".$usuario.", password=".$password.",id_rol=".$id_role." 
            WHERE id=".$iduser."";
    mysql_query($sql, $conexion);
    header("location: list_edit_usuarios.php");
}
$sql = "SELECT * FROM usuarios 
                 WHERE id = ".sqlValue($iduser, "int");
$queUser = mysql_query($sql, $conexion);
$rsUser = mysql_fetch_assoc($queUser);
$total = mysql_num_rows($queUser);
if ($total == 0) {
    header("location: list_edit_usuarios.php");
    exit;
}
?>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<script src="../js/jquery-1.3.1.min.js" type="text/javascript"></script>
<script src="../js/jquery.validate.min.js" type="text/javascript"></script>
<script type="text/javascript">
$(document).ready(function() {
    $("#editFrm").validate();
});
</script>
<link href="../css/styles.css" rel="stylesheet" type="text/css" />
</head>
<body>
<fieldset id="form">
<fieldset id="form">
<legend>Editar Usuario</legend>
<form method="post" id="editFrm" name="editFrm" action="edit_usuarios.php?accion=editar">
<ol>
        <li>
    <label for="nombre">Nombre</label>
    <input type="text" id="nombre" name="nombre" class="required" value="<?php echo $rsUser["nombre"]; ?>" />
</li><li>
    <label for="apellido">Apellido</label>
    <input type="text" id="apellido" name="apellido" value="<?php echo $rsUser["apellido"]; ?>" />
</li><li>
    <label for="usuario">Usuario</label>
    <input type="text" id="usuario" name="usuario" value="<?php echo $rsUser["user"]; ?>" />
</li><li>
    <label for="password">Password</label>
    <input type="text" id="password" name="password" value="<?php echo $rsUser["password"]; ?>" />
</li>
              <li>
            <label>Privilegios</label>
            <?php
                $tablaroles = mysql_query("SELECT * FROM roles"); 
                ?>
                <select name="id_roles"  class="select" id="id_roles">
                     <OPTION>-- Seleccione --</OPTION>
                      <?php
                           while ($roles = mysql_fetch_array($tablaroles))
                            {
                              if ($id_roles == $roles['id_rol']){
                              ?>
                             <OPTION VALUE='<?=$roles['id_rol']; ?>' selected='selected'><?php echo $roles['nombre']; ?></OPTION>                   
                            <?php 
                            }else{
                            ?>
                            <OPTION VALUE='<?=$roles['id_rol']; ?>'><?php echo $roles['nombre']; ?></OPTION>                    
                            <?php 
                            }
                            ?>
                          <?php
                             }
                         ?>
 
            <?php
            mysql_free_result($tablaroles); 
        ?>
  </select>
             </li>
    <p align="center"><input type="submit" class="btn" value="Actualizar">&nbsp;&nbsp;
          <input type="reset" name="limpiar" value="Limpiar" class="btn" /></p>
<p align="center"><a href="JavaScript:window.close()"><input type="button" value="Cerrar" class="btn"></a></p>
    <input type="hidden" id="id" name="id" value="<?php echo $rsUser["id"]; ?>" />
   </ol>
    </form>
</fieldset>
</fieldset>
</body>
</html>

Ahí está todo el código...
Arriba hago la consullta a usuarios y usuarios tiene como FK la PK de roles, el id_rol, que es el que actualizo después.
Abajo antes del select hago la consulta a roles para que me los liste todos.
Saludos y garcias

scod · #12 (**permalink**) 27/04/2012, 11:16

es porque en tu línea:

Cita:

estas poniendo el selected='selected' a todos los registros que estás recorriendo, tienes que comparar el registro que quieres mostrar seleccionado, osea hacer un if al $roles['id_rol'] con $id_rol

saludos

Swab · #13 (**permalink**) 27/04/2012, 11:22

Creo que por ahí no va el camino amigo... gracias de todos modos, se supone que hay un if y luego un else. Y compara con el $id_roles que es el nombre de mi select, no?, por qué sería $id_rol?.

Saludos

santris · #14 (**permalink**) 27/04/2012, 11:40

Primero abogo por escribir código en minúsculas, ustedes vieron a Machado escribir en mayusculas salvo en ocasiones muy concretas???

Luego por simplificar y estructurar bien el cógigo:

Código PHP:

 
<select name="id_roles"  class="select" id="id_roles">
    <option>-- Seleccione --</option>
    <?php
    while ($roles = mysql_fetch_array($tablaroles)){
        if ($id_roles == $roles['id_rol'])$seleccionado = "selected='selected'"; else $seleccionado = '';
        ?>
            <option value='<?php echo $roles['id_rol']; ?>' <?php echo $seleccionado;?> ><?php echo $roles['nombre']; ?></option>                  
        <?php
    }
    mysql_free_result($tablaroles);
    ?>
</select>

y solventado este problema pasamos a otra cosa...

Swab · #15 (**permalink**) 27/04/2012, 11:52

Gracias por toda la ayuda santris, ahora no me selecciona nada, en fin.
Dejemos el tema hasta acá creo, ya es mucha pérdida de tiempo, no crees?, con lo que me muestras pasa directamente al else... me muestra -- Seleccione --
Gracias a todos

Swab

santris · #16 (**permalink**) 27/04/2012, 12:22

Cita:

Iniciado por Swab

Gracias por toda la ayuda santris, ahora no me selecciona nada, en fin.
Dejemos el tema hasta acá creo, ya es mucha pérdida de tiempo, no crees?, con lo que me muestras pasa directamente al else... me muestra -- Seleccione --
Gracias a todos
Swab

Hombre!! no quise decir que sea perdida de tiempo; te digo que un código es más legible si está bien estructurado y si pasa al else directamente será porque la condición no se cumple; antes era todo de un color y ahora es de otro, simplemente!!

prueba ha hacer esto fuera del select:

Código PHP:

  echo $id_roles."=".$roles['id_rol'];

si se cumple la codición el option debería de aparecer seleccionado, a no ser que haya cometid algún error de picaje, que es posible..

y reitero, para mi no es perdida de tiempo

Edito: aunque veo que la variable es $id_role no id_roles

Swab · #17 (**permalink**) 27/04/2012, 12:32

Código PHP:

Ver original<?php
                $tablaroles = mysql_query("SELECT * FROM roles"); 
                ?>
                <select name="id_roles"  class="select" id="id_roles">
    <option>-- Seleccione --</option>
    <?php
    while ($roles = mysql_fetch_array($tablaroles)){
        if ($id_roles == $roles['id_rol'])$seleccionado = "selected='selected'"; else $seleccionado = '';
        ?>
            <option value='<?php echo $roles['id_rol']; ?>' <?php echo $seleccionado;?> ><?php echo $roles['nombre']; ?></option>                  
        <?php
    }
    mysql_free_result($tablaroles);
    ?>
</select>
  <?php 
  echo $id_roles."=".$roles['id_rol'];  
  ?>

Quedando de esa manera el código, pasa al else... :s
Gracias santris.
PD: no existirá un ejemplo funcionando que conozcas...

santris · #18 (**permalink**) 27/04/2012, 12:43

edité mi post anterior porque ví que estabas comparando una variable que no existe:

Cita:

Edito: aunque veo que la variable es $id_role no id_roles

lo de:

Código PHP:

  echo $id_roles."=".$roles['id_rol'];

es simplemente para comprobar, puedes quitarlo no es necesario.

remplaza esta linea de código:

Código PHP:

  if ($id_roles == $roles['id_rol'])$seleccionado = "selected='selected'"; else $seleccionado = '';

por esta:

Código PHP:

  if ($id_role == $roles['id_rol'])$seleccionado = "selected='selected'"; else $seleccionado = '';

Swab · #19 (**permalink**) 27/04/2012, 12:50

sabes que ya lo había probado eso, y nada, jajajaja... es muy extraño... no está entrando al if, donde comparo, por alguna extraña razón que no me doy cuenta, esto me tiene de ayer así, eliminé todo el código y comencé de nuevo, pero llego al mismo problema.
Ya no sé que más hacer, he buscado en google algo y no encuentro nada como para poder adaptarlo y ya... no puedo estar tanto tiempo en esto, estoy haciendo mi proyecto de tesis, me queda como 25 días y esta tontería que me tiene acá es una pequeña parte.
ufff....

Saludos y nuevamente gracias

santris · #20 (**permalink**) 27/04/2012, 13:00

solo te puedo decir que depures el código delimitándolo, reporta el valor de cada variable hasta encontrar donde está el error.

echo $_POST['lo que sea '] etc....

Swab · #21 (**permalink**) 27/04/2012, 13:15

Gracias Santris,
muchas gracias por todo el apoyo

Swab