Thanks for posts by Палыч

[Алг]

Обсуждаем портирование мода Thanks for posts Палыча в расширение для phpBB 3.1.
Исходный код тут: https://github.com/rxu/thanks_for_posts


поставила "спасибо", поплыл весь дизайн

Отправлено спустя 0 часов 38 минут 23 секунды:
http://chugunok.net/forum_test_3_1/index.php
отключаю - нормально, включаю - вот так

Screenshots:

tfp2.jpg (54.7 КБ) 44253 просмотра

[rxu]

Смотрю и понять не могу - проблема идет в списке форумов, что-то связано с рейтингами форумов.
Пока не разобрался, что-то где-то не доделано.
Если в настройках мода отключить рейтинги форумов, должно прийти в норму.

Буду благодарен за подсказку.

[Алг]

Если в настройках мода отключить рейтинги форумов, должно прийти в норму.

к сожалению, нет. У меня рейтинг для форумов всегда был отключен... будем искать

[rxu]

Всё равно, проблема где-то в этом. Если в слушателе закомментировать строку

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

'core.display_forums_modify_template_vars'	=> 'forumlist_modify_template_vars',

вид списка форумов приходит в норму.

Отправлено спустя 0 часов 4 минуты 17 секунд:
дело в функции get_thanks_forum_reput, надо разбираться тут.

[Алг]

эта строчка <!-- IF forumrow.S_IS_CAT or forumrow.S_FIRST_ROW or forumrow.S_NO_CAT --> все переменные в ней пустые, потому часть хтмл кода теряется

где-то переопределяется темплейт forumrow

Отправлено спустя 0 часов 19 минут 1 секунду:
попутно одну ошибку нашла

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

if (isset($this->config['thanks_forum_reput_view']))

надо бы

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

		if (isset($this->config['thanks_forum_reput_view']) && $this->config['thanks_forum_reput_view'])

[rxu]

Всё понял. В событии 'core.display_forums_modify_template_vars' функция get_thanks_forum_reput определяет подблок 'forumrow.reput', при этом основной блок 'forumrow' еще не определен (присвоение идет уже после события).
Отсюда кривизна.
Нужно добавлять событие после строки

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

$template->assign_block_vars('forumrow', $forum_row);

в includes/functions_display.php.

[Алг]

и я на это наткнулась
а из-за ошибки, что я выше написала, у меня вызывалась эта функция при отключенном рейтинге форумов

[rxu]

надо бы

Так и есть, надо исправлять. И добавлять событие

[Алг]

теперь следующая проблема: кнопки "спасибо" не вижу
А кстати, расширение без аякса, как я вижу?

[rxu]

Без Аякса, откуда ж ему там взяться
Кнопки нет, т.к. код кнопок в посте полностью переписан в последней бэте, а в моде она осталась старая. Я так думаю.
Ну, или кэш.