[3.2][3.3] Thanks for posts

[rxu]

Обсуждаем портирование мода Thanks for post (Rating edition) Палыча в расширение для phpBB 3.2/3.3.
Копировать в папку: ext/gfksx/thanksforposts (все символы имени папки в нижнем регистре).

Страница на официальном сайте (версия 2.1.1, прошла валидацию): https://www.phpbb.com/customise/db/exte ... r_posts_2/
Версия для phpBB 3.3 в репозитории: https://github.com/rxu/thanks_for_posts ... elop-3.3.x

Внимание: начиная с версии 2.1.0, AJAX внедрен в ядро расширения, дополнение Addon for thanks for posts не требуется и может привести к его неработоспособности. При обновлении с предыдущих версий дополнение Addon for thanks for posts необходимо предварительно отключить.

Внимание: начиная с версии 2.0.4, имя расширения изменено на gfksx/thanksforposts (ранее - gfksx/ThanksForPosts).

Для обновления с более ранних версий необходимо: 

• Отключить расширение gfksx/ThanksForPosts
• Создать резервную копию таблицы базы данных phpbb_thanks (префикс phpbb_ дан для примера)
• Удалить данные расширения gfksx/ThanksForPosts
• Удалить папку ext/gfksx/ThanksForPosts с сервера
• Восстановить таблицу базы данных phpbb_thanks из резервной копии (префикс phpbb_ дан для примера)
• Загрузить файлы нового расширения версии 2.0.4+ в папку ext/gfksx/thanksforposts (все символы имени папки в нижнем регистре)
• Включить расширение в администраторском разделе

Для версий Thanks for posts ниже, чем 2.1.0, AJAX дополнение: 

Позволяет выдавать благодарности без перезагрузки страницы: Addon for thanks for posts by Алг.
Исходный код: https://github.com/alg5/addonforthanksforposts
Копировать в папку: /ext/alg/addonforthanksforposts/
Внимание: вышеозначенное дополнение может не работать с основным расширением версии 2.0.4 без следующих правок: Re: [dev] Thanks for posts
Для совместимости с версией 2.0.7 требуются следующие правки:
https://github.com/alg5/addonforthanksf ... l/19/files

------------------------
Вопрос-ответ:
Q. Как сделать так чтобы гости видели список поблагодаривших за сообщение при просмотре темы?
W. Разрешить в форумных правах доступа(Или роль "Доступ только для чтения") гостям благодарить за сообщение. Благодарить не смогут, а видеть список поблагодаривших будут.

Q. Как сделать так чтобы гости смогли благодарить за сообщение?
W. В этом нет практического смысла. В phpBB гость - это один пользователь. Соответственно, если любой один гость поблагодарит, следующий только сможет отозвать благодарность, и так по кругу.

Версия для phpBB 3.1: 

https://github.com/rxu/thanks_for_posts/tree/master

Версия для phpBB 3.2: 

https://github.com/rxu/thanks_for_posts ... elop-3.2.x

[LBeaver]

Alecto

Отключил все расширения, очистил кэш, включил только Thanks for posts и Addon for Thanks for posts

[Татьяна5]

Попала на ошибку

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

SQL ERROR [ mysql4 ]

Column 'forum_id' cannot be null [1048]

SQL

UPDATE phpbb_thanks SET forum_id = (SELECT forum_id FROM phpbb_posts WHERE post_id = phpbb_thanks.post_id AND post_id IN (355, 355, 780, 371526, 1976304, 1977177, 1977178, 1977183, 1977185, 1977189))

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 855
CALL: trigger_error()

FILE: [ROOT]/phpbb/db/driver/mysql.php
LINE: 181
CALL: phpbb\db\driver\driver->sql_error()

FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 329
CALL: phpbb\db\driver\mysql->sql_query()

FILE: [ROOT]/ext/gfksx/ThanksForPosts/migrations/v_1_2_8.php
LINE: 107
CALL: phpbb\db\driver\factory->sql_query()

FILE: (not given by php)
LINE: (not given by php)
CALL: gfksx\ThanksForPosts\migrations\v_1_2_8->update_thanks_table()

FILE: [ROOT]/phpbb/db/migrator.php
LINE: 517
CALL: call_user_func_array()

FILE: [ROOT]/phpbb/db/migrator.php
LINE: 463
CALL: phpbb\db\migrator->run_step()

FILE: [ROOT]/phpbb/db/migrator.php
LINE: 301
CALL: phpbb\db\migrator->process_data_step()

FILE: [ROOT]/phpbb/db/migrator.php
LINE: 241
CALL: phpbb\db\migrator->try_apply()

