addon_for_thanks_for_posts
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
addon_for_thanks_for_posts
Для событий шаблона задать приоритет невозможно, к сожалению.
Отправлено спустя 1 минуту 35 секунд:
Скорее всего, это потому, что в аддоне событие из хедера, а в основном эксте из вьютопика, поэтому отрабатывает позже.
Отправлено спустя 1 минуту 13 секунд:
Хотя нет, в аддоне оно в футере. Странно.
Отправлено спустя 13 минут 35 секунд:
Понятно. В основном эксте эта переменная находится в блоке postrow, т.е. postrow.S_FORUM_THANKS. неясно, зачем для каждого поста проверять право выдачи спасибок в данном форуме, вроде бы, одного раза (глобально) достаточно?
Отправлено спустя 1 минуту 35 секунд:
Скорее всего, это потому, что в аддоне событие из хедера, а в основном эксте из вьютопика, поэтому отрабатывает позже.
Отправлено спустя 1 минуту 13 секунд:
Хотя нет, в аддоне оно в футере. Странно.
Отправлено спустя 13 минут 35 секунд:
Понятно. В основном эксте эта переменная находится в блоке postrow, т.е. postrow.S_FORUM_THANKS. неясно, зачем для каждого поста проверять право выдачи спасибок в данном форуме, вроде бы, одного раза (глобально) достаточно?
addon_for_thanks_for_posts
достаточно. У меня тоже этот вопрос возникал, сейчас вспомнила. Спросить почему-то не догадалась
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
addon_for_thanks_for_posts
На локалке сделал, аддон работает без слушателя.
addon_for_thanks_for_posts
жду пр
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
addon_for_thanks_for_posts
Пока еще обдумываю.
То, что яваскрипт не будет включен на странице, не спасает от теоретической возможности запустить контроллер напрямик через адресную строку браузера. Поэтому, всё-таки видимо надо рисовать еррор на такой случай. Для этого придется языковой файл добавить в аддон, ятобы иметь текст типа
Отправлено спустя 52 минуты 15 секунд:
https://github.com/alg5/AddonForThanksForPosts/pull/11
То, что яваскрипт не будет включен на странице, не спасает от теоретической возможности запустить контроллер напрямик через адресную строку браузера. Поэтому, всё-таки видимо надо рисовать еррор на такой случай. Для этого придется языковой файл добавить в аддон, ятобы иметь текст типа
Код: Выделить всё
'NOT_INSTALLED' => 'Расширение «Thanks for posts» не установлено.',
https://github.com/alg5/AddonForThanksForPosts/pull/11
addon_for_thanks_for_posts
ок. взяла
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
addon_for_thanks_for_posts
на локалке срабатывает. При отключении спасибок выдает красную всплывашку с ошибкой.
- rxu
- Сообщения: 1125
- Зарегистрирован: 21 янв 2014, 21:20
- Откуда: Krasnoyarsk
- Github repo: https://github.com/rxu
- Контактная информация:
addon_for_thanks_for_posts
не так чтобы очень, но лучше чем раньше было
addon_for_thanks_for_posts
При попытке включить на RC5 вылазит пустая ошибка, на RC2 все отлично работало. Возможно это из-за украинского языка форума.