Thanks for posts by Палыч

Обсуждение новой системы расширения функциональности phpBB 3.3, разработки новых расширений.
Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

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:
tfp1.jpg
tfp2.jpg
tfp2.jpg (54.7 КБ) 34170 просмотров
tfp3.jpg
tfp4.jpg
tfp5.jpg

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

Re: Опыты с экстами

Сообщение rxu »

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

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

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

Re: Опыты с экстами

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

rxu писал(а):Если в настройках мода отключить рейтинги форумов, должно прийти в норму.
к сожалению, нет. У меня рейтинг для форумов всегда был отключен... будем искать ;)

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

Re: Опыты с экстами

Сообщение rxu »

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

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

'core.display_forums_modify_template_vars'	=> 'forumlist_modify_template_vars',
вид списка форумов приходит в норму.

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

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

Re: Опыты с экстами

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

эта строчка <!-- 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
Сообщения: 1116
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Благодарил (а): 84 раза
Поблагодарили: 108 раз
Контактная информация:

Re: Опыты с экстами

Сообщение 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.

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

Re: Опыты с экстами

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

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

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

Re: Опыты с экстами

Сообщение rxu »

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

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28
Благодарил (а): 36 раз
Поблагодарили: 97 раз
Контактная информация:

Re: Опыты с экстами

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

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

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

Re: Опыты с экстами

Сообщение rxu »

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

Закрыто