Almacenar dato según radiobutton.

[brunob1523]

Buenas! Como va? Soy nuevo por acá...

Estoy husmeando un poco esto de php y mysql, y me dispuse a hacer unas cositas.
La cosa es que se me tranqué con algo y quería consultarles a ver si ustedes pueden decirme que es lo que estoy haciendo mal :P

Lo que quiero hacer es: tengo un textbox donde se ingresa un ID, 2 radiobuttons y un botón de submit.
Y según el radio que se seleccione, se modifique un campo en la base de datos.
Peeeeeero, resulta que con el código que tengo, no hace nada. ja


Este es mi código:

Código:

<?php
$estado=$_POST['radio'];
$id=$_POST['IDSolicitud'];

if($estado==1){
$guardar = mysql_query("UPDATE personas SET estado=\"Confirmado\" WHERE id='$id');");
}

if(estado==2){
$guardar = mysql_query("UPDATE personas SET estado=\"Rechazado\" WHERE id='$id');");
}

mysql_close(); 

?>

Código:

<form id="form1" name="form1" method="post" action="estado.php">
      <label>
      <div align="right"><span class="igual">ID de solicitud</span>
        <input name="IDSolicitud" type="text" id="IDSolicitud" size="10" />
      </div>
      <br />
      </label>
      <label>
      <div align="right"><span class="igual">        Aceptar      </span>
        <input name="radio" type="radio" id="si" value="1" checked="checked" />
        <span class="igual">Rechazar</span>
        <input type="radio" name="radio" id="no" value="2" />
        <br />
        <br />
        <input type="submit" name="Confirmar" id="Confirmar" value="Confirmar" />
      </div>
      </label>
      <label></label>
    </form>

Alguna idea de que estoy haciendo mal?

Gracias!!!

[DeivisAndres]

Por que pones: $guardar = mysql_query("UPDATE personas SET estado=\"Confirmado\" WHERE id='$id');");

no seria de esta manera: $guardar = mysql_query("UPDATE personas SET estado='Confirmado' WHERE id = $id");, la variable si es de tipo INT es sin la comillas simples pero si es de tipo VARCHAR es con las comillas simples. Y aparte de eso no veo que este declarando la conexión que seria; mysql_select_db($database_conexion, $conexion);

Quedaría de esta manera:

Código PHP:

<?php
    $estado = $_POST['radio'];
    $id=$_POST['IDSolicitud'];
 
    if($estado==1){
        $SqlQuery = "UPDATE personas SET estado='Confirmado' WHERE id = $id";
        mysql_select_db($database_conexion, $conexion) or die('No se puede conectar a la base de datos. ' . mysql_error());
        $guardar = mysql_query($SqlQuery, $conexion);
    }
 
    if(estado==2){
        $SqlQuery = "UPDATE personas SET estado='Rechazado' WHERE id = $id";
        mysql_select_db($database_conexion, $conexion) or die('No se puede conectar a la base de datos. ' . mysql_error());
        $guardar = mysql_query($SqlQuery, $conexion);
    }
 
    mysql_close(); 
 
?>
 

[brunob1523]

Sin las comitas tampoco funciona, las puse en un intento de encontrar el error. (Ya las quité).

La consulta funciona bien porque la probé directamente en phpmyadmin.

Y la conexión ya está establecida, solo que no la puse en el código que mostré aquí arriba.

Gracias!

[DeivisAndres]

Que bien que te funcione, y probaste el ejemplo que te puse.

[brunob1523]

No entendí que sería $database_conexion y $conexion, yo lo hago por separado.
Lo tengo así:

<?php

$db_host="xxx";
$db_usuario="xxx";
$db_password="xxx";
$db_nombre="xxx";
$conexion = @mysql_connect($db_host, $db_usuario, $db_password) or die(mysql_error());
$db = @mysql_select_db($db_nombre, $conexion) or die(mysql_error());

$estado=$_POST['radio'];
$id=$_POST['IDSolicitud'];

if($estado==1){
$guardar = mysql_query("UPDATE personas SET estado=\"Confirmado\" WHERE id=$id');");
}

if(estado==2){
$guardar = mysql_query("UPDATE personas SET estado=\"Rechazado\" WHERE id=$id);");
}

mysql_close();

?>


EDIT: Nop, tampoco me anda con ese ejemplo.

[DeivisAndres]

mmmm ok bueno seria de esta forma entonces:

Código PHP:

<?php
    $db_host="xxx"; 
    $db_usuario="xxx"; 
    $db_password="xxx"; 
    $db_nombre="xxx"; 
    $conexion = @mysql_connect($db_host, $db_usuario, $db_password) or die(mysql_error()); 
    $db = @mysql_select_db($db_nombre, $conexion) or die('No se puede conectar a la base de datos. ' . mysql_error());
 
        $estado = $_POST['radio'];
        $id = $_POST['IDSolicitud'];
 
        if($estado==1){
            $SqlQuery = "UPDATE personas SET estado='Confirmado' WHERE id = $id";
            $guardar = mysql_query($SqlQuery, $conexion);
        }
 
        if(estado==2){
            $SqlQuery = "UPDATE personas SET estado='Rechazado' WHERE id = $id";            
            $guardar = mysql_query($SqlQuery, $conexion);
        }
 
        mysql_close(); 
 
?>
 

[brunob1523]

Opa, hay avances. Ahora funciona, pero parcialmente, funciona solo con la parte de Confirmado.

[DeivisAndres]

Porque en la parte donde verifica que si es 2 le falta $:

Asi lo tienes:

Código PHP:

if(estado==2)
 

Debe ser asi:

Código PHP:

if($estado==2){
 

[Erickpc]

Saludos, en el segundo if te falta "$",,, tomando el ejemplo, yo lo realizaría así

<?php
$db_host="xxx";
$db_usuario="xxx";
$db_password="xxx";
$db_nombre="xxx";
$conexion = @mysql_connect($db_host, $db_usuario, $db_password) or die(mysql_error());
$db = @mysql_select_db($db_nombre, $conexion) or die('No se puede conectar a la base de datos. ' . mysql_error());

$estado = $_POST['radio'];
$id = $_POST['IDSolicitud'];

if($estado==1)
{
$estado='Confirmado';
}else
{
$estado='Rechazado';
}


$SqlQuery = "UPDATE personas SET estado=".$estado." WHERE id = $id";
$guardar = mysql_query($SqlQuery, $conexion);


mysql_close();

?>

[brunob1523]

Uhh, que bolu no me di cuenta!
Ahora si, quedó funcionando impecable.

Muchas gracias gente!
Saludos.