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

Обсуждение новой системы расширения функциональности phpBB 4.0-dev, разработки новых расширений.
Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

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

Сообщение Алг »

rxu писал(а):QR_BBPOST По-моему, это путаница - иметь 2 разных элемента для одного и того же действия.
Тут на форуме такая путаница уже имеет место. Если вы зайдёте на страницу формирования ЛС, то у вас будет выбор, каким путём заполнить textarea, или через ссылку "найти пользователя" или через поле "быстрый поиск". Одно другому никак не мешает

Отправлено спустя 3 минуты 32 секунды:
rxu писал(а):QR_BBPOST События шаблона парные, надо понять, где помещать пары
а Ваш предыдуший вопрос был - нужны ли парные события и объяснить почему. Мой ответ - достаточно одного, парные необязательно. Но не откажусь
чем больше событий, тем удобнее полключаться естествееным путём, а не городя горы джаваскпипт кода

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

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

Сообщение rxu »

Я чувствую, что мы так и не решим данный вопрос :)

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

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

Сообщение Алг »

решим, но мест не одно. Если хотите, я дам по одному событию для форумов и для юзеров, как пример сделаем. А другой вариант - сразу собрать все места, ещё и группы туда же до кучи. У меня, например, такая специфика, что по 20-30 групп имеют одинаковое начало. очень неудобно искать через весь большущий список

Аватара пользователя
HAMMER663
Сообщения: 123
Зарегистрирован: 22 янв 2014, 09:23
Откуда: Москва
Github repo: https://github.com/HAMMER663

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

Сообщение HAMMER663 »

Необходимо событие в файле includes/message_parser.php
перед

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

		// Prepare BBcode (just prepares some tags for better parsing)
		if ($allow_bbcode && strpos($this->message, '[') !== false)
Что-то вроде такого:

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

		/**
		* Event to modify the text after it is parsed
		*
		* @event core.modify_format_display_text_before
		* @var string	message				The message text to parse	
		* @var bool		allow_bbcode		Do we allow bbcodes
		* @var bool		allow_magic_url		Do we allow magic urls
		* @var bool		allow_smilies		Do we allow smilies
		* @var bool		update_this_message	Do we update the internal message
		*									with the parsed result
		* @since 3.1.0-RC6
		*/
		$message = $this->message;
		$vars = array('message', 'allow_bbcode', 'allow_magic_url', 'allow_smilies', 'update_this_message');
		extract($phpbb_dispatcher->trigger_event('core.modify_format_display_text_before', compact($vars)));
		$this->message = $message;

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

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

Сообщение rxu »

HAMMER663, тут нужно будет как-то обосновать. Для чего конкретно это используется? После данного места еще происходит парсинг смайлов и ббкода.

Аватара пользователя
HAMMER663
Сообщения: 123
Зарегистрирован: 22 янв 2014, 09:23
Откуда: Москва
Github repo: https://github.com/HAMMER663

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

Сообщение HAMMER663 »

rxu, идея была в преобразовании URL, как в моде Convert Local Url To Link Name. Но с событием core.modify_format_display_text_after ничего не получилось. Видимо потому что там текст уже обработан

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

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

Сообщение rxu »

HAMMER663, так может не в самом парсере, а в function format_display() скажем, перед

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

if ($this->message_status == 'plain')
:?:

Аватара пользователя
HAMMER663
Сообщения: 123
Зарегистрирован: 22 янв 2014, 09:23
Откуда: Москва
Github repo: https://github.com/HAMMER663

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

Сообщение HAMMER663 »

Тогда надо как-то событие переделывать

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

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

Сообщение rxu »

Оно один в один как раз так и встанет. Может, еще надо будет добавить $tmp_message и $return_message, но это опционально.

Аватара пользователя
HAMMER663
Сообщения: 123
Зарегистрирован: 22 янв 2014, 09:23
Откуда: Москва
Github repo: https://github.com/HAMMER663

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

Сообщение HAMMER663 »

Только что-то тогда преобразование URL не работает..

Ответить