Серый форум — OFF: Баг с цитированием

Re: OFF: Баг с цитированием

null@example.com (Malcev) — Sat, 05 Jan 2019 08:50:46 +0000

test
При цитировании первой строки этого сообщения, цитируется всё сообщение.

Re: OFF: Баг с цитированием

null@example.com (atomix) — Thu, 04 May 2017 18:57:38 +0000

Поправил. Нашел на форуме.

Re: OFF: Баг с цитированием

null@example.com (JSmаn) — Thu, 04 May 2017 15:57:23 +0000

Имеются ли ошибки в дебагере?

Re: OFF: Баг с цитированием

null@example.com (mozers) — Wed, 03 May 2017 18:15:58 +0000

becauseim, у меня так же. Более недели назад началось.

Re: OFF: Баг с цитированием

null@example.com (becauseim) — Wed, 03 May 2017 15:54:25 +0000

Наблюдаю у себя баг при попытке скопировать код либо выделить его в браузере Chrome. При нажатии на соответствующие элементы страницы выбрасывает в начало страницы. В IE работает нормально. Только у меня такое наблюдается?

Re: OFF: Баг с цитированием

null@example.com (teadrinker) — Fri, 21 Apr 2017 13:12:13 +0000

Тут я не в теме, в php ничего не понимаю.

Re: OFF: Баг с цитированием

null@example.com (Malcev) — Fri, 21 Apr 2017 13:08:37 +0000

teadrinker пишет:

на месте смайлов их тестовое отображение

Чтобы избавиться от текстового отображения нужно удалить ALT аттрибут у смайлов в include/parser.php:

$text = preg_replace("#(?<=[>\s])".preg_quote($smiley_text, '#')."(?=\W)#m", '', $text);

Re: OFF: Баг с цитированием

null@example.com (teadrinker) — Fri, 21 Apr 2017 13:02:24 +0000

Функция не работает с html-содержимым объектов. В переменной post_content_1 текст, как он напечатан (на месте смайлов их текстовое отображение), в переменной selected_text_1 выделенный текст. Если в выделенном тексте был смайл или любое другое изображение, оно просто исключается.

Re: OFF: Баг с цитированием

null@example.com (Malcev) — Fri, 21 Apr 2017 12:23:38 +0000

Так мы же удаляем конкретные строки.
Что там может что-то нужное удалиться, кроме ссылок на смайлы?

Re: OFF: Баг с цитированием

null@example.com (teadrinker) — Fri, 21 Apr 2017 12:09:06 +0000

Можно, но если в сообщении смайлы специально отключены, может удалиться что-то нужное.

Re: OFF: Баг с цитированием

null@example.com (Malcev) — Fri, 21 Apr 2017 10:27:17 +0000

А я так и не понял, почему BBCode регекспами можно убрать, а смайлы нет.

Re: OFF: Баг с цитированием

null@example.com (teadrinker) — Tue, 11 Apr 2017 18:29:23 +0000

По-моему, проблема здесь:

if (post_content_1.indexOf(selected_text_1) != -1)

Здесь проверяется, является ли выделенный текст частью общего текста поста. Если есть смайл, в переменной post_content_1 есть его текстовое отображение, а в selected_text_1 его нет, соответственно, условие не срабатывает. Если совсем убрать эту проверку, то может вставляться текст, выделенный в другом посте. Чтобы обойти это, нужно в функции getSelectedText() проверять, находится ли выделенный текст в том же посте, где нажата кнопка. Для этого можно передавать ей значение qid_param при вызове из QuickQuote(qid_param). Если текст не из нужного поста, возвращать пустое значение, а из QuickQuote() тогда можно будет убрать эту проверку:

function QuickQuote(qid_param)
{
	var selected_text = getSelectedText();
	var quick_post_value = document.getElementsByName('req_message');
	var cur_pos = getCaretPos();
	var text = quick_post_value[0].value;
	var text_below = text.substring(0, cur_pos);
	var text_above = text.substring(cur_pos, text.length);

	if (selected_text == undefined || selected_text == '')
	{
		var quote = '[quote=' + pun_quote_authors[qid_param] + ']' + ParseMessage(pun_quote_posts[qid_param]) + '[/quote]';
		quick_post_value[0].value = text_below + quote + text_above;
	}
	else
	{
		var quote = '[quote=' + pun_quote_authors[qid_param] + ']' + selected_text + '[/quote]';
		quick_post_value[0].value = text_below + quote + text_above;
	}
	setCaretPos(text_below.length + quote.length);
}

Re: OFF: Баг с цитированием

null@example.com (Malcev) — Tue, 11 Apr 2017 12:08:16 +0000

Может просто стирать все возможные строки со смайликами?

Re: OFF: Баг с цитированием

null@example.com (JSmаn) — Tue, 11 Apr 2017 09:04:39 +0000

Malcev
Пока никак, такой алгоритм. Вот адрес скрипта. Функция QuickQuote(qid_param).
Может у вас идеи будут?

Re: OFF: Баг с цитированием

null@example.com (Malcev) — Mon, 10 Apr 2017 17:44:47 +0000

А можно еще функцию подправить?
А то, если цитируешь текст со смайликом, то цитируется весь пост.
Пример со смайликом.