Topic Tags

Обсуждение новой системы расширения функциональности phpBB 4.0-dev, разработки новых расширений.
Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41

Topic Tags

Сообщение Alecto »

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

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

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

karas
Сообщения: 1
Зарегистрирован: 17 окт 2014, 13:40

Topic Tags

Сообщение karas »

Спасибо Вам за перевод мода.

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

Grewi
Сообщения: 7
Зарегистрирован: 29 мар 2014, 17:17

Topic Tags

Сообщение Grewi »

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

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

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

Как можно победить эту беду?
Последний раз редактировалось Grewi 02 ноя 2014, 19:13, всего редактировалось 1 раз.

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

Topic Tags

Сообщение rxu »

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

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

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

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

Grewi
Сообщения: 7
Зарегистрирован: 29 мар 2014, 17:17

Topic Tags

Сообщение Grewi »

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

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

Topic Tags

Сообщение rxu »

Ну правильно, так и должно быть :ugeek:
Перепутал. Вот так правильно

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

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

Grewi
Сообщения: 7
Зарегистрирован: 29 мар 2014, 17:17

Topic Tags

Сообщение Grewi »

Нет. Кириллицу не пропускает, пишет: "Следующие теги неправильные...".

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

Topic Tags

Сообщение rxu »

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

Grewi
Сообщения: 7
Зарегистрирован: 29 мар 2014, 17:17

Topic Tags

Сообщение Grewi »

Мда, печально. А расширение-то полезное было.
rxu, спасибо за внимание к проблеме.

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

Topic Tags

Сообщение rxu »

Ну почему же "было", оно и есть, если автор доработает - будет и дальше полезным.

Ответить