addon_for_thanks_for_posts

Обсуждение новой системы расширения функциональности phpBB 4.0-dev, разработки новых расширений.
Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

addon_for_thanks_for_posts

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

офф а склейка у нас отключена что ли?

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

addon_for_thanks_for_posts

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

офф 2 и снова пропал пост на границе

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

addon_for_thanks_for_posts

Сообщение rxu »

Склейка пока отключена.

Аватара пользователя
Татьяна5
Сообщения: 570
Зарегистрирован: 02 фев 2014, 15:05

addon_for_thanks_for_posts

Сообщение Татьяна5 »

Скачала только что последние версии основного расширения и адд-она с гитхаба, установила, адд-он не работает - конфликт с QuickReply

При нажатии на кнопку формируется ссылка вида

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

http://phpbb31.zz/app.php/AddonForThanksForPosts/thanks/null/2/2/324
Код из адд-она

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

                var url =$(post_block).find('.postprofile').find('dt').find('a').attr('href');
                var poster_id =  $.urlParam('u', url);
Но в той ссылке при включённом обращении по нику нет параметра u

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

<a id="48" class="username" href="javascript:void(0);">test</a>
Алг, может, можно взять параметр to_id из ссылки с кнопки "спасибо" /viewtopic.php?f=2&p=324&thanks=324&to_id=48&from_id=2 , чтобы адд-он не зависел от других расширений?

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

addon_for_thanks_for_posts

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

можно взять. не проблема даже удобнее

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

Хотел сделать ПР для уведомлений, но не понял, откуда взять дополнительные данные как в хелпере основного экста - а именно post_subject. Там есть SQL запрос, тут нет.

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

Как говорил один известный сын турецкоподданного в одном культовом фильме, "Шура, я бы взял частями, но мне нужно сразу" :mrgreen:
Кстати, функцию отправки уведомления в хелпер добавил.

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

addon_for_thanks_for_posts

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

Как говорил один известный мастер "не выходит каменный цветок."
а вот чего выходит
The service definition "notification.type.thanks" does not exist.
мои правки в гитхабе, гляньте, чего не хватает

Ответить