[3.1] RH Topic Tags

[romaamor]

Название расширения: RH Topic Tags
Автор: combuster
Описание расширения: Расширение добавляет возможность помечать темы с ключевыми словами.
Добавлять ключевые слова можно только при размещении новой темы или при редактировании первого сообщения.
Так же есть облако тегов, белый и чёрный списки тегов.
Скачать: https://www.phpbb.com/customise/db/exte ... opic_tags/
Репозиторий на GitHub https://github.com/RobertHeim/phpbb-ext-topictags
Язык: EN RU ES

Установка:
Скопировать содержимое архива в ext/robertheim/topictags/

Перейти в ACP -> вкладка Настройка -> Управление расширениями -> Включить

Так же надо перейти во вкладку прав доступа, права администратора, разное, Может редактировать RH topic tags -да.

По умолчанию расширение не понимает буквы русского языка. Заходим в админку, настройка расширений, RH TOPIC TAGS, настройки, Настройки тегов и в поле - Регулярное выражение для разрешенных тегов: ставим это :

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

/^[\- a-zячсмитьбюэждлорпавыфйцукенгшщзхъёЯЧСМИТЬБЮЭЖДЛОРПАВЫФЙЦУКЕНГШЩЗХЁ0-9+]{3,30}$/i

Можете там же добавить и свои знаки.
 
В расширении есть баг - ссылки на русские теги отображаются вот так:
http://forum.ru/tag/%25D0%259F%25D0%25B5%25D1%2580%25D0%25B2%25D1%258B%25D0%25B9
Это происходит из-за двойного URL-кодирования (urlencode). Если хочется, чтобы ссылки отображались по-человечески:
http://forum.ru/tag/Первый
- нужно внести правки в три файла:

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

ext\robertheim\topictags\event\main_listener.php
найти
					'tags'	=> urlencode($tag),
заменить на
					'tags'	=> $tag,
ext\robertheim\topictags\controller\main.php
найти
				'tags'	=> urlencode($tags_string),
заменить на
				'tags'	=> $tags_string,
ext\robertheim\topictags\service\tagcloud_manager.php
найти
						'tags'	=> urlencode($tag['tag'])
заменить на
						'tags'	=> $tag['tag']

[southklad]

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

<title><!-- IF UNREAD_NOTIFICATIONS_COUNT -->({UNREAD_NOTIFICATIONS_COUNT}) <!-- ENDIF --><!-- IF S_IN_MCP -->{L_MCP} - <!-- ELSEIF S_IN_UCP -->{L_UCP} - <!-- ENDIF -->{PAGE_TITLE}<!-- IF not S_VIEWTOPIC and not S_VIEWFORUM --> - {SITENAME}<!-- ENDIF --><!-- IF S_VIEWTOPIC or S_VIEWFORUM --> - {SITENAME}<!-- ENDIF --></title>

У меня вот так и темы идет норм, название темы и потом название форума, но почему то в расширении наоборот

[dimassamid]

А вы сейчас это поменяли или раньше? Может кэш?

По идее всё правильно, {SITENAME} стоит после {PAGE_TITLE}. У меня всё то же самое и всё работает, в том числе и в расширении.

[southklad]

поменяли или раньше?

У меня так изначально было в файлах стиля

[manager42]

Не могли бы скинуть архив этого расширения с адаптацией под phpbb 3.3.9?
После установки расширения, в его настройках недоступны разделы «Белый список» и «Черный список» – выдаёт белый экран. И само расширение в принципе не работает – ломает форумы: белый экран при редактировании сообщения или при создании новой темы.

Сделал правки рекомендованные dimassamid, но не помогло.

Если что, стиль родной (prosilver), я его скопировал (для мелких правок) и назвал как mystyle-prosilver. В папке расширения я перенёс jss и css, сделал правки в файлах, и на всякий ещё в расширении скопировал целиков папку prosilver, назвав её mystyle-prosilver (как название папки действующего стиля). И всё равно не помогло, не заработало.

[rxu]

manager42, что в логе ошибок?

[southklad]

В версии 3.3.8 тоже ошибка в списках, только увидел

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

