Ошибка синтаксиса MySQL

Question

$sql = "UPDATE galleries SET name='$name', desc='$desc', mainthumb='$mt' 
     WHERE id='$id'"; 

это порождает ошибку по какой-то божественной причине. Должно быть, я слишком устал, потому что не вижу этого.

Я подтвердил, что все значения публикуются. Что еще хуже, это почти точная копия любого запроса, который отлично работает.

Update:

Это была решена. Это был факт, что desc не имел обратных ссылок. Я также собираюсь использовать PDO, как было предложено.

Answer 1

Не desc не ключевое слово, которое нельзя использовать в качестве имени столбца?

Answer 2

echo $sql и посмотреть, что это на самом деле становится. Это похоже на легкую цель для SQL-инъекций, если вы не позаботились об этом.

Answer 3

У вас есть столбец desc, который является reserved word. Вам нужно будет процитировать его с обратными выводами.

`desc`='$desc' 

Answer 4

да, убедитесь, что вы первый дезинфицировать данные, используя mysql_real_escape_string, например.

Затем повторите ошибку mysql (mysql_error()), это даст вам больше подсказок о том, где ошибка;

<?php 
$link = mysql_connect("localhost", "mysql_user", "mysql_password"); 

mysql_select_db("nonexistentdb", $link); 
echo mysql_errno($link) . ": " . mysql_error($link). "\n"; 

mysql_select_db("kossu", $link); 
mysql_query("SELECT * FROM nonexistenttable", $link); 
echo mysql_errno($link) . ": " . mysql_error($link) . "\n"; 
?> 

Answer 5

Вы использовали sanitize all the parameters перед смешиванием их с оператором sql?
desc является reserved word in MySQL, вы должны явно mark it as an identifier:

Идентификатор может быть кавычках или без. Если идентификатор содержит специальные символы или зарезервированное слово, вы должны указывать его всякий раз, когда вы ссылаетесь на него. [...]
Идентификатор кавычка является кавычка («`»):

$mysql = mysql_connect(... 

$sql = " 
    UPDATE 
     galleries 
    SET 
     name='" . mysql_real_escape_string($_POST['name'], $mysql) . "', 
     `desc`='" . mysql_real_escape_string($_POST['desc'], $mysql) . "', 
     mainthumb='" . mysql_real_escape_string($_POST['mt'], $mysql) . "' 
    WHERE 
    id='" . mysql_real_escape_string($_POST['id'], $mysql) . "' 
"; 

или даже лучше: use prepared statements

Answer 6

$sql = "UPDATE `galleries` SET 
      name='".$name."', 
      desc='".$desc."', 
      mainthumb='".$mt."' 
     WHERE id='".$id."'"; 

Это может быть один альтернативный способ справиться с этим. Хотя я бы пошел PDO, как сказал VolkerK. Я также хотел бы Эхо посмотреть, что он будет выводить. Также, как предположил Бен, Desc может быть резервным словом.