Update no funciona....

[luxo]

Amigos estoy aplicando un archivo editar.php obviamente el cual edita los registros de la base de datos, en realidad los edita pero me deja el id=0, agradecería mucho me ayudarán en que parte del codigo estoy cometiendo el error....

Código PHP:

<?php 
include 'includes/config.php'; 
include 'menu.php';
?>
<?php
$edit =$_GET["id"];
if ($_POST["action"] == "edit") {
$query = "UPDATE producto SET id_prod='".$id_prod."', nombre_prod='".$nombre_prod."', descripcion_prod='".$descripcion_prod."', img_prod='".$img_prod."', categoria='".$categoria."' WHERE id_prod = '{$_GET['id']}' LIMIT 1";
$result = mysql_query($query) or die(mysql_error());
}   
?>
 

Código HTML:

<form action="editar.php" method="POST" enctype="multipart/form-data">  
      <label for="field_1">Nombre Producto</label>
      <input type="text" name="nombre_prod" id="field_1" class="short" value="<?php echo $nombre_prod ?>" />
                    <label for="field_2">Categoría</label>
                
                    <? echo $producto['cat']; ?>
 
                    <label for="field_4">Descripción del Producto</label>
                    <textarea name="descripcion_prod" rows="3" cols="3" id="field_4"><?php echo $descripcion_prod ?></textarea>
                    <em>Help instructions for each form section goes below input field</em>
                    <input name="archivo" type="file" class="casilla" id="archivo" size="35" />
                    <input name="enviar" type="submit" class="boton" id="enviar" value="Grabar" />
                    <input name="id" value="<?php $_GET["id"]; ?>" type="hidden" />
                    <input name="action" type="hidden" value="edit" />
                    </form>
 

Muchas gracias por todo!!!

[pateketrueke]

pero si el método del formulario es POST, ¿porque utilizas $_GET??

además, te recuerdo que las variables no se imprimen solas...

Código PHP:

 <input name="id" value="<?php $_GET["id"]; ?>" type="hidden" />  

y ahí te falta mínimo un echo, inclusive estás usando register_globals otro punto negativo mas...

Código PHP:

  // MAL
$nombre_prod;

// BIEN
$_POST['nombre_prod'];  
   

[luxo]

OK muchas gracias pateketrueke,. pero no se bien como cambiar los valores para que logre editar me recomiendas usar POST o GET

[David]

¿De dónde sale el valor de la variable $id_prod? El ID no deberías cambiarlo, por lo tanto, sólo necesitas excluirlo de la lista de campos en el UPDATE ... SET ...

[luxo]

Ahi esta el codigo corregido por sus sugerencias, sin embargo aun no logro editar y guardar los registros en la base de datos.===

Código PHP:

<?php include 'includes/config.php'; ?>
<?php
$edit =$_GET["id"];
$link=conectar();
if ($_POST["action"] == "edit") {
$query = "UPDATE producto SET id_prod = ".$_POST['id'].", nombre_prod = ".$_POST['nombre_prod'].", descripcion_prod = ".$_POST['descripcion_prod'].", img_prod = ".$_POST['img_prod'].", categoria = ".$_POST['categoria']." WHERE id_prod = " . $_GET["id"];
$result = mysql_query($query) or die(mysql_error());
}
?>
 

Código HTML:

<form action="editar.php" method="POST" enctype="multipart/form-data">
<label for="field_1">Nombre Producto</label>
  <input type="text" name="nombre_prod" id="field_1" class="short" value="<?php echo $nombre_prod ?>" />
<label for="field_2">Categoría</label>
<? echo $producto['cat'];  ?>
<label for="field_4">Descripción del Producto</label>
<textarea name="descripcion_prod" rows="3" cols="3" id="field_4"><?php echo $descripcion_prod ?></textarea>
 <input name="archivo" type="file" class="casilla" id="archivo" size="35" />
