[SOLUCIONADO] #1064 - You have an error in your SQL syntax

[Jose789]

Hola buenas gente, pues tengo un problema a la hora de hacer un DELETE.

#1064 - 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 ' product_id IN ('126,127')' at line 1

Recojo los datos del check de unos checkbox
y el userid de un campo oculto con valor al id del correspondiente usuario.

Código PHP:

  $userid = $_POST['userid'];
$check = $_POST['seleccion'];
$ids = implode(',', $check);
 
mysql_query("DELETE FROM user_badges WHERE user_id='".$userid."', product_id IN ('".$ids."')") or die(mysql_error()); 
   

Si me podrían echar una mano, se lo agradecería.

[GatorV]

Digo fuera de que tienes un grave problema propenso de SQL inyection, el problema es que te falta un AND en tus condiciones para borrar, ejemplo:

Código SQL:

DELETE FROM `foo` WHERE `bar`='baz' AND `bat`=`bar`
 

[gnzsoloyo]

Adicionalmente a eso, hay un error de construcción que se percibe en el mensaje de error.
Cuando poner un IN(), los valores deben quedar separados por comas, pero no de esta forma:

Código SQL:

DELETE FROM user_badges 
WHERE user_id='23' AND product_id IN ('126, 127')
 

Esa sintaxis asume que PRODUCT_ID es un campo VARCHAR que contendrá valores como "126,127", es decir una cadena de texto con los dos números con una coma intermedia, y francamente no creo que sea el caso.

Quítale los apóstrofos...

Consejo: Los valores números no necesitan apóstrofos cuando los pones en SQL, a menos que sea para comparar contra campos VARCHAR.

La sintaxis correcta asía:

Código PHP:

mysql_query("DELETE FROM user_badges WHERE user_id = ".$userid." AND product_id IN (".$ids.")") or die(mysql_error());
 

que podrá dar un resultado mas o menos así (es un ejemplo imaginario):

Código SQL:

DELETE FROM user_badges 
WHERE user_id = 23 AND product_id IN (126, 127)
 

Eso podrá hacer que si un valor no está llegando en una de as variables a la query se dispare un error de sintaxis, pero ese error te puede indicar precisamente que hay una variable vacía... Lo que es bueno para depurar.

[Jose789]

Si así es le faltaba el AND

Asi fue como quedo y por el momento funciona de maravilla. muchas gracias a los 2

Código PHP:

  $check = $_POST['seleccion'];
            foreach ($check as $value){
                mysql_query("DELETE FROM user_badges WHERE user_id= '".$userid."' AND badge_id='".$value."'") or die(mysql_error());
            }