Страница 1 из 2

Topic Tags

Добавлено: 08 окт 2014, 16:54
Alecto
Расширение для PHPBB 3.1, которое добавляет возможность создавать в темах теги, в дальнейшем выбирать темы по созданным ранее меткам.

Тема на офсайте https://www.phpbb.com/community/viewtop ... &t=2263616 там же скрины и ссылки для скачивания.
Скачать с гитхаба https://github.com/RobertHeim/phpbb-ext-topictags

Сделал перевод для актуальной версии.
Буду благодарен за помощь в выявлении ошибок перевода.

Topic Tags

Добавлено: 17 окт 2014, 14:05
karas
Спасибо Вам за перевод мода.

Предлагаю немного переделать строку
ПокОзаны популярные %d теги
т.к. в зависимости от числа, должно меняться окончание (популярные (-ных), тег(-а, -ов)), а в данном окончание остается неизменным и как-то становится не читаемой надпись "Показаны популярные 15 теги"... Может быть лучше так: "Самые популярные теги (15):"?

Topic Tags

Добавлено: 02 ноя 2014, 18:22
Grewi
Если ставлю версию с переводом, то расширение не функционирует. Ставится нормально, но теги нигде не отображаются и не сохраняются, после отправки сообщения, как будто их и не вписывали.
А версия с оф. сайта работает нормально, но только с латиницей. Пытался изменить условие в админке на

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

/^[\- а-яА-ЯёЁa-zA-Z0-9]{3,30}$/i
кириллица стала обрабатываться, но система пропускала только одно русское слово, а потом пишет об ошибке. Ни в этой же, ни в другой теме вставить русский тег уже нельзя.

Как можно победить эту беду?

Topic Tags

Добавлено: 02 ноя 2014, 18:44
rxu

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

[\- а-яА-ЯёЁa-zA-Z0-9]
А если попробовать

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

[\P{L}\- 0-9]

Topic Tags

Добавлено: 02 ноя 2014, 19:09
Grewi
rxu писал(а):QR_BBPOST
КОД: ВЫДЕЛИТЬ ВСЁ
[\P{L}\- 0-9]
Так пропускает только цифры и тире. :(

Topic Tags

Добавлено: 02 ноя 2014, 20:17
rxu
Ну правильно, так и должно быть :ugeek:
Перепутал. Вот так правильно

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

[\p{L}\- 0-9]

Topic Tags

Добавлено: 02 ноя 2014, 20:26
Grewi
Нет. Кириллицу не пропускает, пишет: "Следующие теги неправильные...".

Topic Tags

Добавлено: 02 ноя 2014, 20:33
rxu
[ref]Grewi[/ref], очевидно, данное расширение не рассчитано на юникод. например, здесь https://github.com/RobertHeim/phpbb-ext ... r.php#L609 при проверке валидности тега используется substr(), которая неверно выберет подстроку из строки в юникоде.
Вероятно, и другие места есть, требующие адаптации к юникоду.

Topic Tags

Добавлено: 02 ноя 2014, 20:37
Grewi
Мда, печально. А расширение-то полезное было.
rxu, спасибо за внимание к проблеме.

Topic Tags

Добавлено: 02 ноя 2014, 20:43
rxu
Ну почему же "было", оно и есть, если автор доработает - будет и дальше полезным.