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

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

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

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

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

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

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

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

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

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

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

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
Сообщения: 14034
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 326 раз
Поблагодарили: 1418 раз
Контактная информация:

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

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

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

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

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

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
Сообщения: 14034
Зарегистрирован: 12.05.2006 18:16
Откуда: Красноярск
Благодарил (а): 326 раз
Поблагодарили: 1418 раз
Контактная информация:

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

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

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

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

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

Сообщение rxu » 18.12.2017 20:11

Pazh писал(а):
20.11.2017 11:53
добавить событие для исправления/дополнения $notifications_data
HAMMER663 писал(а):
24.11.2017 10:28
Я бы тоже предпочел новое событие
https://github.com/phpbb/phpbb/pull/5064
Изображение

Аватара пользователя
Sheer
phpBB Guru
phpBB Guru
Сообщения: 11027
Зарегистрирован: 18.02.2007 19:01
Откуда: Рига, Латвия (страна-недоразумение)
Благодарил (а): 43 раза
Поблагодарили: 2398 раз
Контактная информация:

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

Сообщение Sheer » 19.12.2018 17:02

Нужно событие шаблона в posting_attach_body.html перед

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

	<div class="panel<!-- IF not .attach_row --> hidden<!-- ENDIF --> file-list-container" id="file-list-container">
Изображение
Общие ошибки новичков (07.11.2005) & Как задавать вопросы
Если ничто другое не помогает, прочтите, наконец, инструкцию!
"Никакая инструкция не может перечислить всех обязанностей должностного лица, предусмотреть все отдельные случаи и дать вперёд соответствующие указания, а поэтому господа инженеры должны проявить инициативу и, руководствуясь знаниями своей специальности и пользой дела, принять все усилия для оправдания своего назначения".
Циркуляр Морского технического комитета №15 от 29.11.1910 г.

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

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

Сообщение rxu » 19.12.2018 21:23

Sheer писал(а):
19.12.2018 17:02
Нужно событие шаблона в posting_attach_body.html
https://github.com/phpbb/phpbb/pull/5484
Изображение

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

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

Сообщение rxu » 13.04.2019 8:09

Не прошло и полгода. В 3.2.6 будет.
Изображение

Аватара пользователя
Sheer
phpBB Guru
phpBB Guru
Сообщения: 11027
Зарегистрирован: 18.02.2007 19:01
Откуда: Рига, Латвия (страна-недоразумение)
Благодарил (а): 43 раза
Поблагодарили: 2398 раз
Контактная информация:

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

Сообщение Sheer » 13.04.2019 8:21

Супер! Только я уже забыл, зачем оно мне понадобилось.
Изображение
Общие ошибки новичков (07.11.2005) & Как задавать вопросы
Если ничто другое не помогает, прочтите, наконец, инструкцию!
"Никакая инструкция не может перечислить всех обязанностей должностного лица, предусмотреть все отдельные случаи и дать вперёд соответствующие указания, а поэтому господа инженеры должны проявить инициативу и, руководствуясь знаниями своей специальности и пользой дела, принять все усилия для оправдания своего назначения".
Циркуляр Морского технического комитета №15 от 29.11.1910 г.

Ответить

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