Сообщаем об ошибках phpBB 3.2.x разработчикам

[Pazh]

В продолжении: интересней использовать версионность не в рамках всего проекта (т.е. одним махом меняем версию на все ресурсы движка и юзер перезаливает все js, css), а версии "по файлам", т.е. у каждого файла своя версия в зависимости например от времени последнего изменения. Вот аналог обсуждается: Symfony2 assets versioning by file

[rxu]

Короче надо исправлять

https://github.com/phpbb/phpbb/pull/5117/files

[Sheer]

Если запретить группу расширений (например Изображения), то если на конференции есть вложение с расширениями из этой группы, например jpg, то при переходе в ACP--СООБЩЕНИЯ--Вложения--Управление вложениями имеем

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

[phpBB Debug] PHP Notice: in file [ROOT]/includes/acp/acp_attachments.php on line 1273: Undefined index: jpg

на каждое вложение с расширением jpg (будут так же gif, если есть вложения с таки расширением и т.д)

Отправлено спустя 18 минут 8 секунд:
Исправление.
Открыть [ROOT]/includes/acp/acp_attachments.php
Найти

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

					$display_cat = $extensions[$row['extension']]['display_cat'];

Заменить на

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

					$display_cat = (!isset($extensions[$row['extension']]['display_cat'])) ? 0 : $extensions[$row['extension']]['display_cat'];

PS. Актуально также и для phpBB3.1

[rxu]

Undefined index: jpg

Попробовал на локалке на 3.2.2, не повторилось.

[Sheer]

А вложения там есть?

[rxu]

Ну конечно. Специально сделал.

Отправлено спустя 24 секунды:
Может надо не запретить, а удалить расширение?

[Sheer]

Может надо не запретить, а удалить расширение?

Нет

images.jpg

att_warning.jpg

Отправлено спустя 7 минут 31 секунду:

не повторилось

А debug включен?

Отправлено спустя 11 минут 44 секунды:
Ага предупреждающее сообщение только при включенном debug

[rxu]

Заменить на

От этого зависит, будет написано Viewed или Downloaded
С данной правкой будет написано что-то одно независимо от типа файла.
Надо подумать.

[Sheer]

И без правки тоже.

view.jpg

Отправлено спустя 2 часа 4 минуты 22 секунды:

будет написано Viewed или Downloaded

Не знаю, насколько это критично, загружено или просмотрено, но тогда так
Найти

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

				$extensions = $cache->obtain_attach_extensions(true);

Добавить после

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

				$viewed_exts = array();
				if (($viewed_exts = $cache->get('_viewed_exts')) === false)
				{
					$sql = 'SELECT group_id
						FROM ' . EXTENSION_GROUPS_TABLE . '
						WHERE group_name = \'IMAGES\'';
					$result = $db->sql_query($sql);
					$group_id = (int) $db->sql_fetchfield('group_id');
					$db->sql_freeresult($result);
					if ($group_id)
					{
						$sql = 'SELECT extension
							FROM ' . EXTENSIONS_TABLE . '
							WHERE group_id = ' . $group_id;
						$result = $db->sql_query($sql);
						while ($row = $db->sql_fetchrow($result))
						{
							$viewed_exts[] = $row['extension'];
						}
						$db->sql_freeresult($result);
						$cache->put('_viewed_exts', $viewed_exts);
					}
				}

Найти

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

					$display_cat = $extensions[$row['extension']]['display_cat'];
					$l_downloaded_viewed = ($display_cat == ATTACHMENT_CATEGORY_NONE) ? 'DOWNLOAD_COUNTS' : 'VIEWED_COUNTS';

Заменить на

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

					$display_cat = (!isset($extensions[$row['extension']]['display_cat'])) ? ATTACHMENT_CATEGORY_NONE : $extensions[$row['extension']]['display_cat'];
					$l_downloaded_viewed = ($display_cat == ATTACHMENT_CATEGORY_NONE && !in_array($row['extension'], $viewed_exts)) ? 'DOWNLOAD_COUNTS' : 'VIEWED_COUNTS';

[rxu]

но тогда так

Может тупо выбрать запросом все display_cat для нужных групп расширений, да и всё.

[Sheer]

Я думаю ваше нафиг не нужно, предыдущей правки вполне достаточно. Какая разница, загружено или просмотрено.
Просто чтоб по феншуйски надо ATTACHMENT_CATEGORY_NONE вместо 0

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

					$display_cat = (!isset($extensions[$row['extension']]['display_cat'])) ? ATTACHMENT_CATEGORY_NONE : $extensions[$row['extension']]['display_cat'];

Отправлено спустя 57 секунд:
Вот неплохо было бы выводить сообщение, что данная группа расширений запрещена.

[photonik]

Если не настроена защита от спам-ботов, спам-боты будут регистрироваться.

в настройках "Средства против спам-ботов" посмотрел, вроде выставлено всё, ну по умолчанию, gd-картинка и прочее.
Регистрация пользователей открыта, так со вчера уже 60 штук зарегилось и тем насоздавали, ладно я выставил лимит для них в 10 сообщений и таким образом мне уведомления приходят, а не сразу постятся в форум их рекламы.
Почему так, сразу легко пробиваются они через защиту? Встроенной защиты недостаточно, какие-то расширения надо ставить?

[Татьяна5]

photonik, потому что картинки любого типа сто лет как пробиты. Читайте форум, тут обо всём писали

[rxu]

Причем тут данная тема? Вам существующих про спам недостаточно, или ЖК захотелось?

[photonik]

потому что картинки любого типа сто лет как пробиты. Читайте форум, тут обо всём писали

понял, спасибо. Просто я как раз из поиска по запросу про спам ботов и вышел в эту тему.