Ошибки и исправления в версии 3.2.x

[Gubkin]

я не хочу чтоб подвыпивший пользователь взял и удалил свои вложения

[Mr. Anderson]

Gubkin, не хотите? Отключите.

[Gubkin]

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

2018-02-07_181220.jpg

Отправлено спустя 15 секунд:

Отключите

как ? права смотрел, не нашел такого

[Татьяна5]

Отключите модуль в личном разделе

[Mr. Anderson]

Gubkin, ваше "может стоить" ничем не отличается от "а если бы он нёс патроны?". Вам надо? Ваши данные ценны? Вам их и защищать, логично?
Универсальных преднастроек нет, а запретить всё - это уже какое-то анальное огорожение.

[Siava]

По логике, если запрещено редактировать сообщение, то и вложение из него надо запретить удалять.

[Gubkin]

Вам их и защищать, логично?

логично то оно логично, но кто ж знал что есть такой подарок обезумевшему пользователю
ещё раз, если есть защита от редактирования/удаления поста, то почему с вложениями иначе
надо теперь прочесать все настройки ... вдруг есть кнопка "удалить все свои сообщения"

то и вложение из него надо запретить удалять

спасибо! меня услышали, я то как был удивлен узнав что это не так ))

[Mr. Anderson]

Siava, позволю себе не согласиться. Сообщения и вложения - идеологически разные вещи. Поэтому для вложений и управления ими ЕСТЬ свой модуль (и "файлопомойники" с прочими "релизеры" осанну этой фиче поют), а для сообщений - нет.

[Siava]

Сообщения и вложения - идеологически разные вещи.

Но в подавляющем большинстве случаев одно связанно с другим.
Удалил вложения и сообщения, где они упоминались, потеряли смысл.

И расширением это безобразие не исправить, только код править.
У себя сделаю следующее, что и в двойке применял. Оставлю только админам такую возможность.
Подходит одинаково для 3.2 и 3.1.
Да не сожгут меня за эту ересь

styles/prosilver/template/ucp_attachments.html
найти

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

<dd class="mark">{L_MARK}</dd>

заменить на

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

<!-- IF U_ACP --><dd class="mark">{L_MARK}</dd><!-- ENDIF -->

найти

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

<dd class="mark"><input type="checkbox" name="attachment[{attachrow.ATTACH_ID}]" value="1" /></dd>

заменить на

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

<!-- IF U_ACP --><dd class="mark"><input type="checkbox" name="attachment[{attachrow.ATTACH_ID}]" value="1" /></dd><!-- ENDIF -->

includes/ucp/ucp_attachments.php
найти

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

global $template, $user, $db, $config, $phpEx, $phpbb_root_path, $phpbb_container;

заменить на

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

global $auth, $template, $user, $db, $config, $phpEx, $phpbb_root_path, $phpbb_container;

найти

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

$template->assign_var('S_ATTACHMENT_ROWS', true);

заменить на

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

if ($auth->acl_get('a_'))
{	
	$template->assign_var('S_ATTACHMENT_ROWS', true);
}

[rxu]

Этот вопрос давным давно задавался разрабам. Логика описана выше.
Посты постами, файлы файлами.

Отправлено спустя 3 минуты 9 секунд:
Думаю, можно решить, если только добавить новое право - Может удалять собственные вложения.

[Mr. Anderson]

только код править

Зачем умножать сущности?
Убрать модуль и всего делов.
Ну или глобальнее, как Руслан указал, - правом.

[KEMnEP]

3. Предупреждение при отправке по e-mail уведомления об ответе или цитировании

из первого поста в том виде как сейчас может приводить к ошибке 500 ибо из коробки в файле functions_messenger.php

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

var $msg, $extra_headers, $replyto, $from, $subject;
var $addresses = array();

если следовать инструкции как сейчас то получаем

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

var $msg, $extra_headers, $replyto, $from, $subject;
var $addresses = array();
var $extra_headers = array();

Что в итоге может привести к PHP Fatal error: Cannot redeclare messenger::$extra_headers
правильней в инструкции не добавить после, а:
найти

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

var $msg, $extra_headers, $replyto, $from, $subject;
var $addresses = array();

заменить

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

var $msg, $replyto, $from, $subject;
var $addresses = array();
var $extra_headers = array();

[SStone]

Эту проблему можно словить только на PHP 7.2?

[KEMnEP]

и на 7.1 может быть актуальна (причём тут вообще версия пхп...)

[Sheer]

KEMnEP, Понял, не обратил внимания, в официальном фиксе так и сделано https://github.com/phpbb/phpbb/blob/3.2 ... hp#L25-L30
Исправил инструкцию в первом сообщении.