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 »

Алг писал(а):мои правки в гитхабе, гляньте, чего не хватает
В гитхабе не вижу ничего.
Но фишка тут в том, что пока, в RC3, сервис должен обзываться notification.type.thanks, и только начиная с RC4 (или в текущем снапшоте) - gfksx.ThanksForPosts.notification.type.thanks.
Так что, для тестирования надо временно в сервисах основного экста его переименовать, если RC3.

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

addon_for_thanks_for_posts

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

rxu писал(а):В гитхабе не вижу ничего
как говорит мой любимый бардТимур Шаов "Посидите чуть... а щас?"
Фишка в том, что я хотела достать все через хелпер и никак ничего не обзывать. У меня-таки стоит RC3

Алг
Сообщения: 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 »

Резонно.
Ну, тогда так оставляем?
Пытаюсь оптимизировать число запросов в основном эксте, пока не выходит. Если будут мысли - велкам :)

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

addon_for_thanks_for_posts

Сообщение rxu »

Там бы еще ошибки подправить, о которых Травис сообщает https://travis-ci.org/alg5/AddonForThan ... s/33430710
Для работоспособности некритично, но всё-таки.

Аватара пользователя
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 писал(а):Если будут мысли - велкам
мысли были всегда - поменять архитектуру мода :oops:
Травис видела, не очень поняла, что ему не нравится

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

addon_for_thanks_for_posts

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

rxu писал(а):Жмешь на кнопку, не срабатывает.
файрбаг писал(а):Call to undefined method gfksx\ThanksForPosts\core\helper::add_notification() in /var/www/phpbbguru/data/www/anderson.phpbbguru.net/ext/alg/AddonForThanksForPosts/controller/thanks_ajax_handler.php on line 114

Ответить