Error: Fatal error: Uncaught Twig\Error\LoaderError: Looks like you try to load a template outside configured directories (@robertheim_topictags/../../prosilver/theme/rh_topictags.css) in "@robertheim_topictags/css_includes.html" at line 16. in /var/www/data/www/forum/vendor/twig/twig/src/Loader/FilesystemLoader.php:291 Stack trace: #0 /var/www/data/www/forum/vendor/twig/twig/src/Loader/FilesystemLoader.php(209): Twig\Loader\FilesystemLoader->validateName('@robertheim_top...') #1 /var/www/data/www/forum/phpbb/template/twig/loader.php(135): Twig\Loader\FilesystemLoader->findTemplate('@robertheim_top...', true) #2 /var/www/data/www/forum/vendor/twig/twig/src/Loader/FilesystemLoader.php(150): phpbb\template\twig\loader->findTemplate('@robertheim_top...') #3 /var/www/data/www/forum/phpbb/template/twig/environment.php(328): Twig\Loader\FilesystemLoader->getCacheKey('@robertheim_top...') #4 /var/w at file /var/www/data/www/forum/vendor/twig/twig/src/Loader/FilesystemLoader.php line 291

[Perfecthus]

назвав её mystyle-prosilver (как название папки действующего стиля).

Нужно было скопированный каталог переименовать в all

[dimassamid]

Пересобрал версию 3.0.0 с гитхаба c правками под phpBB 3.3.9

RH Topic Tags 3.0.0.zip

[manager42]

Пересобрал версию 3.0.0 с гитхаба c правками под phpBB 3.3.9

Спасибо большое! На 3.3.9 встало без ошибок: всё корректно работает и на форумах, и в админке.

[ReXtor]

Расширение разработано для версии phpBB 3.1.9 поэтому на 3.3.5 права доступа в настройках прав не отображаются и настроить их невозможно.
Исправление

Казалось бы столько времени уже прошло и на гитхабе у автора в описании указана версия 3.3, но эта правка им не выполнена. Кто ставит расширение (версии 3.0.0) сейчас (на phpBB 3.3.8), то не забудьте допилить как указано Re: [3.1] RH Topic Tags

Расширение исправно работает в 3.3.8

Не совсем.

Нашел вот такую ошибку:

2023-01-20_00-16-41.png

В причине возникновения ошибки вопроса нет, но не понимаю почему оно берет текст на англ. языке, вероятно из language/en/topictags.php

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

	'RH_TOPICTAGS_TAGS_INVALID'			=> 'The following tags are invalid: %s',

когда в language/ru/topictags.php имеется перевод

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

	'RH_TOPICTAGS_TAGS_INVALID'			=> 'Следующие теги неправильные: %s',

Где искать причину?

Также заметил что в language/ru/topictags.php отсутствуют некоторые строки:

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

	'RH_TOPICTAGS_NO_TOPICS_FOR_NO_TAG'		=> 'Please search for at least one valid tag to show topics here.',
	
		'RH_TOPICTAGS_TAG_SEARCH' => 'Tag-Search',

	'RH_TOPICTAGS_TAG_SUGGEST_TAG_ROUTE_ERROR' => 'No route found for “%s”',

Про отсутствие RH_TOPICTAGS_TAG_SEARCH упоминалось выше viewtopic.php?p=557285#p557285

И в language/ru/topictags_acp.php не хватает строк:

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

	'ACP_RH_TOPICTAGS_ERROR'								=> 'Error',
	'ACP_RH_TOPICTAGS_UNKNOWN_ERROR'						=> 'Unknown error. See javascript-console for server response.',

Пока просто добавил эти строки в RU скопировав из EN.

Но не понятно откуда берется текст "Information" в том сообщении, тк в lang-файлах самого расширения такого слова нет.

[ReXtor]

В причине возникновения ошибки

Если кого-то интересует воспроизведение ошибки:
- Была тема с тегами test, test2
- Регулярка в настройках расширения была подправлена таким образом, чтобы разрешить тегам начинаться только с символа #
- Таким образом ранее установленные test и test2 стали недействительными
- Недействительные теги сами по себе не исчезают, но при редактировании темы где они есть и попытке предпросмотра, не удалив недействительные теги и возникла такая ошибка.

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

[dimassamid]

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

[ReXtor]

скорее всего, правок и доработок с его стороны не планируется

Да, так и живем.

