Problema UPDATE

[samyb8]

Hola,


Está bien esta declaración?

Código PHP:

  $result2 = mysql_query ("UPDATE horarios WHERE fecha = '$_POST[fecha]' SET (mincha, shajrit, shiur, minja) VALUES ('$_POST[mincha]', '$_POST[shajrit]', '$_POST[shiur]', '$_POST[minja]')");  
   

Es que no me actualiza el campo que le indico...

Gracias!

[El-FeNiX]

No. .

[miguelension]

yo lo haria asi:

$fecha = $_POST[fecha];
$mincha = $_POST[mincha];
$shajrit = $_POST[shajrit];
$shiur = $_POST[shiur];
$minja = $_POST[minja;



$result2 = mysql_query ("UPDATE horarios SET `mincha` = '$mincha', `shajrit` = '$shajrit', ` shiur` = '$ shiur', `minja` = '$minja' WHERE `fecha` = '$fecha'";

$resultado_update = mysql_query($result2, $conexion);

[Vun]

Indicar VALUES es para indicar los nombres de los campos de la tabla unicamente, y no es necesario, o al menos yo jamas lo uso. El Where va despues de los SET. Y ahi ves como se usa el SET, dando un valor a cada campo. Prueba y me dices si funciona

Código PHP:

  $result2 = mysql_query ("UPDATE horarios WHERE fecha = '$_POST[fecha]' SET  
mincha='$_POST[mincha]', shajrit='$_POST[shajrit]', shiur='$_POST[shiur]',  
minja='$_POST[minja]' WHERE fecha = '$_POST[fecha]' ");  
   

Saludos y buen karma

[El-FeNiX]

Eso es inseguro, tiene un riesgo enorme de consultas fallidas y mysql injection, ¿porque no formateamos los datos antes de mandarlos a la base de datos?

Código PHP:

  $fecha = addslashes(trim($_POST[fecha]));  
   

Con todos.

[miguelension]

para que sirve addslashes(trim())