<input name="enviar" type="submit" class="boton" id="enviar" value="Grabar" />
<input name="action" type="hidden" value="edit" />
<input name="id" value="<?php $_GET["id"]; ?>" type="hidden" />
</form>
 

De hecho ahora me arroja el error

Código MySQL:

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 ' nombre_prod = LU XXXX, descripcion_prod = A World Trade Center brings together ' at line 1
 

Cual es el error en la consulta SQL

[darkasecas]

Los valores de texto en consultas sql van entre comillas simples -_-

[rfadgrmm]

Hola Luxo,

No olvides usar las comillas simples para los campos del tipo texto y/o fecha/hora en tu sentencia SQL para que funcione el UPDATE...

Saludos...

[luxo]

Gracias darkasecas y rfadgrmm, aqui esta el codigo como lo señalaron sin embargo no se en donde tengo el error d verdad me esta complicando la cosa busco el erro y no lo encuento... MySQL me indica que tengo " adicional pero no la veo.

Código MySQL:

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 ''nombre_prod' = LU XXXX NO , 'descripcion_prod' = A World Trade Center' at line 1
 

Código PHP:

<?php include 'includes/config.php'; ?>
<?php
$edit =$_GET["id"];
$link=conectar();
if ($_POST["action"] == "edit") {
$query = "UPDATE producto SET 'nombre_prod' = ".$_POST['nombre_prod'].", 'descripcion_prod' = ".$_POST['descripcion_prod'].", 'img_prod' = ".$_POST['img_prod'].", 'categoria' = ".$_POST['categoria']." WHERE 'id_prod' = $edit";
$result = mysql_query($query) or die(mysql_error());
}
?>
 

Gracias a todos como siempe...

[rfadgrmm]

Lo que sucede luxo es que a cada nombre del campo le estás indicando con comillas simples, sólo las comillas se usan para indicar los valores a ingresar del tipo texto y/o fecha/hora, quitándole esas comillas de más en cada nombre del campo y asignándole al valor, la sentencia SQL puede funcionar...

Código PHP:

"UPDATE producto SET nombre_prod = '".$_POST['nombre_prod']."', descripcion_prod = '".$_POST['descripcion_prod']."', ..."
 

Revisa cada nombre de campo para retirar esas comillas simples, y recuerda que los datos en un campo de texto y/o fecha/hora van siempre entre comillas simples...

Saludos...

[luxo]

Gracias rfadgrmm, lo estoy verificando...

[pilucho]

Hola Luxo

Cambia tu codigo por este.

Código PHP:

  <? 
if (isset($_POST['action']) && $_POST['action'] == 'edit') 
{ 
    $id_prod = $_POST['id']; 
    $nombre_prod = $_POST['nombre_prod']; 
    $descripcion_prod = $_POST['descripcion_prod']; 
    $img_prod = $_POST['img_prod']; 
    $categoria = $_POST['categoria']; 
    $query = "UPDATE producto SET id_prod='id', nombre_prod='nombre_prod', descripcion_prod='descripcion_prod', img_prod='img_prod', categoria='categoria' WHERE id_prod='$id'"; 
    $result = mysql_query($query) or die(mysql_error()); 
 
    echo "<div align=center>Producto Actualizado</div>"; 
    else 
    echo "<div align=center>Producto No Actualizado</div>"; 
} 
 
?>   

Y tu formulario por este

Código HTML:

<form action="editar.php" method="POST" enctype="multipart/form-data">
<input type="hidden" name="id" value="<? echo $id; ?>">
<input type="hidden" name="action" value="edit">

Nombre Producto
<input type="text" name="nombre_prod" id="field_1" class="short" value="<?php echo $nombre_prod ?>" /> <br> 
Categoría <? echo $producto['cat']; ?><br> 
Descripción del Producto<br> 
<textarea name="descripcion_prod" rows="3" cols="3" id="field_4"><?php echo $descripcion_prod ?></textarea><br> 
<input name="archivo" type="file" class="casilla" id="archivo" size="35" /><br> 
<input name="enviar" type="submit" class="boton" id="enviar" value="Grabar" />
</form> 

