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

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

Ваш вопрос может быть удален без объяснения причин, если на него есть ответы по приведённым ссылкам (а вы рискуете получить предупреждение ;) ).
Аватара пользователя
HAMMER663
Модератор
Модератор
Сообщения: 2031
Зарегистрирован: 17.04.2010 12:59
Откуда: Зеленоград
Благодарил (а): 68 раз
Поблагодарили: 244 раза
Контактная информация:

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

Сообщение HAMMER663 » 24.11.2017 10:28

Поддержу Pazh. Я бы тоже предпочел новое событие ;)

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

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

Сообщение rxu » 24.11.2017 20:58

Я так и не пойму, зачем вам новое.
Если не подходит core.user_add_after, есть core.user_add_modify_data, в котором можно добавить свои данные в $notifications_data и наслаждаться.
Изображение

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

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

Сообщение Pazh » 25.11.2017 14:32

rxu, такое ощущение что Вы или стебетесь, или не удосужились внимательно разобрать функцию user_add на предмет обработки $notifications_data
Как отработает этот код:

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

	// 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 будет добавлено значение в $notifications_data ???
форум ЖК Пироговская ривьера Помощь в ЛС/email только за WM или ЯД

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

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

Сообщение rxu » 25.11.2017 14:35

Зачем ему отрабатывать? Он вам нужен тут что ли?
Если нужен, в том же расширении и добавляете.

Отправлено спустя 4 минуты 49 секунд:
Если коротко, то смысл в том, что если что-то можно сделать без нового события, надо это делать, используя существующие.
Во-первых, лучше обратная совместимость, во-вторых, не надо долго ждать следующего релиза.
Изображение

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

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

Сообщение Pazh » 25.11.2017 14:52

rxu писал(а):
25.11.2017 14:39
Зачем ему отрабатывать? Он вам нужен тут что ли?
это код из движка - ПРЕДУСТАНОВЛЕННЫЙ!!! И если по Вашему совету использовать core.user_add_modify_data, то в результате необходимость отсылать email при подписке на новые темы и посты не будет прописана юзеру при его создании!!! А прописывать этот код в расширении так же не логично, т.к. при следующем релизе он может поменяться, а в расширении останется старым.
Короче бестолковый разговор - надо идти на phpbb.com и там писать свои пожелания...
форум ЖК Пироговская ривьера Помощь в ЛС/email только за WM или ЯД

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

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

Сообщение rxu » 25.11.2017 14:56

А еще лучше, чтобы не тратить бестолково свое красноречие - установите Git и сделайте свой PR.
Изображение

Ответить

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