Страница 5 из 12

addon_for_thanks_for_posts

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

addon_for_thanks_for_posts

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

addon_for_thanks_for_posts

Добавлено: 24 авг 2014, 21:57
Алг
с переименованием аддон заработал!

addon_for_thanks_for_posts

Добавлено: 24 авг 2014, 22:07
rxu
М-да, лишний запросец появился. А без него никак нельзя? Может, как и остальное - выдернуть из кода и передать параметром контроллеру, нет?

addon_for_thanks_for_posts

Добавлено: 24 авг 2014, 22:15
Алг
да думала об этом, но текстовое поле контроллеру передавать как-то... не знаю. а там ещё и символы разные, неадекватные могут встретиться - значит, экранировать надо

addon_for_thanks_for_posts

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

addon_for_thanks_for_posts

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

addon_for_thanks_for_posts

Добавлено: 24 авг 2014, 22:58
rxu
Обновил, не фурычит. Жмешь на кнопку, не срабатывает. После перезагрузки страницы видно спасибку. Жмешь на отмену - молчит. После перезагрузки страницы - отменилось.

addon_for_thanks_for_posts

Добавлено: 24 авг 2014, 22:59
Алг
rxu писал(а):Если будут мысли - велкам
мысли были всегда - поменять архитектуру мода :oops:
Травис видела, не очень поняла, что ему не нравится

addon_for_thanks_for_posts

Добавлено: 24 авг 2014, 23:00
Алг
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