Me dices como te fue

[luxo]

Pilucho gracias por tu gran ejemplo.... el problema que tengo ahora que me sobreescribe la categoria dejandola en 0.... por lo tanto oluego no aparece en la lista de productos.....

Nota: modifique un tanto tu ejemplo debido a que contenia un error.... esta editando bien .. gracias...

el codigo es:

Código PHP:

<?php include 'includes/config.php'; ?>
<?
if (isset($_POST['action']) && $_POST['action'] == 'edit')
{
    $id_prod = $_POST['id'];
    $nombre_prod = $_POST['nombre_prod'];
    $descripcion_prod = $_POST['descripcion_prod'];
    $img_prod = $_POST['img_prod'];
    $categoria = $_POST['categoria'];
    $query = "UPDATE producto SET nombre_prod='$nombre_prod', descripcion_prod='$descripcion_prod', img_prod='$img_prod', categoria='$categoria' WHERE id_prod='$id'";
    $result = mysql_query($query) or die(mysql_error());
 
    echo "<div align=center>Producto Actualizado</div>";
 
}
else {    echo "<div align=center>Producto No Actualizado</div>";
}
 
?> 
 
<?php
if ( !empty($_GET['id']) ) {
$query = "SELECT id_prod, nombre_prod, descripcion_prod, img_prod, categoria FROM producto WHERE id_prod = {$_GET['id']} limit 1";
$response = mysql_query($query, $con);
//$row = mysql_fetch_assoc($response);
 
while ($row = mysql_fetch_array($response))                 
    {
        $id_prod=$row[0];
        $nombre_prod= $row[1];
        $descripcion_prod= $row[2];
        $img_prod= $row[3];
        $categoria= $row[4];
    }
 
}
?>
 

Código HTML:

<div id="page-content" class="clearfix">
            
            <h1><?php echo "$nombre_prod" ?></h1>
            <?php
                            $cat=$_GET["id"];
                            $result = mysql_query("SELECT id_cat,cat_nombre FROM categoria") or die("Could not connect: " . mysql_error());  
                            $combo = "<select name=categoria>\n";
                            while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) 
                            {
                                $combo .= " <option value='".$row['id_cat'];
                                $combo .= "'>".$row['cat_nombre']."</option>\n";
                    }
                                $combo .= "</select>\n";
                    
                    ?> 
            <h2>Edición de producto<span> <?php echo "$nombre_prod" ?></span> de Categoría </h2>
            <div class="inner-box clearfix">
            
                <?php include 'sidebar.php'; ?>
            
                <div id="form-block">
                    <form action="editar.php" method="POST" enctype="multipart/form-data">
                    <input type="hidden" name="id" value="<? echo $id; ?>">
                    <input type="hidden" name="action" value="edit">
                    
                    Nombre Producto
                    <input type="text" name="nombre_prod" id="field_1" class="short" value="<?php echo $nombre_prod ?>" /> <br> 
                    Categoría 
                    
                    
                    <?php echo $combo; ?> 
                    
                    
                    
                    
                    
                    Descripción del Producto<br> 
                    <textarea name="descripcion_prod" rows="3" cols="3" id="field_4"><?php echo $descripcion_prod ?></textarea><br> 
                    <input name="archivo" type="file" class="casilla" id="archivo" size="35" /><br> 
                    <input name="enviar" type="submit" class="boton" id="enviar" value="Grabar" />
                    </form>
 
                </div>
            
            </div>
 
            
            
                
        </div>
 

Que error tendré o bien .. com opuedo asocial la categoria... al producto... son dos tablas una categoria y otra productos en la cual esta asociado el campo categoria al campo id de la tabla categorias.

[pilucho]

Hola Luxo

Prueba de esta forma.