FILE: [ROOT]/phpbb/db/migrator.php
LINE: 181
CALL: phpbb\db\migrator->try_apply()

FILE: [ROOT]/phpbb/extension/base.php
LINE: 80
CALL: phpbb\db\migrator->update()

FILE: [ROOT]/ext/gfksx/ThanksForPosts/ext.php
LINE: 52
CALL: phpbb\extension\base->enable_step()

FILE: [ROOT]/phpbb/extension/manager.php
LINE: 187
CALL: gfksx\ThanksForPosts\ext->enable_step()

FILE: [ROOT]/includes/acp/acp_extensions.php
LINE: 184
CALL: phpbb\extension\manager->enable_step()

FILE: [ROOT]/includes/functions_module.php
LINE: 674
CALL: acp_extensions->main()

FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()

Повторить можно при установке расширения "с нуля" при включённом дебаге, на phpBB 3.1.2
Причина обсуждалась на anderson.phpbbguru.net

[rxu]

Если бы кто-то пояснил, что делает этот запрос, можно было бы попробовать исправить. Я не понимаю.

[Татьяна5]

Синхронизирует значения forum_id таблицы phpbb_thanks с таблицей phpbb_posts. Когда phpbb_thanks пустая получаем ошибку

[rxu]

Когда phpbb_thanks пустая получаем ошибку

Вот код миграции. Сначала идет выборка из таблиц THANKS_TABLE и POSTS_TABLE, и если первая - пустая, по идее, должен быть пустой результат. соответственно, ($posts_list будет пустой и синхронизационный запрос не должен выполняться.

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

	public function update_thanks_table()
	{
		$posts_list = array();
		if (!defined('THANKS_TABLE'))
		{
			define('THANKS_TABLE', $this->table_prefix . 'thanks');
		}
		$sql = 'SELECT tt.post_id
			FROM ' . THANKS_TABLE . ' tt, ' . POSTS_TABLE . ' p
			WHERE tt.post_id = p.post_id';
		$result = $this->db->sql_query($sql);
		while ($row = $this->db->sql_fetchrow($result))
		{
			$posts_list[] = (int) $row['post_id'];
		}
		$this->db->sql_freeresult($result);
		if (!empty($posts_list))
		{
			$sql = 'UPDATE '. THANKS_TABLE . '
				SET forum_id = (SELECT forum_id 
				FROM '. POSTS_TABLE . '
				WHERE post_id = ' . THANKS_TABLE . '.post_id
					AND ' . $this->db->sql_in_set('post_id', $posts_list) . ')';
			$this->db->sql_query($sql);
			$sql = 'UPDATE '. THANKS_TABLE . '
				SET topic_id = (SELECT topic_id
				FROM '. POSTS_TABLE . '
				WHERE post_id = ' . THANKS_TABLE . '.post_id
					AND ' . $this->db->sql_in_set('post_id', $posts_list) . ')';
			$this->db->sql_query($sql);
		}
	}

[COB16]

Не отображаются аватары пользователей в уведомлениях

111.JPG

[Alecto]

стесняюсь спросить, вы ото самое, или прикидываетесь?
в последнем уведомлении аватарка видна, а у верхних - один пользователь, поставьте ему аватару и будет отображаться.

[Slash]

Alecto, видимо аватарка видна от другого расширения, вовсе не от спасибо.

[COB16]

Alecto аватарка стоит у пользователя который сказал спасибо

[rxu]

Уже сказал в другой теме, возьмите свежий код с репозитория, должно работать.

[nusman]

Спасибо огромное! Все работает прекрасно! Подскажите пожалуйста, как заменить иконки, на прилагаемом скрине ?

vopros2.png

spasibo.gif

[Romnik]

Поменяйте картинку на вашу.
Картинки в папке: ext/ThanksForPosts/styles/prosilver/theme/images/

[nusman]

Romnik спасибо пробую, не получается, там нашел картинку которая отображается И заменил ее и переименовал ее так же как и предыдущею, ... но все равно все также... может размер не подходящий?

[владимир1983]

Так просто не получится заменить кнопку.
Кнопка сделана спрайтом. Посмотри в архиве экста спрайт /styles/prosilver/theme/images/icons_button_likes.png
Посмотри css кнопки благодарности .thanks-icon:before и отмены .removethanks-icon:before может станет яснее.

[spoler]

Romnik спасибо пробую, не получается, там нашел картинку которая отображается И заменил ее и переименовал ее так же как и предыдущею, ... но все равно все также... может размер не подходящий?

Кэш, может быть, надо после этого очистить.