Мне пока что поиск по нескольким тегам за раз не нужен. Планирую использовать его в качестве "поиска" по хештегам, по одному за раз. Собственно поэтому и поставил регулярку чтобы теги начинались только с символа #. Пока даже киррические символы не добавлял (и не делал правки по отображению кирилиццы в "правильном" виде), чтобы тестировать и отлаживать по порядку.

[ReXtor]

Нашел вот такую ошибку:

Только сейчас заметил что там вообще вся страница на EN-локализации отображается.
И это не какая-то своя от расширения, а вполне обычный posting.php?mode=edit&p=308#preview
А значит расширение недослало правильные параметры, либо прислало не те.

Нашел этот самый RH_TOPICTAGS_TAGS_INVALID в main_listener.php

Код 

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

	public function modify_posting_parameters($event)
	{
		if ($this->auth->acl_gets(permissions::USE_TAGS, permissions::ADMIN_EDIT_TAGS, permissions::MOD_EDIT_TAGS))
		{

			$data = $event->get_data();
			$tags = $this->get_tags_from_post_request();

			$all_tags = $this->tags_manager->split_valid_tags($tags);
			$invalid_tags = $all_tags['invalid'];

			if (sizeof($invalid_tags))
			{
				$this->user->add_lang_ext('robertheim/topictags', 'topictags');
				$data['error'][] = $this->user->lang('RH_TOPICTAGS_TAGS_INVALID', join(', ', $invalid_tags));
			}

			$event->set_data($data);
		}
	}

Меня смущает вот этот кусок:

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

				$this->user->add_lang_ext('robertheim/topictags', 'topictags');
				$data['error'][] = $this->user->lang('RH_TOPICTAGS_TAGS_INVALID', join(', ', $invalid_tags));

Вообще, $this->user->add_lang_ext в расширении замечен только три раза (два других в acp_listener.php),
где есть схожая конструкция вида $data['errors'][] = $this->user->lang('ACP_RH_TOPICTAGS_PRUNING_REQUIRES_TAGGING_DISABLED');,
вот только где-либо найти и воспроизвести то что должно отобразить ACP_RH_TOPICTAGS_PRUNING_REQUIRES_TAGGING_DISABLED я не смог. Потому подозреваю что дело где-то с этим.

Я в писательстве расширений вообще не силен, как и в PHP в целом, поэтому прошу сильно не пинать если заблуждаюсь...

[ReXtor]

в качестве "поиска" по хештегам

Без применения правок из первого сообщения ссылки на тег #txt приобретали вид tag/%2523txt.

Если

внести правки в три файла

То они становились tag/%23txt, что уже лучше, но тоже не очень красиво. Пока пришлось отказаться от идеи использования тегов начинающихся с символа #. Не потому, что в адресной строке оно выглядит через %23 (я понимаю что решетка это ссылка на якорь), а из-за возможной путаницы если вручную вводить в адресную строку /tag/#txt, потому что в таком случае расширение скажет что "ничего не найдено". Теоретически можно придумать какую-нибудь mod_rewrite-фиговину, которая будет делать из /tag/#txt "правильную" /tag/%23txt но я пока не умею.

Отправлено спустя 1 час 8 минут 5 секунд:

какую-нибудь mod_rewrite-фиговину, которая будет делать из /tag/#txt "правильную" /tag/%23txt

Размечтался конечно. 

Идея ни в какую не удалась. Все что начинается с # сервером не воспринимается иначе как ссылка на якорь.
Между тем посмотрел как там с хештегами в том же VK - и тоже, если взять ссылку и заменить в ней %23txt на #txt, то магии не произойдет, хештег не распознается со всеми вытекающими. Видимо придется вернуться к регулярке с # в начале и смириться с невозможностью открыть поиск по тегам расширения по ручной ссылке /tag/#хештег, либо отказываться от символа #. Но ведь все пользователи социалок сильно привыкшие к хештегам через #. Нужно что-то придумывать. Тем более при интеграции phpBB с социалками в виде выгрузки/загрузки контента (кросспостинга) эти самые хештеги лишними не будут.

Пишу свои размышления по поводу тегов/хештегов именно в теме этого расширения, так как иных расширений для тегов не имеем. Придется работать с тем, что есть. И как-то допиливать это непонятное и видимо кривое расширение.