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

Форум для авторов расширений для phpBB. Здесь можно попросить помощи в разработке у коллег.
Правила форума
Местная Конституция | Шаблон запроса | Документация (phpBB3) | Мини [FAQ] по phpBB3.1.x/3.3.x | FAQ | Как задавать вопросы | Как устанавливать расширения

Ваш вопрос может быть удален без объяснения причин, если на него есть ответы по приведённым ссылкам (а вы рискуете получить предупреждение ;) ).
Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 14.08.2017 7:44

Татьяна5 писал(а):
13.08.2017 23:27
Возможно добавить?
Конечно. Только уточним пару моментов.
Татьяна5 писал(а):
13.08.2017 23:27
posting.php
Существующее core.modify_posting_auth не подходит?
Татьяна5 писал(а):
13.08.2017 23:27
Сейчас attach_row в шаблоне невозможно изменить
В принципе возможно: добавлять новые обычным способом, а править старые - через alter_block_array. Но можно и событие воткнуть.
Татьяна5 писал(а):
13.08.2017 23:27
Передать туда этот массив, который потом пойдёт в шаблон
Простые переменные шаблона можно переопределять тем же assign_vars, просто задавая новое значение. Или старые тоже нужны будут?
Изображение

Аватара пользователя
Татьяна5
Поддержка
Поддержка
Сообщения: 9299
Зарегистрирован: 08.08.2011 2:02
Благодарил (а): 164 раза
Поблагодарили: 2594 раза
Контактная информация:

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

Сообщение Татьяна5 » 14.08.2017 14:19

rxu писал(а):
14.08.2017 7:44
через alter_block_array
Подходящие события, по переменным, есть только до отправки массива в шаблон, а требуется добавить несколько пар "ключ-значение"
rxu писал(а):
14.08.2017 7:44
Простые переменные шаблона можно переопределять тем же assign_vars
Это можно, дело удобства

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 15.08.2017 15:48

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

Отправлено спустя 3 минуты 32 секунды:
Татьяна5 писал(а):
13.08.2017 23:27
posting.php
Перед
А если добавить чуть ниже, перед

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

$orig_poll_options_size = sizeof($post_data['poll_options']);
:?:
Изображение

Аватара пользователя
Татьяна5
Поддержка
Поддержка
Сообщения: 9299
Зарегистрирован: 08.08.2011 2:02
Благодарил (а): 164 раза
Поблагодарили: 2594 раза
Контактная информация:

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

Сообщение Татьяна5 » 15.08.2017 16:07

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

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 15.08.2017 16:51

Татьяна5 писал(а):
13.08.2017 23:27
viewforum.php
Какие параметры требуются?

Отправлено спустя 40 минут 55 секунд:
Татьяна5 писал(а):
13.08.2017 23:27
posting_attach_body.html
После кода
Именно после спана надо, или можно после законченного элемента какого-то? например, после <td class="attach-name">, или чего-то подобного осмысленного.
Изображение

Аватара пользователя
Татьяна5
Поддержка
Поддержка
Сообщения: 9299
Зарегистрирован: 08.08.2011 2:02
Благодарил (а): 164 раза
Поблагодарили: 2594 раза
Контактная информация:

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

Сообщение Татьяна5 » 16.08.2017 12:31

rxu писал(а):
15.08.2017 17:32
Какие параметры требуются?
Мне никаких. Просто выполнить код после цикла понадобилось
rxu писал(а):
15.08.2017 17:32
Именно после спана надо
Желательно там, результат красивей выходит

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 16.08.2017 19:25

Татьяна5 писал(а):
13.08.2017 23:27
Параметры attach_row и массив, который идёт в шаблон
Тут будет немного по-другому, чтобы событие не вызывалось в цикле, но тоже рабочий вариант.

https://github.com/phpbb/phpbb/pull/4905
Изображение

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 07.09.2017 20:42

В ядре.
Изображение

Аватара пользователя
Pazh
Former team member
Сообщения: 2121
Зарегистрирован: 09.11.2009 17:46
Благодарил (а): 37 раз
Поблагодарили: 430 раз
Контактная информация:

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

Сообщение Pazh » 20.11.2017 11:53

Уведомления, есть необходимость при создании юзера включить ему 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 уведомления, а если они в дальнейшем изменятся/дополнятся...
форум ЖК Вестердам Помощь в ЛС/email только за WM или ЯД

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 20.11.2017 15:53

Изображение

Аватара пользователя
Pazh
Former team member
Сообщения: 2121
Зарегистрирован: 09.11.2009 17:46
Благодарил (а): 37 раз
Поблагодарили: 430 раз
Контактная информация:

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

Сообщение Pazh » 21.11.2017 11:55

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

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 21.11.2017 16:17

Pazh писал(а):
21.11.2017 11:55
потребует в том числе повторный вызов notification_manager и add_subscription на стороне расширения
А в чем костыль-то?
Изображение

Аватара пользователя
Pazh
Former team member
Сообщения: 2121
Зарегистрирован: 09.11.2009 17:46
Благодарил (а): 37 раз
Поблагодарили: 430 раз
Контактная информация:

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

Сообщение Pazh » 21.11.2017 16:44

излишние дерганье функционала движка - это уже из области оптимизации: что выдаст большую нагрузку на сервер и более целесообразно - добавление и обработка нового события ядра или повторное дерганье notification_manager и add_subscription внутри расширений.
форум ЖК Вестердам Помощь в ЛС/email только за WM или ЯД

Аватара пользователя
rxu
phpBB Guru
phpBB Guru
Сообщения: 14173
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 334 раза
Поблагодарили: 1456 раз
Контактная информация:

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

Сообщение rxu » 21.11.2017 17:35

Pazh писал(а):
21.11.2017 16:44
что выдаст большую нагрузку на сервер и более целесообразно - добавление и обработка нового события ядра или повторное дерганье notification_manager
Я думаю, что разница стремится к нулю.
Изображение

Аватара пользователя
Pazh
Former team member
Сообщения: 2121
Зарегистрирован: 09.11.2009 17:46
Благодарил (а): 37 раз
Поблагодарили: 430 раз
Контактная информация:

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

Сообщение Pazh » 22.11.2017 12:11

Если разница нулевая, то удобство и логичность наоборот. Ну это все лирика и как говорится из названия темы - "пожелание"
форум ЖК Вестердам Помощь в ЛС/email только за WM или ЯД

Ответить

Вернуться в «Для авторов»