В чём ошибка SQL-запроса ?

Question

[doublench21]

Был запрос без update поля `short`. Всё работало как надо. Затем нужно было в таблицу добавить поле `short`. Изменил и сам запрос:

$sql = "UPDATE indexing_link SET title='" . $title . "', short='" . $short . "' WHERE id=" . $id_current_url;

Все переменные были проверены перед запросом. Всё нормально. Тот же самый запрос из phpmyadmin работает. А со скрипта выдаёт ошибку. В чём дело ?

$sql = "UPDATE indexing_link SET title='" . $title . "', short='" . $short . "' WHERE id=" . $id_current_url;
mysqli_query($mysqli, $sql) or die('Error_ (' . mysqli_connect_errno() . ') ' . mysqli_connect_error());

Error_ (0)
Так работает:

$sql = "UPDATE indexing_link SET title='" . $title . "' WHERE id=" . $id_current_url;

А так нет:

$sql = "UPDATE indexing_link SET title='" . $title . "', short='" . $short . "' WHERE id=" . $id_current_url;

Comments

[Александр Зеленин]

А текст ошибки будет?

[doublench21]

@Zav добавил. Ток не думаю что поможет

[# a r t u r #]

А что в $mysqli?

[doublench21]

@passshift

$mysqli = mysqli_connect("localhost", "root", "",
    "u360508016_base") or die('Error (' . mysqli_connect_errno() . ') ' . mysqli_connect_error());
mysqli_set_charset($mysqli, "utf8");

[Александр Зеленин]

А с чего ты взял что там connect_error? Может там ошибка запроса. Её и выводи. Убери вот это "or die ..." и глянь на чем валится.

[Александр Зеленин]

Т.е. просто выполни mysqli_query($mysqli, $sql);

[doublench21]

@Zav в том та и дело, ничего не выводит она.

[doublench21]

@Zav номер ошибки та вывел, ток он не говорит ни о чем.

[doublench21]

@Zav Гляньте вопрос, обновил маленько.

[Александр Зеленин]

Смотри PHP конфиг. Настрой вывод ошибок.

Answer 1

[Евгений Самойленко]

Вроде запрос написан правильно ,в таблицу добавили же это поле "short"?

Comments

[doublench21]

Ага

[Евгений Самойленко]

@doublench21 после того, как вы написали ошибку стало понятно - плохо экранируете short вот и все

[doublench21]

@samoilenkoevgeniy видимо я слепой. покажите если не трудно ?

[doublench21]

@samoilenkoevgeniy сам текст перед запросом:
string 'From Minecraft Wiki Jump to: navigation, search “ I like achievements. I know a lot of people don't, but I like them. I've had the idea to make achievem...' (length=157)

[doublench21]

@samoilenkoevgeniy разобрался.

Answer 2

[Rsa97]

Сделайте вывод сформированного запроса. После mysqli_query смотрите не mysqli_connect_error(), а mysqli_error().

$sql = "UPDATE indexing_link SET title='".$title."', short='".$short ."' WHERE id=".$id_current_url;
print $sql;
mysqli_query($mysqli, $sql) or die('Error_ ('.mysqli_errno($mysqli).') '.mysqli_error($mysqli));

И покажите ещё структуру самой таблицы, скажем через SHOW CREATE TABLE `indexing_link`

Comments

[doublench21]

Блин. Чёрт попутал после коннекта писал везде mysqli_connect_error() вместо mysqli_error($mysqli). Так теперь такое выдает: Error_ (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 't, but I like them. I've had the idea to make achievem...' WHERE id=339' at line 1

[Rsa97]

Всё ясно. у Вас $short содержит в себе одинарную кавычку. Надо как минимум делать real_escape_string(), а лучше использовать параметризованные запросы:

$sql = mysqli_prepare($mysqli, "UPDATE `indexing_link` SET `title`=?, `short`=? WHERE `id`=?");
mysqli_stmt_bind_param($sql, "ssi", $title, $short, $id_current_url);
mysqli_stmt_execute($sql) or die('Error_ ('.mysqli_errno($mysqli).') '.mysqli_error($mysqli));