Cant update multiple fields... =/

[robertoduran]

require('../dbconection.php');
$id_producto = $_POST['id_producto'];
$nombre = $_POST['nombre'];
$desc1 = $_POST['desc1'];
$desc2 = $_POST['desc2'];
$peso = $_POST['peso'];
$costo = $_POST['costo'];
$venta = $_POST['venta'];
$stock = $_POST['stock'];
$imagen = $_POST['imagen'];
$provedor = $_POST['proveedor'];
$id_cat = $_POST['id_cat'];
$cod_fabrica = $_POST['cod_fabrica'];

$mod = mysql_query("UPDATE don_producto SET id_producto = '$id_producto', nombre = '$nombre', desc1 = '$desc1', desc2 = '$desc2', peso = '$peso', costo = '$costo', venta = '$venta', stock = '$stock', imagen = '$imagen', proveedor = '$proveedor', id_cat = '$id_cat', $cod_fabrica = '$cod_fabrica' WHERE id_producto = '$id_producto'");


if ($mod){

echo ("<a href='index.html'>Product  $cod_fabrica  updated </a>");
}else

    echo "error!!";

what is wrong ..??? 😕 😕 😕 😕 it always goes for the ELse and doesnt updte the table..

[leatherback]

you should -certainly when designing- add a 'or die(mysql_error()) after your mysql_query(). this will tell you why the query failed.

Next wopuld be to echo your query, and run it in the database, and see what happens.

Probably you are trying to insert in a field that doesn't exist, or you try to insert a number in a string field'

[robertoduran]

hey thanks for the help.. 🙂

"$cod_fabrica = '$cod_fabrica'"

that was the problem =)

[bradgrafelman]

Note that your script is vulnerable to MySQL injection attacks - you should run all user-supplied data through [man]mysql_real_escape_string/man.

Don't forget to mark this thread resolved.