addon_for_thanks_for_posts

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

addon_for_thanks_for_posts

Сообщение rxu »

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

Отправлено спустя 1 минуту 35 секунд:
Скорее всего, это потому, что в аддоне событие из хедера, а в основном эксте из вьютопика, поэтому отрабатывает позже.

Отправлено спустя 1 минуту 13 секунд:
Хотя нет, в аддоне оно в футере. Странно.

Отправлено спустя 13 минут 35 секунд:
Понятно. В основном эксте эта переменная находится в блоке postrow, т.е. postrow.S_FORUM_THANKS. неясно, зачем для каждого поста проверять право выдачи спасибок в данном форуме, вроде бы, одного раза (глобально) достаточно?

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

addon_for_thanks_for_posts

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

достаточно. У меня тоже этот вопрос возникал, сейчас вспомнила. Спросить почему-то не догадалась

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

addon_for_thanks_for_posts

Сообщение rxu »

На локалке сделал, аддон работает без слушателя.

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

addon_for_thanks_for_posts

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

жду пр

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

addon_for_thanks_for_posts

Сообщение rxu »

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

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

'NOT_INSTALLED'				=> 'Расширение «Thanks for posts» не установлено.',
Отправлено спустя 52 минуты 15 секунд:
https://github.com/alg5/AddonForThanksForPosts/pull/11

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

addon_for_thanks_for_posts

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

ок. взяла

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

addon_for_thanks_for_posts

Сообщение rxu »

на локалке срабатывает. При отключении спасибок выдает красную всплывашку с ошибкой.

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

addon_for_thanks_for_posts

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

супер

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

addon_for_thanks_for_posts

Сообщение rxu »

не так чтобы очень, но лучше чем раньше было :ugeek:

AndriyUA
Сообщения: 24
Зарегистрирован: 18 июл 2014, 15:27
Контактная информация:

addon_for_thanks_for_posts

Сообщение AndriyUA »

При попытке включить на RC5 вылазит пустая ошибка, на RC2 все отлично работало. Возможно это из-за украинского языка форума.

Ответить