[3.3] [release] Posts merging by rxu

[rxu]

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

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

[COB16]

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

[Siava]

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

[COB16]

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

[COB16]

Разобрался, благодарю Pazh

[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 ругается

[is_pronin]

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

[is_pronin]

Склеивание работает, но чекбокс "Не склеивать с предыдущим" не появляется.

Отставить. Проблема была на моей стороне.

[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'],
		);

[rxu]

заменить на:

А в каком месте первичное экранирование?

[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);

[Pazh]

А в каком месте первичное экранирование?

первое в коде расширения которое я привел для замены. Второе в функции _sql_validate_value, вызываемой в sql_build_array

[digitalfarseer]

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

[Nekstati]

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

[Mushu-svbk]

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

[Siava]

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