Страница 23 из 28

[release] Posts merging by rxu

Добавлено: 09.01.2015 15:03
rxu
Расширение прошло валидацию на phpbb.com: https://www.phpbb.com/customise/db/exte ... tsmerging/

Исходный код: https://github.com/rxu/posts_merging
Копировать в папку /ext/rxu/postsmerging/.

Re: [release] Posts merging by rxu

Добавлено: 18.11.2018 20:27
COB16
Не могу изменить цвет сообщения о переносе, при нажатии на палитру просто кидает выше по странице. Вручную пишешь цвет бб кодом все равно не меняет.
Версия phpbb 3.2.3 / php 7.1 /

Re: [release] Posts merging by rxu

Добавлено: 18.11.2018 20:31
Siava
COB16, у меня всё работает корректно. Версия расширения 2.1.0 + php 7.2.12 + Firefox.
А вот "Предпросмотр разделителя" в настройках в админке отображает одинаково зелёным и до и после правки))

Re: [release] Posts merging by rxu

Добавлено: 18.11.2018 20:38
COB16
Siava, пробовал переустановить та же ситуация почему то. И цвет если вручную задать не меняет потом на страницах форума. И почему то при установке пишет что есть более свежая версия но скачивает ту же

Re: [release] Posts merging by rxu

Добавлено: 19.11.2018 19:32
COB16
Разобрался, благодарю Pazh

Re: [release] Posts merging by rxu

Добавлено: 13.12.2018 11:57
Dook
rxu, обновите номер unstable-версии:
http://rxu.github.io/versions/posts_mer ... rsion.json - 2.1.1-dev
https://github.com/rxu/posts_merging/bl ... poser.json - 2.1.0
А то Upload extensions ругается

Re: [release] Posts merging by rxu

Добавлено: 14.03.2019 5:08
is_pronin
Склеивание работает, но чекбокс "Не склеивать с предыдущим" не появляется. В пользовательских ролях право "Может игнорировать склеивание сообщений" установлено в "Да". Версия расширения 2.1.0, версия phpBB 3.2.5.

Re: [release] Posts merging by rxu

Добавлено: 14.03.2019 12:19
is_pronin
is_pronin писал(а): 14.03.2019 5:08 Склеивание работает, но чекбокс "Не склеивать с предыдущим" не появляется.
Отставить. Проблема была на моей стороне.

Re: [release] Posts merging by rxu

Добавлено: 07.05.2019 14:52
Pazh
Баг: при наличии в заголовке сообщения/темы или имени юзера символов, требующих экранирования - например одинарной кавычки - суть ошибки на скрине:
Screenshot_11.jpg
Символ экранирования \ сохраняется как обычный символ в БД из-за двойного экранирования функцией sql_escape

fix:
в файле rxu/PostMerging/core/helper.php найти

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

		$sql_data[FORUMS_TABLE]['sql'] = array(
			'forum_last_post_time'		=> $data['post_time'],
			'forum_last_post_id'		=> $data['post_id'],
			'forum_last_post_subject'	=> $this->db->sql_escape($data['post_subject']),
			'forum_last_poster_id'		=> (int) $this->user->data['user_id'],
			'forum_last_poster_name'	=> $this->db->sql_escape($this->user->data['username']),
			'forum_last_poster_colour'	=> $this->db->sql_escape($this->user->data['user_colour']),
		);
заменить на:

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

		$sql_data[FORUMS_TABLE]['sql'] = array(
			'forum_last_post_time'		=> $data['post_time'],
			'forum_last_post_id'		=> $data['post_id'],
			'forum_last_post_subject'	=> $data['post_subject'],
			'forum_last_poster_id'		=> (int) $this->user->data['user_id'],
			'forum_last_poster_name'	=> $this->user->data['username'],
			'forum_last_poster_colour'	=> $this->user->data['user_colour'],
		);

Re: [release] Posts merging by rxu

Добавлено: 07.05.2019 18:31
rxu
Pazh писал(а): 07.05.2019 14:52заменить на:
А в каком месте первичное экранирование?

Re: [release] Posts merging by rxu

