Страница 12 из 14
Re: Пожелания дополнений Event List
Добавлено: 14.08.2017 7:44
rxu
Татьяна5 писал(а): ↑13.08.2017 23:27Возможно добавить?
Конечно. Только уточним пару моментов.
Татьяна5 писал(а): ↑13.08.2017 23:27posting.php
Существующее
core.modify_posting_auth
не подходит?
Татьяна5 писал(а): ↑13.08.2017 23:27Сейчас attach_row в шаблоне невозможно изменить
В принципе возможно: добавлять новые обычным способом, а править старые - через
alter_block_array
. Но можно и событие воткнуть.
Татьяна5 писал(а): ↑13.08.2017 23:27Передать туда этот массив, который потом пойдёт в шаблон
Простые переменные шаблона можно переопределять тем же
assign_vars
, просто задавая новое значение. Или старые тоже нужны будут?
Re: Пожелания дополнений Event List
Добавлено: 14.08.2017 14:19
Татьяна5
rxu писал(а): ↑14.08.2017 7:44через
alter_block_array
Подходящие события, по переменным, есть только до отправки массива в шаблон, а требуется добавить несколько пар "ключ-значение"
rxu писал(а): ↑14.08.2017 7:44Простые переменные шаблона можно переопределять тем же
assign_vars
Это можно, дело удобства
Re: Пожелания дополнений Event List
Добавлено: 15.08.2017 15:48
rxu
OK.
Все добавления попадут только в 3.2, 3.1 больше не обновляется.
Отправлено спустя 3 минуты 32 секунды:
Татьяна5 писал(а): ↑13.08.2017 23:27posting.php
Перед
А если добавить чуть ниже, перед
Код: Выделить всё
$orig_poll_options_size = sizeof($post_data['poll_options']);
Re: Пожелания дополнений Event List
Добавлено: 15.08.2017 16:07
Татьяна5
Там можно, код работает
Re: Пожелания дополнений Event List
Добавлено: 15.08.2017 16:51
rxu
Татьяна5 писал(а): ↑13.08.2017 23:27viewforum.php
Какие параметры требуются?
Отправлено спустя 40 минут 55 секунд:
Татьяна5 писал(а): ↑13.08.2017 23:27posting_attach_body.html
После кода
Именно после спана надо, или можно после законченного элемента какого-то? например, после
<td class="attach-name">
, или чего-то подобного осмысленного.
Re: Пожелания дополнений Event List
Добавлено: 16.08.2017 12:31
Татьяна5
rxu писал(а): ↑15.08.2017 17:32Какие параметры требуются?
Мне никаких. Просто выполнить код после цикла понадобилось
rxu писал(а): ↑15.08.2017 17:32Именно после спана надо
Желательно там, результат красивей выходит
Re: Пожелания дополнений Event List
Добавлено: 16.08.2017 19:25
rxu
Татьяна5 писал(а): ↑13.08.2017 23:27Параметры attach_row и массив, который идёт в шаблон
Тут будет немного по-другому, чтобы событие не вызывалось в цикле, но тоже рабочий вариант.
https://github.com/phpbb/phpbb/pull/4905
Re: Пожелания дополнений Event List
Добавлено: 07.09.2017 20:42
rxu
В ядре.
Re: Пожелания дополнений Event List
Добавлено: 20.11.2017 11:53
Pazh
Уведомления, есть необходимость при создании юзера включить ему email оповещение на какое-то уведомление. В файле
includes/functions_user.php
в функции
user_add
есть два события:
core.user_add_modify_data
и
core.user_add_after
. Первое позволяет исправить/дополнить
$notifications_data
, но ниже по коду идет проверка и добавление default значений email оповещений (строка 364).
Логичней было добавить событие для исправления/дополнения
$notifications_data
после прописывания default уведомлений, но до их записи в БД, т.е. после
Код: Выделить всё
// Use default notifications settings if notifications_data is not set
if ($notifications_data === null)
{
$notifications_data = array(
array(
'item_type' => 'notification.type.post',
'method' => 'notification.method.email',
),
array(
'item_type' => 'notification.type.topic',
'method' => 'notification.method.email',
),
);
}
,
в противном случае приходится использовать событие
core.user_add_modify_data
и прописывать в расширении дефолтные email уведомления, а если они в дальнейшем изменятся/дополнятся...
Re: Пожелания дополнений Event List
Добавлено: 20.11.2017 15:53
rxu
Re: Пожелания дополнений Event List
Добавлено: 21.11.2017 11:55
Pazh
rxu,
Re: Пожелания дополнений Event List можно и так, но логически - это "костыль", т.к. потребует в том числе повторный вызов
notification_manager
и
add_subscription
на стороне расширения. Событие напрашивается именно после определения дефолтных подписок и до вызова
notification_manager
Re: Пожелания дополнений Event List
Добавлено: 21.11.2017 16:17
rxu
Pazh писал(а): ↑21.11.2017 11:55потребует в том числе повторный вызов notification_manager и add_subscription на стороне расширения
А в чем костыль-то?
Re: Пожелания дополнений Event List
Добавлено: 21.11.2017 16:44
Pazh
излишние дерганье функционала движка - это уже из области оптимизации: что выдаст большую нагрузку на сервер и более целесообразно - добавление и обработка нового события ядра или повторное дерганье notification_manager и add_subscription внутри расширений.
Re: Пожелания дополнений Event List
Добавлено: 21.11.2017 17:35
rxu
Pazh писал(а): ↑21.11.2017 16:44 что выдаст большую нагрузку на сервер и более целесообразно - добавление и обработка нового события ядра или повторное дерганье notification_manager
Я думаю, что разница стремится к нулю.
Re: Пожелания дополнений Event List
Добавлено: 22.11.2017 12:11
Pazh
Если разница нулевая, то удобство и логичность наоборот. Ну это все лирика и как говорится из названия темы - "пожелание"