Расширения - это просто!

[rxu]

global сейчас не поощряется, видимо, по соображениям ресурсоемкости, ну и архитектурно.
Везде, где можно, используются зависимости и сервисы.

[Shredder]

Нужно в файл message_parser к стандартным проверкам содержимого текста добавить ещё одну, свою. Например, после этой:

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

		// Minimum message length check for post only
		if ($mode === 'post')
		{
			if (!$message_length || $message_length < (int) $config['min_post_chars'])
			{
				$this->warn_msg[] = (!$message_length) ? $user->lang['TOO_FEW_CHARS'] : ($user->lang('CHARS_POST_CONTAINS', $message_length) . '<br />' . $user->lang('TOO_FEW_CHARS_LIMIT', (int) $config['min_post_chars']));
				return (!$update_this_message) ? $return_message : $this->warn_msg;
			}
		}

Как я понимаю, нужных событий нету?

[rxu]

нужных событий нету?

https://github.com/phpbb/phpbb/pull/3114
Ждем-с.

[Shredder]

rxu
Вижу, уже что-то добавили:
https://github.com/rxu/phpbb3/commit/89 ... 8702010c40

А ошибку там свою можно будет задать? Как в примере выше, чтобы при определённом условии выводилась языковая строка.

[rxu]

Вижу, уже что-то добавили:

Еще нет, только PR.

ошибку там свою можно будет задать

Думаю, что да.

[Shredder]

А как можно задать, чтобы расширение не включалось на phpBB ниже 3.1.2? Или так сделать нельзя?

[rxu]

composer.json

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

        "soft-require": {
           "phpbb/phpbb": ">=3.1.2"
        },

[Shredder]

Я так и сделал, но оно всё равно включается.
Можно, конечно, сделать эту проверку в администраторском модуле, чтобы там выдавалось предупреждение, но всё же.

[rxu]

Shredder
OK, тогда в ext.php добавить функцию is_enableable() с телом вроде

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

public function is_enableable()
{
	$version_helper = $this->container->get('version_helper');
	return $version_helper->compare($version_helper->current, '3.1.2', '>=');
}

[Webliberty]

Есть два расширения и оба они используют одно и тоже php событие, например, core.viewtopic_get_post_data. Но событию сопоставлены разные функции:

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

'core.viewtopic_get_post_data' => 'data1',
'core.viewtopic_get_post_data' => 'data2',

Будут ли в таком случае работать оба расширения или необходимо использовать разные события?

[rxu]

seodirection
Тут всё просто. Функция static public function getSubscribedEvents() возвращает ассоциативный массив, в котором ключ - имя события, значение - его подписчик. Таким образом, у вас не получится задать более одного подписчика с одним и тем же ключом, т.е. именем события. Если вы сначала сделаете 'core.viewtopic_get_post_data' => 'data1', и потом 'core.viewtopic_get_post_data' => 'data2', - то последнее значение перезапишет предыдущее и выполнится подписчик, указанный последним.
Т.е. здесь ничего особенного нет, обычная работа с массивами PHP.

[Shredder]

Есть два расширения и оба они используют одно и тоже php событие

Будут. И думаю, это не сложно проверить. Если случится конфликт, то можно использовать приоритет: Карта сайта для ботов (phpBB3 SEO Sitemap)
Приоритет делается обычной цифрой:

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

			'core.viewtopic_get_post_data'			=> array('viewtopic_handle_thanks', -2),

Если у первого расширения он меньше, чем у второго, то код первого выполняется позднее.

rxu
Он про разные расширения говорит. Либо я что-то не так понял.

[Webliberty]

Так и получилось у меня, расширение которое включил вторым работает, зато первое перестало, хотя до написания второго работало) Поэтому объединил функции в одно расширение и заработало.

[rxu]

seodirection
И в разных будут, надо определить приоритет, как указал выше Shredder.

[Webliberty]

Да, говорю про два разных расширения, когда они по отдельности используют одно событие - не работает.

Shredder, т.е. если задать приоритеты то все должно получиться?

Добавлено спустя 44 секунды:
rxu, Shredder, спасибо, теперь понял. Задам приоритеты)