[OLD] File Attachment Mod - Вложение файлов

[And_Y]

Не помогло

В смысле, файл больше двух мегабайт так и не удаётся закачать?

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

[Alexis1805]

Папка Files пуста. В настройках в админке пробовал всё.

Атрибуты папки какие? Надо установить права на запись 777.

[scrill]

Атрибуты папки какие? Надо установить права на запись 777.

Какие атрибуты? На виндах всё стоит...

[Balamut]

Какие атрибуты? На виндах всё стоит...

на ntfs тоже атрибуты есть. :wink:
(оффтопиком, не обращайте внимания)

[MXM]

'Ya\''ll Must\''ve Forgot'

не отработался в коментарии знак '
?

Шаг за шагом к ошибке:

1. В сообщение сделать вложение файла с комментарием - ya'll must've forgot.
2. Отправить сообщение на форум.
3. Зайти в правку сообщения чтобы загрузить новую версию файла.
4. В сообщении выбрать новую версию файла с комментарием - ya'll must've forgot или другой комментарий содержащий знак '.
5. Напротив файла старой версии нажимаем кнопку – «загрузить новую версию»
6. Ошибка.

Написал об этом на форуме разработчика, жду ответа.

[Siava]

MXM
Подтверждаю, некорректная обработка комментариев с символом '

[Xpert]

MXM
Видел ваше сообщение на opentools.de.
Для таких вещей существует багтрек. Если же вы хотите максимально быстро получить ответ, напишите разработчику в приват ссылку на тему.

[Siava]

Наверно копать надо тут:
posting_attachments.php 628 строка.

Код: Выделить всё

                                                // Update Entry
                                                $sql = "UPDATE " . ATTACHMENTS_DESC_TABLE . "
                                                SET physical_filename = '" . str_replace("'", "''", basename($this->attach_filename)) . "', real_filename = '" . str_replace("'", "''", basename($this->filename)) . "', comment = '" . str_replace("'", "''", $comment) . "', extension = '" . str_replace("'", "''", $this->extension) . "', mimetype = '" . str_replace("'", "''", $this->type) . "', filesize = " . $this->filesize . ", filetime = " . $this->filetime . ", thumbnail = " . $this->thumbnail . "
                                                WHERE attach_id = " . (int) $attachment_id;

Добавлено спустя 2 минуты 11 секунд:

Код: Выделить всё

comment = '" . str_replace("'", "''", $comment) . "'

Хм.. интересно зачем это в комментариях?

[Xpert]

Ага...
Все замены должны быть имхо такого вида:

Код: Выделить всё

str_replace("'", "''",

->

Код: Выделить всё

str_replace("\'", "''",

Код не смотрел, но судя по всему stripslashes автор к данным не применяет.

Добавлено спустя 57 секунд:

Siava
Все верно, это преобразование кавычек к т.н. "стилю sybase".

[Siava]

Xpert
В принципе для исправления хватило замены

Код: Выделить всё

comment = '" . str_replace("'", "''", $comment) . "'

на

Код: Выделить всё

comment = '" . str_replace("\'", "''", $comment) . "'

[Xpert]

Чтобы не было других глюков, лучше поменять все. Можешь проэкспериментировать - попробуй закачать файл, в имени которого есть одинарная кавычка. С моей заменой (imho) удастся, без оной - получим ошибку SQL-запроса.

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

Добавлено спустя 5 минут:

Отправил ЛС автору мода, посмотрим что он скажет.

[Siava]

Xpert
Да, так и есть.

Тогда вот решение проблемы:
заменить

Код: Выделить всё

                                                SET physical_filename = '" . str_replace("'", "''", basename($this->attach_filename)) . "', real_filename = '" . str_replace("'", "''", basename($this->filename)) . "', comment = '" . str_replace("'", "''", $comment) . "', extension = '" . str_replace("'", "''", $this->extension) . "', mimetype = '" . str_replace("'", "''", $this->type) . "', filesize = " . $this->filesize . ", filetime = " . $this->filetime . ", thumbnail = " . $this->thumbnail . "

на

Код: Выделить всё

                                                SET physical_filename = '" . str_replace("\'", "''", basename($this->attach_filename)) . "', real_filename = '" . str_replace("\'", "''", basename($this->filename)) . "', comment = '" . str_replace("\'", "''", $comment) . "', extension = '" . str_replace("\'", "''", $this->extension) . "', mimetype = '" . str_replace("\'", "''", $this->type) . "', filesize = " . $this->filesize . ", filetime = " . $this->filetime . ", thumbnail = " . $this->thumbnail . "

[Xpert]

Т.е. насчет кавычки в имени файла я был прав?
Кстати, если не сложно, посмотри обработку данных при вставке в таблицу, т.е. при первом отправлении вложения. У меня сейчас кода под рукой нет, а любопытство распирает. Выложи сюда, еcли не сложно.
Можно аттачем, у нас теперь такая возможность тоже имеется.

[Siava]

Xpert
Ок, вроде бы оно (строка 698 в том же файле)

Как ни странно, но там почти то же самое =)

[MXM]

MXM
Видел ваше сообщение на opentools.de.
Для таких вещей существует багтрек. Если же вы хотите максимально быстро получить ответ, напишите разработчику в приват ссылку на тему.

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