Добавлено: 07.05.2019 19:50
Siava
rxu, в includes/functions_posting.php есть в парочке мест:

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

		while ($row = $db->sql_fetchrow($result))
		{
			$update_sql[$row["{$type}_id"]][] = $type . '_last_post_id = ' . (int) $row['post_id'];
			$update_sql[$row["{$type}_id"]][] = "{$type}_last_post_subject = '" . $db->sql_escape($row['post_subject']) . "'";
			$update_sql[$row["{$type}_id"]][] = $type . '_last_post_time = ' . (int) $row['post_time'];
			$update_sql[$row["{$type}_id"]][] = $type . '_last_poster_id = ' . (int) $row['poster_id'];
			$update_sql[$row["{$type}_id"]][] = "{$type}_last_poster_colour = '" . $db->sql_escape($row['user_colour']) . "'";
			$update_sql[$row["{$type}_id"]][] = "{$type}_last_poster_name = '" . (($row['poster_id'] == ANONYMOUS) ? $db->sql_escape($row['post_username']) : $db->sql_escape($row['username'])) . "'";
		}
		$db->sql_freeresult($result);

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

		if ($post_visibility == ITEM_APPROVED)
		{
			if ($post_mode == 'post')
			{
				$config->increment('num_topics', 1, false);
			}
			$config->increment('num_posts', 1, false);

			$sql_data[FORUMS_TABLE]['stat'][] = 'forum_last_post_id = ' . $data_ary['post_id'];
			$sql_data[FORUMS_TABLE]['stat'][] = "forum_last_post_subject = '" . $db->sql_escape($subject) . "'";
			$sql_data[FORUMS_TABLE]['stat'][] = 'forum_last_post_time = ' . $current_time;
			$sql_data[FORUMS_TABLE]['stat'][] = 'forum_last_poster_id = ' . (int) $user->data['user_id'];
			$sql_data[FORUMS_TABLE]['stat'][] = "forum_last_poster_name = '" . $db->sql_escape((!$user->data['is_registered'] && $username) ? $username : (($user->data['user_id'] != ANONYMOUS) ? $user->data['username'] : '')) . "'";
			$sql_data[FORUMS_TABLE]['stat'][] = "forum_last_poster_colour = '" . $db->sql_escape($user->data['user_colour']) . "'";
		}

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

	// Update the forum's last post info
	$sql = 'UPDATE ' . FORUMS_TABLE . "
		SET forum_last_post_id = " . $post_data['topic_last_post_id'] . ",
			forum_last_poster_id = " . $post_data['topic_last_poster_id'] . ",
			forum_last_post_subject = '" . $db->sql_escape($post_data['topic_last_post_subject']) . "',
			forum_last_post_time = $bump_time,
			forum_last_poster_name = '" . $db->sql_escape($post_data['topic_last_poster_name']) . "',
			forum_last_poster_colour = '" . $db->sql_escape($post_data['topic_last_poster_colour']) . "'
		WHERE forum_id = $forum_id";
	$db->sql_query($sql);

Re: [release] Posts merging by rxu

Добавлено: 07.05.2019 20:16
Pazh
rxu писал(а): 07.05.2019 18:31 А в каком месте первичное экранирование?
первое в коде расширения которое я привел для замены. Второе в функции _sql_validate_value, вызываемой в sql_build_array

Re: [release] Posts merging by rxu

Добавлено: 11.03.2020 0:50
digitalfarseer
Как изменить цвет надписи "добавлено через"? Цвет в настройках шаблона разделителя не влияет на цвет основной надписи.

Re: [release] Posts merging by rxu

Добавлено: 11.03.2020 1:06
Nekstati
digitalfarseer, зелёный цвет задан в файле language\ru\posts_merging.php. Либо поменяйте цвет в этом файле, либо в админке введите другой текст разделителя вместо переменной {L_MERGE_SEPARATOR}.

Re: [release] Posts merging by rxu

Добавлено: 25.09.2020 21:10
Mushu-svbk
расширение установлено, включено. настройки расширения установила, но сообщения не склеиваются. Что еще может быть? и где?

Re: [release] Posts merging by rxu

Добавлено: 25.09.2020 21:24
Siava
Mushu-svbk, права доступа - пользовательские роли > ... > Может использовать склеивание сообщений