Пожелания дополнений Event List

[rxu]

Возможно добавить?

Конечно. Только уточним пару моментов.

posting.php

Существующее core.modify_posting_auth не подходит?

Сейчас attach_row в шаблоне невозможно изменить

В принципе возможно: добавлять новые обычным способом, а править старые - через alter_block_array. Но можно и событие воткнуть.

Передать туда этот массив, который потом пойдёт в шаблон

Простые переменные шаблона можно переопределять тем же assign_vars, просто задавая новое значение. Или старые тоже нужны будут?

[Татьяна5]

через alter_block_array

Подходящие события, по переменным, есть только до отправки массива в шаблон, а требуется добавить несколько пар "ключ-значение"

Простые переменные шаблона можно переопределять тем же assign_vars

Это можно, дело удобства

[rxu]

OK.
Все добавления попадут только в 3.2, 3.1 больше не обновляется.

Отправлено спустя 3 минуты 32 секунды:

posting.php
Перед

А если добавить чуть ниже, перед

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

$orig_poll_options_size = sizeof($post_data['poll_options']);

[Татьяна5]

Там можно, код работает

[rxu]

viewforum.php

Какие параметры требуются?

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

posting_attach_body.html
После кода

Именно после спана надо, или можно после законченного элемента какого-то? например, после <td class="attach-name">, или чего-то подобного осмысленного.

[Татьяна5]

Какие параметры требуются?

Мне никаких. Просто выполнить код после цикла понадобилось

Именно после спана надо

Желательно там, результат красивей выходит

[rxu]

Параметры attach_row и массив, который идёт в шаблон

Тут будет немного по-другому, чтобы событие не вызывалось в цикле, но тоже рабочий вариант.

https://github.com/phpbb/phpbb/pull/4905

[rxu]

В ядре.

[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 уведомления, а если они в дальнейшем изменятся/дополнятся...

[rxu]

Pazh, Re: Пожелания дополнений Event List

[Pazh]

rxu, Re: Пожелания дополнений Event List можно и так, но логически - это "костыль", т.к. потребует в том числе повторный вызов notification_manager и add_subscription на стороне расширения. Событие напрашивается именно после определения дефолтных подписок и до вызова notification_manager

[rxu]

потребует в том числе повторный вызов notification_manager и add_subscription на стороне расширения

А в чем костыль-то?

[Pazh]

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

[rxu]

что выдаст большую нагрузку на сервер и более целесообразно - добавление и обработка нового события ядра или повторное дерганье notification_manager

Я думаю, что разница стремится к нулю.

[Pazh]

Если разница нулевая, то удобство и логичность наоборот. Ну это все лирика и как говорится из названия темы - "пожелание"