Código PHP:

  <?php include 'includes/config.php'; ?> 
<? 
if (isset($_POST['action']))    $action = $_POST['action']; 
if (isset($_POST['id']))        $id = (int)$_POST['id']; 
if (isset($_GET['id']))            $id = (int)$_GET['id']; 
if (isset($_POST['id_cat']))    $id_cat = (int)$_POST['id_cat']; 
if (isset($_GET['id_cat']))        $id_cat = (int)$_GET['id_cat']; 
 
if (isset($_POST['action']) && $_POST['action'] == 'edit') 
{ 
    $id_prod = $_POST['id']; 
    $nombre_prod = $_POST['nombre_prod']; 
    $descripcion_prod = $_POST['descripcion_prod']; 
    $img_prod = $_POST['img_prod']; 
    $categoria = $_POST['categoria']; 
    $query = "UPDATE producto SET nombre_prod='$nombre_prod', descripcion_prod='$descripcion_prod', img_prod='$img_prod', categoria='$categoria' WHERE id_prod='$id'"; 
    $result = mysql_query($query) or die(mysql_error()); 
  
    echo "<div align=center>Producto Actualizado</div>"; 
  
} 
else {    echo "<div align=center>Producto No Actualizado</div>"; 
} 
  
?>  
  
<?php 
if ( !empty($_GET['id']) ) { 
$query = "SELECT id_prod, nombre_prod, descripcion_prod, img_prod, categoria FROM producto WHERE id_prod = {$_GET['id']} limit 1"; 
$response = mysql_query($query, $con); 
//$row = mysql_fetch_assoc($response); 
  
while ($row = mysql_fetch_array($response))                  
    { 
        $id_prod=$row[0]; 
        $nombre_prod= $row[1]; 
        $descripcion_prod= $row[2]; 
        $img_prod= $row[3]; 
        $categoria= $row[4]; 
    } 
  
} 
?>   

Codigo HTML

Código PHP:

 <div id="page-content" class="clearfix"> 
             
            <h1><?php echo "$nombre_prod" ?></h1> 
            <?php 
                            if (isset($_GET['id']))    $id = $_GET['id']; 
                            // $cat=$_GET["id"]; 
                            $result = mysql_query("SELECT id_cat,cat_nombre FROM categoria") or die("Could not connect: " . mysql_error());   
                            $combo = "<select name=categoria>\n"; 
                            while ($row = mysql_fetch_array($result, MYSQL_ASSOC))  
                            { 
                                $combo .= " <option value='".$row['id_cat']; 
                                $combo .= "'>".$row['cat_nombre']."</option>\n"; 
                    } 
                                $combo .= "</select>\n"; 
                     
                    ?>  
            <h2>Edición de producto<span> <?php echo "$nombre_prod" ?></span> de Categoría </h2> 
            <div class="inner-box clearfix"> 
             
                <?php include 'sidebar.php'; ?> 
             
                <div id="form-block"> 
                    <form action="editar.php" method="POST" enctype="multipart/form-data"> 
                    <input type="hidden" name="id" value="<? echo $id; ?>"> 
                    <input type="hidden" name="action" value="edit"> 
                     
                    Nombre Producto 
                    <input type="text" name="nombre_prod" id="field_1" class="short" value="<?php echo $nombre_prod ?>" /> <br>  
                    Categoría  
                     
                     
                    <?php echo $combo; ?>  
                     
                     
                     
                     
                     
                    Descripción del Producto<br>  
                    <textarea name="descripcion_prod" rows="3" cols="3" id="field_4"><?php echo $descripcion_prod ?></textarea><br>  
                    <input name="archivo" type="file" class="casilla" id="archivo" size="35" /><br>  
                    <input name="enviar" type="submit" class="boton" id="enviar" value="Grabar" /> 
                    </form> 
  
                </div> 
             
            </div> 
  
             
             
                 
        </div>  

Me dices como te fue